merge to trunk all changes from 36817 until 37378 under the packages: polymer,

pkg/polymer/lib/transformer.dart

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 /// Transfomer used for pub-serve and pub-deploy.
 library polymer.transformer;
 
 import 'package:barback/barback.dart';
 import 'package:observe/transformer.dart';
 
@@ skipping 21 matching lines @@
 
   PolymerTransformerGroup.asPlugin(BarbackSettings settings)
       : this(_parseSettings(settings));
 }
 
 TransformOptions _parseSettings(BarbackSettings settings) {
   var args = settings.configuration;
   bool releaseMode = settings.mode == BarbackMode.RELEASE;
   bool jsOption = args['js'];
   bool csp = args['csp'] == true; // defaults to false
+  bool lint = args['lint'] != false; // defaults to true
   return new TransformOptions(
       entryPoints: _readEntrypoints(args['entry_points']),
       directlyIncludeJS: jsOption == null ? releaseMode : jsOption,
       contentSecurityPolicy: csp,
-      releaseMode: releaseMode);
+      releaseMode: releaseMode,
+      lint: lint);
 }
 
 _readEntrypoints(value) {
   if (value == null) return null;
   var entryPoints = [];
   bool error;
   if (value is List) {
     entryPoints = value;
     error = value.any((e) => e is! String);
   } else if (value is String) {
     entryPoints = [value];
     error = false;
   } else {
     error = true;
   }
   if (error) {
     print('Invalid value for "entry_points" in the polymer transformer.');
   }
   return entryPoints;
 }
 
 /// Create deploy phases for Polymer. Note that inlining HTML Imports
 /// comes first (other than linter, if [options.linter] is enabled), which
 /// allows the rest of the HTML-processing phases to operate only on HTML that
 /// is actually imported.
 List<List<Transformer>> createDeployPhases(
     TransformOptions options, {String sdkDir}) {
+  // TODO(sigmund): this should be done differently. We should lint everything
+  // that is reachable and have the option to lint the rest (similar to how
+  // dart2js can analyze reachable code or entire libraries).
   var phases = options.lint ? [[new Linter(options)]] : [];
   return phases..addAll([
     [new ImportInliner(options)],
     [new ObservableTransformer()],
     [new ScriptCompactor(options, sdkDir: sdkDir)],
     [new PolyfillInjector(options)],
     [new BuildFilter(options)]
   ]);
 }