[vm] Fix passing --enable_interpreter in tests.

Also changes an Assert back to a test in Exceptions::JumpToFrame

Change-Id: I3088a342caaf7a1560ba6225ecbdb8bb363ac5fb
Reviewed-on: https://dart-review.googlesource.com/74009
Reviewed-by: Alexander Markov <[email protected]>
Reviewed-by: Régis Crelier <[email protected]>
Commit-Queue: Zach Anderson <[email protected]>

Author: zanderso

runtime/vm/exceptions.cc

@@ -470,8 +470,7 @@ void Exceptions::JumpToFrame(Thread* thread,
   // are callee frames of the C++ frame handling the exception.
   if (FLAG_enable_interpreter) {
     Interpreter* interpreter = thread->isolate()->interpreter();
-    ASSERT(interpreter != NULL);
-    if (interpreter->HasFrame(frame_pointer)) {
+    if ((interpreter != NULL) && interpreter->HasFrame(frame_pointer)) {
       interpreter->JumpToFrame(program_counter, stack_pointer, frame_pointer,
                                thread);
     }

tools/testing/dart/compiler_configuration.dart

@@ -244,7 +244,9 @@ class VMKernelCompilerConfiguration extends CompilerConfiguration
     } else if (_configuration.hotReloadRollback) {
       args.add('--hot-reload-rollback-test-mode');
     }
-
+    if (_configuration.useKernelBytecode) {
+      args.add('--enable_interpreter');
+    }
     return args
       ..addAll(vmOptions)
       ..addAll(sharedOptions)

tools/testing/dart/runtime_configuration.dart

@@ -217,16 +217,11 @@ class StandaloneDartRuntimeConfiguration extends DartVmRuntimeConfiguration {
       throw "Dart VM cannot run files of type '$type'.";
     }
 
-    List<String> args = arguments;
-    if (suite.configuration.compiler == Compiler.dartkb) {
-      args.add('--enable_interpreter');
-    }
-
     String executable = suite.dartVmBinaryFileName;
     if (type == 'application/kernel-ir-fully-linked') {
       executable = suite.dartVmExecutableFileName;
     }
-    return [Command.vm(executable, args, environmentOverrides)];
+    return [Command.vm(executable, arguments, environmentOverrides)];
   }
 }