| Index: pkg/polymer/lib/src/build/import_inliner.dart | 
| diff --git a/pkg/polymer/lib/src/build/import_inliner.dart b/pkg/polymer/lib/src/build/import_inliner.dart | 
| index d80adc11b6e6af52551e9a37631c8bdbc4585bd1..40d5ca58d197a51ebea3106ccf5fb457baad3470 100644 | 
| --- a/pkg/polymer/lib/src/build/import_inliner.dart | 
| +++ b/pkg/polymer/lib/src/build/import_inliner.dart | 
| @@ -2,7 +2,7 @@ | 
| // 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 that inlines polymer-element definitions from html imports. */ | 
| +/// Transfomer that inlines polymer-element definitions from html imports. | 
| library polymer.src.build.import_inliner; | 
|  | 
| import 'dart:async'; | 
| @@ -55,14 +55,12 @@ class _HtmlInliner extends PolymerTransformer { | 
| })); | 
| } | 
|  | 
| -  /** | 
| -   * Visits imports in [document] and add the imported documents to documents. | 
| -   * Documents are added in the order they appear, transitive imports are added | 
| -   * first. | 
| -   * | 
| -   * Returns `true` if and only if the document was changed and should be | 
| -   * written out. | 
| -   */ | 
| +  /// Visits imports in [document] and add the imported documents to documents. | 
| +  /// Documents are added in the order they appear, transitive imports are added | 
| +  /// first. | 
| +  /// | 
| +  /// Returns `true` if and only if the document was changed and should be | 
| +  /// written out. | 
| Future<bool> _visitImports(Document document, AssetId sourceId) { | 
| bool changed = false; | 
|  | 
| @@ -94,15 +92,13 @@ class _HtmlInliner extends PolymerTransformer { | 
| }).then((_) => changed); | 
| } | 
|  | 
| -  /** | 
| -   * To preserve the order of scripts with respect to inlined | 
| -   * link rel=import, we move both of those into the body before we do any | 
| -   * inlining. | 
| -   * | 
| -   * Note: we do this for stylesheets as well to preserve ordering with | 
| -   * respect to eachother, because stylesheets can be pulled in transitively | 
| -   * from imports. | 
| -   */ | 
| +  /// To preserve the order of scripts with respect to inlined | 
| +  /// link rel=import, we move both of those into the body before we do any | 
| +  /// inlining. | 
| +  /// | 
| +  /// Note: we do this for stylesheets as well to preserve ordering with | 
| +  /// respect to eachother, because stylesheets can be pulled in transitively | 
| +  /// from imports. | 
| // TODO(jmesserly): vulcanizer doesn't need this because they inline JS | 
| // scripts, causing them to be naturally moved as part of the inlining. | 
| // Should we do the same? Alternatively could we inline head into head and | 
| @@ -146,12 +142,10 @@ class _HtmlInliner extends PolymerTransformer { | 
| }); | 
| } | 
|  | 
| -  /** | 
| -   * Split Dart script tags from all the other elements. Now that Dartium | 
| -   * only allows a single script tag per page, we can't inline script | 
| -   * tags. Instead, we collect the urls of each script tag so we import | 
| -   * them directly from the Dart bootstrap code. | 
| -   */ | 
| +  /// Split Dart script tags from all the other elements. Now that Dartium | 
| +  /// only allows a single script tag per page, we can't inline script | 
| +  /// tags. Instead, we collect the urls of each script tag so we import | 
| +  /// them directly from the Dart bootstrap code. | 
| void _extractScripts(Document document) { | 
| bool first = true; | 
| for (var script in document.querySelectorAll('script')) { | 
| @@ -182,21 +176,20 @@ class _HtmlInliner extends PolymerTransformer { | 
| } | 
| } | 
|  | 
| -/** | 
| - * Recursively inlines the contents of HTML imports. Produces as output a single | 
| - * HTML file that inlines the polymer-element definitions, and a text file that | 
| - * contains, in order, the URIs to each library that sourced in a script tag. | 
| - * | 
| - * This transformer assumes that all script tags point to external files. To | 
| - * support script tags with inlined code, use this transformer after running | 
| - * [InlineCodeExtractor] on an earlier phase. | 
| - */ | 
| +/// Recursively inlines the contents of HTML imports. Produces as output a | 
| +/// single HTML file that inlines the polymer-element definitions, and a text | 
| +/// file that contains, in order, the URIs to each library that sourced in a | 
| +/// script tag. | 
| +/// | 
| +/// This transformer assumes that all script tags point to external files. To | 
| +/// support script tags with inlined code, use this transformer after running | 
| +/// [InlineCodeExtractor] on an earlier phase. | 
| class ImportInliner extends Transformer { | 
| final TransformOptions options; | 
|  | 
| ImportInliner(this.options); | 
|  | 
| -  /** Only run on entry point .html files. */ | 
| +  /// Only run on entry point .html files. | 
| Future<bool> isPrimary(Asset input) => | 
| new Future.value(options.isHtmlEntryPoint(input.id)); | 
|  | 
| @@ -205,11 +198,11 @@ class ImportInliner extends Transformer { | 
| } | 
|  | 
|  | 
| -/** Internally adjusts urls in the html that we are about to inline. */ | 
| +/// Internally adjusts urls in the html that we are about to inline. | 
| class _UrlNormalizer extends TreeVisitor { | 
| final Transform transform; | 
|  | 
| -  /** Asset where the original content (and original url) was found. */ | 
| +  /// Asset where the original content (and original url) was found. | 
| final AssetId sourceId; | 
|  | 
| _UrlNormalizer(this.transform, this.sourceId); | 
| @@ -229,7 +222,7 @@ class _UrlNormalizer extends TreeVisitor { | 
| static final _URL = new RegExp(r'url\(([^)]*)\)', multiLine: true); | 
| static final _QUOTE = new RegExp('["\']', multiLine: true); | 
|  | 
| -  /** Visit the CSS text and replace any relative URLs so we can inline it. */ | 
| +  /// Visit the CSS text and replace any relative URLs so we can inline it. | 
| // Ported from: | 
| // https://github.com/Polymer/vulcanize/blob/c14f63696797cda18dc3d372b78aa3378acc691f/lib/vulcan.js#L149 | 
| // TODO(jmesserly): use csslib here instead? Parsing with RegEx is sadness. | 
| @@ -278,13 +271,11 @@ class _UrlNormalizer extends TreeVisitor { | 
| } | 
| } | 
|  | 
| -/** | 
| - * HTML attributes that expect a URL value. | 
| - * <http://dev.w3.org/html5/spec/section-index.html#attributes-1> | 
| - * | 
| - * Every one of these attributes is a URL in every context where it is used in | 
| - * the DOM. The comments show every DOM element where an attribute can be used. | 
| - */ | 
| +/// HTML attributes that expect a URL value. | 
| +/// <http://dev.w3.org/html5/spec/section-index.html#attributes-1> | 
| +/// | 
| +/// Every one of these attributes is a URL in every context where it is used in | 
| +/// the DOM. The comments show every DOM element where an attribute can be used. | 
| const _urlAttributes = const [ | 
| 'action',     // in form | 
| 'background', // in body | 
|  |