Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(657)

Unified Diff: runtime/observatory/tests/service/test_helper.dart

Issue 2521413002: Revert "Implement rewind: drop one or more frames from the debugger." (Closed)
Patch Set: Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « runtime/observatory/tests/service/step_over_await_test.dart ('k') | runtime/vm/code_generator.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/observatory/tests/service/test_helper.dart
diff --git a/runtime/observatory/tests/service/test_helper.dart b/runtime/observatory/tests/service/test_helper.dart
index f444bf83be1d207e22a8d6306819c90f974f97a2..9e8caa883ff1aedd8bcb4ba2f3e3ea02daa1a8ce 100644
--- a/runtime/observatory/tests/service/test_helper.dart
+++ b/runtime/observatory/tests/service/test_helper.dart
@@ -90,12 +90,15 @@ class _ServiceTesteeLauncher {
Future<Process> _spawnProcess(bool pause_on_start,
bool pause_on_exit,
bool pause_on_unhandled_exceptions,
+ bool trace_service,
+ bool trace_compiler,
bool testeeControlsServer,
- bool useAuthToken,
- List<String> extraArgs) {
+ bool useAuthToken) {
assert(pause_on_start != null);
assert(pause_on_exit != null);
assert(pause_on_unhandled_exceptions != null);
+ assert(trace_service != null);
+ assert(trace_compiler != null);
assert(testeeControlsServer != null);
assert(useAuthToken != null);
@@ -103,29 +106,39 @@ class _ServiceTesteeLauncher {
return _spawnSkyProcess(pause_on_start,
pause_on_exit,
pause_on_unhandled_exceptions,
- testeeControlsServer,
- extraArgs);
+ trace_service,
+ trace_compiler,
+ testeeControlsServer);
} else {
return _spawnDartProcess(pause_on_start,
pause_on_exit,
pause_on_unhandled_exceptions,
+ trace_service,
+ trace_compiler,
testeeControlsServer,
- useAuthToken,
- extraArgs);
+ useAuthToken);
}
}
Future<Process> _spawnDartProcess(bool pause_on_start,
bool pause_on_exit,
bool pause_on_unhandled_exceptions,
+ bool trace_service,
+ bool trace_compiler,
bool testeeControlsServer,
- bool useAuthToken,
- List<String> extraArgs) {
+ bool useAuthToken) {
assert(!_shouldLaunchSkyShell());
String dartExecutable = Platform.executable;
var fullArgs = [];
+ if (trace_service) {
+ fullArgs.add('--trace-service');
+ fullArgs.add('--trace-service-verbose');
+ }
+ if (trace_compiler) {
+ fullArgs.add('--trace-compiler');
+ }
if (pause_on_start) {
fullArgs.add('--pause-isolates-on-start');
}
@@ -135,9 +148,6 @@ class _ServiceTesteeLauncher {
if (pause_on_unhandled_exceptions) {
fullArgs.add('--pause-isolates-on-unhandled-exceptions');
}
- if (extraArgs != null) {
- fullArgs.addAll(extraArgs);
- }
fullArgs.addAll(Platform.executableArguments);
if (!testeeControlsServer) {
@@ -156,14 +166,22 @@ class _ServiceTesteeLauncher {
Future<Process> _spawnSkyProcess(bool pause_on_start,
bool pause_on_exit,
bool pause_on_unhandled_exceptions,
- bool testeeControlsServer,
- List<String> extraArgs) {
+ bool trace_service,
+ bool trace_compiler,
+ bool testeeControlsServer) {
assert(_shouldLaunchSkyShell());
String dartExecutable = _skyShellPath();
var dartFlags = [];
var fullArgs = [];
+ if (trace_service) {
+ dartFlags.add('--trace_service');
+ dartFlags.add('--trace_service_verbose');
+ }
+ if (trace_compiler) {
+ dartFlags.add('--trace_compiler');
+ }
if (pause_on_start) {
dartFlags.add('--pause_isolates_on_start');
fullArgs.add('--start-paused');
@@ -176,9 +194,6 @@ class _ServiceTesteeLauncher {
}
// Override mirrors.
dartFlags.add('--enable_mirrors=true');
- if (extraArgs != null) {
- fullArgs.addAll(extraArgs);
- }
fullArgs.addAll(Platform.executableArguments);
if (!testeeControlsServer) {
@@ -208,15 +223,17 @@ class _ServiceTesteeLauncher {
Future<Uri> launch(bool pause_on_start,
bool pause_on_exit,
bool pause_on_unhandled_exceptions,
+ bool trace_service,
+ bool trace_compiler,
bool testeeControlsServer,
- bool useAuthToken,
- List<String> extraArgs) {
+ bool useAuthToken) {
return _spawnProcess(pause_on_start,
pause_on_exit,
pause_on_unhandled_exceptions,
+ trace_service,
+ trace_compiler,
testeeControlsServer,
- useAuthToken,
- extraArgs).then((p) {
+ useAuthToken).then((p) {
Completer<Uri> completer = new Completer<Uri>();
process = p;
Uri uri;
@@ -271,11 +288,12 @@ void setupAddresses(Uri serverAddress) {
class _ServiceTesterRunner {
void run({List<String> mainArgs,
- List<String> extraArgs,
List<VMTest> vmTests,
List<IsolateTest> isolateTests,
bool pause_on_start: false,
bool pause_on_exit: false,
+ bool trace_service: false,
+ bool trace_compiler: false,
bool verbose_vm: false,
bool pause_on_unhandled_exceptions: false,
bool testeeControlsServer: false,
@@ -283,8 +301,9 @@ class _ServiceTesterRunner {
var process = new _ServiceTesteeLauncher();
process.launch(pause_on_start, pause_on_exit,
pause_on_unhandled_exceptions,
+ trace_service, trace_compiler,
testeeControlsServer,
- useAuthToken, extraArgs).then((Uri serverAddress) async {
+ useAuthToken).then((Uri serverAddress) async {
if (mainArgs.contains("--gdb")) {
var pid = process.process.pid;
var wait = new Duration(seconds: 10);
@@ -345,11 +364,12 @@ Future runIsolateTests(List<String> mainArgs,
testeeConcurrent(),
bool pause_on_start: false,
bool pause_on_exit: false,
+ bool trace_service: false,
+ bool trace_compiler: false,
bool verbose_vm: false,
bool pause_on_unhandled_exceptions: false,
bool testeeControlsServer: false,
- bool useAuthToken: false,
- List<String> extraArgs}) async {
+ bool useAuthToken: false}) async {
assert(!pause_on_start || testeeBefore == null);
if (_isTestee()) {
new _ServiceTesteeRunner().run(testeeBefore: testeeBefore,
@@ -359,10 +379,11 @@ Future runIsolateTests(List<String> mainArgs,
} else {
new _ServiceTesterRunner().run(
mainArgs: mainArgs,
- extraArgs: extraArgs,
isolateTests: tests,
pause_on_start: pause_on_start,
pause_on_exit: pause_on_exit,
+ trace_service: trace_service,
+ trace_compiler: trace_compiler,
verbose_vm: verbose_vm,
pause_on_unhandled_exceptions: pause_on_unhandled_exceptions,
testeeControlsServer: testeeControlsServer,
@@ -385,9 +406,10 @@ void runIsolateTestsSynchronous(List<String> mainArgs,
void testeeConcurrent(),
bool pause_on_start: false,
bool pause_on_exit: false,
+ bool trace_service: false,
+ bool trace_compiler: false,
bool verbose_vm: false,
- bool pause_on_unhandled_exceptions: false,
- List<String> extraArgs}) {
+ bool pause_on_unhandled_exceptions: false}) {
assert(!pause_on_start || testeeBefore == null);
if (_isTestee()) {
new _ServiceTesteeRunner().runSync(testeeBeforeSync: testeeBefore,
@@ -397,10 +419,11 @@ void runIsolateTestsSynchronous(List<String> mainArgs,
} else {
new _ServiceTesterRunner().run(
mainArgs: mainArgs,
- extraArgs: extraArgs,
isolateTests: tests,
pause_on_start: pause_on_start,
pause_on_exit: pause_on_exit,
+ trace_service: trace_service,
+ trace_compiler: trace_compiler,
verbose_vm: verbose_vm,
pause_on_unhandled_exceptions: pause_on_unhandled_exceptions);
}
@@ -417,9 +440,10 @@ Future runVMTests(List<String> mainArgs,
testeeConcurrent(),
bool pause_on_start: false,
bool pause_on_exit: false,
+ bool trace_service: false,
+ bool trace_compiler: false,
bool verbose_vm: false,
- bool pause_on_unhandled_exceptions: false,
- List<String> extraArgs}) async {
+ bool pause_on_unhandled_exceptions: false}) async {
if (_isTestee()) {
new _ServiceTesteeRunner().run(testeeBefore: testeeBefore,
testeeConcurrent: testeeConcurrent,
@@ -428,10 +452,11 @@ Future runVMTests(List<String> mainArgs,
} else {
new _ServiceTesterRunner().run(
mainArgs: mainArgs,
- extraArgs: extraArgs,
vmTests: tests,
pause_on_start: pause_on_start,
pause_on_exit: pause_on_exit,
+ trace_service: trace_service,
+ trace_compiler: trace_compiler,
verbose_vm: verbose_vm,
pause_on_unhandled_exceptions: pause_on_unhandled_exceptions);
}
« no previous file with comments | « runtime/observatory/tests/service/step_over_await_test.dart ('k') | runtime/vm/code_generator.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698