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

Unified Diff: bin/dwc_browser.dart

Issue 11450020: (Fix #215) better error printing in editor & extension, adds mapping for editor (Closed) Base URL: git@github.com:dart-lang/dart-web-components.git@master
Patch Set: Created 8 years 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 | « no previous file | extension/background.js » ('j') | extension/content_script.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: bin/dwc_browser.dart
diff --git a/bin/dwc_browser.dart b/bin/dwc_browser.dart
index 157a0591fa0f8dbf15c8c6a054712c4caba0c720..720aea9eb569205ab40a8a0dc37dfd8fe3e7d5d6 100644
--- a/bin/dwc_browser.dart
+++ b/bin/dwc_browser.dart
@@ -50,6 +50,10 @@ void parse(js.Proxy sourcePagePort, String sourceUri) {
fileSystem.writeString(file.path, file.contents);
}
var ret = fileSystem.flush();
+ js.scoped(() {
+ js.context.proxyMessages(sourcePagePort,
Jennifer Messerly 2012/12/06 04:00:08 out of curiousity, can you inline proxyMessages?
Siggi Cherem (dart-lang) 2012/12/06 18:46:42 yes, I believe so, I wasn't sure if 'map' worked o
+ js.array(messages.messages.map((m) => m.toString())));
+ });
js.release(sourcePagePort);
return ret;
});
« no previous file with comments | « no previous file | extension/background.js » ('j') | extension/content_script.js » ('J')

Powered by Google App Engine
This is Rietveld 408576698