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

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

Issue 2759973004: Fix observatory tests broken by running dartfmt. Temporarily reverted formatting for evaluate_activ… (Closed)
Patch Set: Created 3 years, 9 months 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
Index: runtime/observatory/tests/service/rewind_optimized_out_test.dart
diff --git a/runtime/observatory/tests/service/rewind_optimized_out_test.dart b/runtime/observatory/tests/service/rewind_optimized_out_test.dart
index d6a246d939a72f2676c2a39e346414874b454f75..653b92e92b4e8ac2f3c9f6bf9f5da37aac985109 100644
--- a/runtime/observatory/tests/service/rewind_optimized_out_test.dart
+++ b/runtime/observatory/tests/service/rewind_optimized_out_test.dart
@@ -32,26 +32,25 @@ b3(x) {
if (global >= 100) {
debugger();
}
- global = global + 1; // Line A
+ global = global + 1; // Line A
return sum;
}
@alwaysInline
-b2(x) => b3(x); // Line B
+b2(x) => b3(x); // Line B
@alwaysInline
-b1(x) => b2(x); // Line C
+b1(x) => b2(x); // Line C
test() {
while (true) {
- b1(10000); // Line D
+ b1(10000); // Line D
}
}
var tests = [
hasStoppedAtBreakpoint,
stoppedAtLine(LINE_A),
-
(Isolate isolate) async {
// We are at our breakpoint with global=100.
var result = await isolate.rootLibrary.evaluate('global');
@@ -63,25 +62,25 @@ var tests = [
bool caughtException;
try {
result = await isolate.rewind(1);
- expect(false, isTrue, reason:'Unreachable');
- } on ServerRpcException catch(e) {
+ expect(false, isTrue, reason: 'Unreachable');
+ } on ServerRpcException catch (e) {
caughtException = true;
expect(e.code, equals(ServerRpcException.kCannotResume));
- expect(e.message,
- startsWith('Cannot rewind to frame 1 due to conflicting compiler '
- 'optimizations. Run the vm with --no-prune-dead-locals '
- 'to disallow these optimizations. Next valid rewind '
- 'frame is '));
+ expect(
+ e.message,
+ startsWith('Cannot rewind to frame 1 due to conflicting compiler '
+ 'optimizations. Run the vm with --no-prune-dead-locals '
+ 'to disallow these optimizations. Next valid rewind '
+ 'frame is '));
}
expect(caughtException, isTrue);
},
];
-
-main(args) => runIsolateTests(args, tests, testeeConcurrent: test,
- extraArgs:
- ['--trace-rewind',
- '--prune-dead-locals',
- '--enable-inlining-annotations',
- '--no-background-compilation',
- '--optimization-counter-threshold=10']);
+main(args) => runIsolateTests(args, tests, testeeConcurrent: test, extraArgs: [
+ '--trace-rewind',
+ '--prune-dead-locals',
+ '--enable-inlining-annotations',
+ '--no-background-compilation',
+ '--optimization-counter-threshold=10'
+ ]);
« no previous file with comments | « runtime/observatory/tests/service/reload_sources_test.dart ('k') | runtime/observatory/tests/service/rewind_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698