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

Unified Diff: sky/engine/core/script/dart_controller.cc

Issue 936563002: Make tests/clipping/canvas-rounded-corners.sky actually run. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 10 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: sky/engine/core/script/dart_controller.cc
diff --git a/sky/engine/core/script/dart_controller.cc b/sky/engine/core/script/dart_controller.cc
index 73d410f6dd14846a584fac37b39bf5ce02e94fcb..a46fcc256a4803d8cfb571070d373c630f121df7 100644
--- a/sky/engine/core/script/dart_controller.cc
+++ b/sky/engine/core/script/dart_controller.cc
@@ -100,7 +100,9 @@ void DartController::ExecuteModule(RefPtr<AbstractModule> module) {
DCHECK(Dart_CurrentIsolate() == dart_state()->isolate());
DartApiScope dart_api_scope;
- LogIfError(Dart_FinalizeLoading(true));
+ // Don't continue if we failed to load the module.
+ if (LogIfError(Dart_FinalizeLoading(true)))
+ return;
Dart_Handle library = module->library()->dart_value();
const char* name = module->isApplication() ? "main" : "init";
Dart_Handle closure_name = Dart_NewStringFromCString(name);
« no previous file with comments | « sky/engine/core/html/canvas/CanvasRenderingContext2D.idl ('k') | sky/tests/clipping/canvas-rounded-corners.sky » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698