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

Side by Side Diff: pkg/polymer/lib/src/build/mirrors_remover.dart

Issue 223553008: Only pass an AssetId to isPrimary and declareOutputs. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: fix script_compactor Created 6 years, 8 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, 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 4
5 /// Transformer that removes uses of mirrors from the polymer runtime, so that 5 /// Transformer that removes uses of mirrors from the polymer runtime, so that
6 /// deployed applications are thin and small. 6 /// deployed applications are thin and small.
7 library polymer.src.build.mirrors_remover; 7 library polymer.src.build.mirrors_remover;
8 8
9 import 'dart:async'; 9 import 'dart:async';
10 import 'package:barback/barback.dart'; 10 import 'package:barback/barback.dart';
11 11
12 /// Removes the code-initialization logic based on mirrors. 12 /// Removes the code-initialization logic based on mirrors.
13 class MirrorsRemover extends Transformer { 13 class MirrorsRemover extends Transformer {
14 MirrorsRemover.asPlugin(); 14 MirrorsRemover.asPlugin();
15 15
16 /// Only apply to `lib/polymer.dart`. 16 /// Only apply to `lib/polymer.dart`.
17 Future<bool> isPrimary(Asset input) => new Future.value( 17 // TODO(nweiz): This should just take an AssetId when barback <0.13.0 support
18 input.id.package == 'polymer' && 18 // is dropped.
19 input.id.path == 'lib/polymer.dart'); 19 Future<bool> isPrimary(idOrAsset) {
20 var id = idOrAsset is AssetId ? idOrAsset : idOrAsset.id;
21 return new Future.value(
22 id.package == 'polymer' && id.path == 'lib/polymer.dart');
23 }
20 24
21 Future apply(Transform transform) { 25 Future apply(Transform transform) {
22 var id = transform.primaryInput.id; 26 var id = transform.primaryInput.id;
23 return transform.primaryInput.readAsString().then((code) { 27 return transform.primaryInput.readAsString().then((code) {
24 // Note: this rewrite is highly-coupled with how polymer.dart is 28 // Note: this rewrite is highly-coupled with how polymer.dart is
25 // written. Make sure both are updated in sync. 29 // written. Make sure both are updated in sync.
26 var start = code.indexOf('@MirrorsUsed('); 30 var start = code.indexOf('@MirrorsUsed(');
27 if (start == -1) _error(); 31 if (start == -1) _error();
28 var end = code.indexOf('show MirrorsUsed;', start); 32 var end = code.indexOf('show MirrorsUsed;', start);
29 if (end == -1) _error(); 33 if (end == -1) _error();
30 end = code.indexOf('\n', end); 34 end = code.indexOf('\n', end);
31 var sb = new StringBuffer() 35 var sb = new StringBuffer()
32 ..write(code.substring(0, start)) 36 ..write(code.substring(0, start))
33 ..write(code.substring(end)); 37 ..write(code.substring(end));
34 38
35 transform.addOutput(new Asset.fromString(id, sb.toString())); 39 transform.addOutput(new Asset.fromString(id, sb.toString()));
36 }); 40 });
37 } 41 }
38 } 42 }
39 43
40 /** Transformer phases which should be applied to the smoke package. */ 44 /** Transformer phases which should be applied to the smoke package. */
41 List<List<Transformer>> get phasesForSmoke => 45 List<List<Transformer>> get phasesForSmoke =>
42 [[new MirrorsRemover.asPlugin()]]; 46 [[new MirrorsRemover.asPlugin()]];
43 47
44 _error() => throw new StateError("Couldn't remove imports to mirrors, maybe " 48 _error() => throw new StateError("Couldn't remove imports to mirrors, maybe "
45 "polymer.dart was modified, but mirrors_remover.dart wasn't."); 49 "polymer.dart was modified, but mirrors_remover.dart wasn't.");
OLDNEW
« no previous file with comments | « pkg/polymer/lib/src/build/import_inliner.dart ('k') | pkg/polymer/lib/src/build/polyfill_injector.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698