Polymer transformer logs now show on the frontend for pub serve.

pkg/polymer/lib/src/build/log_combiner.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.
+
+/// Logic to combine all the ._buildLog.* logs into one ._buildLog file.
+library polymer.src.build.log_combiner;
+
+import 'dart:async';
+
+import 'package:barback/barback.dart';
+
+import 'common.dart';
+import 'wrapped_logger.dart';
+
+class LogCombiner extends Transformer with PolymerTransformer {

[Siggi Cherem (dart-lang), 2014/08/01 21:31:52]

(a) nit: let's add a Dart doc here too. Might be the same that you have in the
library.

(b) maybe rename to BuildLogCombiner? or MessagesCombiner, or MessagesMerger?

The main thing is that 'log' has the same name for runtime and build-time logs.
I commonly think of 'log' of the runtime logs, and not the build-time warning
and error messages. So it would be nice to make the distinction.

[jakemac, 2014/08/04 19:49:58]

Done.

+

[Siggi Cherem (dart-lang), 2014/08/01 21:31:52]

nit: remove empty line

[jakemac, 2014/08/04 19:49:58]

Done.

+  final TransformOptions options;
+
+  LogCombiner(this.options);
+
+  // Run only on entry point html files in !releaseMode.
+  Future<bool> isPrimary(idOrAsset) {

[Siggi Cherem (dart-lang), 2014/08/01 21:31:52]

same here: change to 'bool', and remove new Future.value() below?

[jakemac, 2014/08/04 19:49:58]

Done.

+    if (options.releaseMode) return false;
+    var id = idOrAsset is AssetId ? idOrAsset : idOrAsset.id;
+    return new Future.value(options.isHtmlEntryPoint(id));
+  }
+
+  Future apply(Transform transform) {
+    return WrappedLogger.combineLogFiles(transform);
+  }
+

[Siggi Cherem (dart-lang), 2014/08/01 21:31:52]

nit: remove empty line

[jakemac, 2014/08/04 19:49:58]

Done.

+}