Chromium Code Reviews| OLD | NEW |
|---|---|
| (Empty) | |
| 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 | |
| 3 // BSD-style license that can be found in the LICENSE file. | |
| 4 | |
| 5 /// Transformer that replaces the default mirror-based implementation of smoke, | |
| 6 /// so that during deploy smoke doesn't include any dependency on dart:mirrors. | |
| 7 library smoke.src.default_transformer; | |
| 8 | |
| 9 import 'dart:async'; | |
| 10 import 'package:barback/barback.dart'; | |
| 11 | |
| 12 /// Replaces the default mirror-based implementation of smoke in | |
| 13 /// `pacakge:smoke/implementation.dart`, so that during deploy smoke doesn't | |
| 14 /// include any dependency on dart:mirrors. | |
| 15 // TODO(sigmund): include tests that run this transformation automatically. | |
| 16 class DefaultTransformer extends Transformer { | |
| 17 DefaultTransformer.asPlugin(); | |
| 18 | |
| 19 /// Only apply to `lib/src/implementation.dart`. | |
| 20 Future<bool> isPrimary(Asset input) => new Future.value( | |
| 21 input.id.package == 'smoke' && | |
| 22 input.id.path == 'lib/src/implementation.dart'); | |
| 23 | |
| 24 Future apply(Transform transform) { | |
| 25 var id = transform.primaryInput.id; | |
| 26 return transform.primaryInput.readAsString().then((code) { | |
| 27 // Note: this rewrite is highly-coupled with how implementation.dart is | |
| 28 // written. Make sure both are updated in sync. | |
| 29 transform.addOutput(new Asset.fromString(id, code | |
| 30 .replaceAll(new RegExp('new Reflective[^;]*;'), | |
| 31 'throwNotConfiguredError();') | |
| 32 .replaceAll("import 'package:smoke/mirrors.dart'';", ''))); | |
|
Jennifer Messerly
2014/02/20 21:24:47
is there an extra ' here? mirrors.dart''
Siggi Cherem (dart-lang)
2014/02/20 22:19:48
good catch! I had renamed the library and when I f
| |
| 33 }); | |
| 34 } | |
| 35 } | |
| 36 | |
| OLD | NEW |