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

Unified Diff: pkg/polymer/lib/deploy.dart

Issue 157583007: Removing polymer-specific code from barback runner (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 10 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/polymer/lib/deploy.dart
diff --git a/pkg/polymer/lib/deploy.dart b/pkg/polymer/lib/deploy.dart
index d3e41a2bde0fbe1b0411f862767ec642aa424119..d2c2c17a42377fbf5f6e1015026c9106632cf383 100644
--- a/pkg/polymer/lib/deploy.dart
+++ b/pkg/polymer/lib/deploy.dart
@@ -44,7 +44,9 @@ main(List<String> arguments) {
directlyIncludeJS: args['js'],
contentSecurityPolicy: args['csp'],
releaseMode: !args['debug']);
- options = new BarbackOptions(createDeployPhases(transformOps), outDir);
+ var phases = createDeployPhases(transformOps);
+ options = new BarbackOptions(phases, outDir,
Siggi Cherem (dart-lang) 2014/02/11 23:15:20 it's fine in it's current form, but I wonder if we
blois 2014/02/12 02:39:27 Yeah- I was considering removing the phases comple
Siggi Cherem (dart-lang) 2014/02/12 03:05:28 sounds good
+ packagePhases: {'polymer': phases});
} else {
options = _createTestOptions(
test, outDir, args['js'], args['csp'], !args['debug']);
@@ -84,6 +86,7 @@ BarbackOptions _createTestOptions(String testFile, String outDir,
return new BarbackOptions(phases, outDir,
currentPackage: packageName,
packageDirs: {packageName : pubspecDir},
+ packagePhases: {'polymer': phases},
transformTests: true);
}

Powered by Google App Engine
This is Rietveld 408576698