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

Side by Side Diff: pkg/kernel/lib/target/flutter.dart

Issue 2912503003: Perform MixinFullResolution only for source libraries. (Closed)
Patch Set: Assert that only classes that declared in the libraries to be transformed are transformed. Created 3 years, 6 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 unified diff | Download patch
« no previous file with comments | « pkg/kernel/bin/transform.dart ('k') | pkg/kernel/lib/target/vm.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2016, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2016, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 library kernel.target.flutter; 4 library kernel.target.flutter;
5 5
6 import '../ast.dart'; 6 import '../ast.dart';
7 import '../transformations/continuation.dart' as cont; 7 import '../transformations/continuation.dart' as cont;
8 import '../transformations/erasure.dart'; 8 import '../transformations/erasure.dart';
9 import '../transformations/sanitize_for_vm.dart'; 9 import '../transformations/sanitize_for_vm.dart';
10 import '../transformations/mixin_full_resolution.dart' as mix; 10 import '../transformations/mixin_full_resolution.dart' as mix;
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
43 'dart:_builtin', 43 'dart:_builtin',
44 'dart:nativewrappers', 44 'dart:nativewrappers',
45 'dart:io', 45 'dart:io',
46 46
47 // Required for flutter. 47 // Required for flutter.
48 'dart:ui', 48 'dart:ui',
49 'dart:vmservice_sky', 49 'dart:vmservice_sky',
50 ]; 50 ];
51 51
52 void performModularTransformations(Program program) { 52 void performModularTransformations(Program program) {
53 new mix.MixinFullResolution(this).transform(program); 53 mix.transformLibraries(this, program.libraries);
54 } 54 }
55 55
56 void performGlobalTransformations(Program program) { 56 void performGlobalTransformations(Program program) {
57 cont.transformProgram(program); 57 cont.transformProgram(program);
58 58
59 // Repair `_getMainClosure()` function in dart:{_builtin,ui} libraries. 59 // Repair `_getMainClosure()` function in dart:{_builtin,ui} libraries.
60 setup_builtin_library.transformProgram(program); 60 setup_builtin_library.transformProgram(program);
61 setup_builtin_library.transformProgram(program, libraryUri: 'dart:ui'); 61 setup_builtin_library.transformProgram(program, libraryUri: 'dart:ui');
62 62
63 if (strongMode) { 63 if (strongMode) {
64 new Erasure().transform(program); 64 new Erasure().transform(program);
65 } 65 }
66 66
67 new SanitizeForVM().transform(program); 67 new SanitizeForVM().transform(program);
68 } 68 }
69 69
70 @override 70 @override
71 Expression instantiateInvocation(Member target, Expression receiver, 71 Expression instantiateInvocation(Member target, Expression receiver,
72 String name, Arguments arguments, int offset, bool isSuper) { 72 String name, Arguments arguments, int offset, bool isSuper) {
73 // TODO(ahe): This should probably return the same as VmTarget does. 73 // TODO(ahe): This should probably return the same as VmTarget does.
74 return new InvalidExpression(); 74 return new InvalidExpression();
75 } 75 }
76 } 76 }
OLDNEW
« no previous file with comments | « pkg/kernel/bin/transform.dart ('k') | pkg/kernel/lib/target/vm.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698