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

Unified Diff: pkg/kernel/lib/target/flutter.dart

Issue 2671653003: Split the Kernel transformations into modular and global ones (Closed)
Patch Set: Revert inadvertent change Created 3 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
« no previous file with comments | « pkg/kernel/bin/dartk.dart ('k') | pkg/kernel/lib/target/targets.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/kernel/lib/target/flutter.dart
diff --git a/pkg/kernel/lib/target/flutter.dart b/pkg/kernel/lib/target/flutter.dart
index 5673a22a570145ed89a58c602e3bdc8bacc9c529..4ba79644c2b45edd71f4360bb881edd12fc4ffb7 100644
--- a/pkg/kernel/lib/target/flutter.dart
+++ b/pkg/kernel/lib/target/flutter.dart
@@ -50,8 +50,11 @@ class FlutterTarget extends Target {
'dart:vmservice_sky',
];
- void transformProgram(Program program) {
+ void performModularTransformations(Program program) {
new mix.MixinFullResolution().transform(program);
+ }
+
+ void performGlobalTransformations(Program program) {
asgerf 2017/02/02 11:46:00 The transformations here are not global, I think.
Kevin Millikin (Google) 2017/02/02 12:03:48 They're just not rewritten in this change to skip
cont.transformProgram(program);
// Repair `_getMainClosure()` function in dart:{_builtin,ui} libraries.
« no previous file with comments | « pkg/kernel/bin/dartk.dart ('k') | pkg/kernel/lib/target/targets.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698