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

Unified Diff: pkg/compiler/lib/src/js_emitter/program_builder.dart

Issue 868473003: dart2js: Run prepareNativeClasses in the program builder. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 5 years, 11 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: pkg/compiler/lib/src/js_emitter/program_builder.dart
diff --git a/pkg/compiler/lib/src/js_emitter/program_builder.dart b/pkg/compiler/lib/src/js_emitter/program_builder.dart
index 4e2f55577e1f5162fdeb4978ad9e506d378751da..d5a24a678e14407c6fbfa662779f5a921a5e10d4 100644
--- a/pkg/compiler/lib/src/js_emitter/program_builder.dart
+++ b/pkg/compiler/lib/src/js_emitter/program_builder.dart
@@ -101,12 +101,23 @@ class ProgramBuilder {
}
});
+ Map<Class, Map<String, js.Expression>> additionalProperties =
+ new Map<Class, Map<String, js.Expression>>();
+
+ List<Class> nativeNeededClasses =
herhut 2015/01/26 13:50:15 This reads strange. Why not neededNativeClasses?
floitsch 2015/01/26 17:50:40 Done.
+ _task.nativeEmitter.prepareNativeClasses(
+ nativeClasses,
+ additionalProperties);
+
_markEagerClasses();
- return new Program(outputs,
- nativeClasses,
- _task.outputContainsConstantList,
- _buildLoadMap());
+ return new Program(
+ outputs,
+ nativeNeededClasses,
+ additionalProperties,
+ _buildLoadMap(),
+ outputContainsNativeClasses: nativeNeededClasses.isNotEmpty,
+ outputContainsConstantList: _task.outputContainsConstantList);
}
void _markEagerClasses() {

Powered by Google App Engine
This is Rietveld 408576698