Bring back initPolymer, allow boot.js only if using "polymer_experimental.html".

pkg/polymer/lib/src/mirror_loader.dart

 // Copyright (c) 2014, 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.
 
 /// Contains logic to initialize polymer apps during development. This
 /// implementation uses dart:mirrors to load each library as they are discovered
 /// through HTML imports. This is only meant to be during development in
 /// dartium, and the polymer transformers replace this implementation with
 /// codege generation in the polymer-build steps.
 library polymer.src.mirror_loader;
 
 import 'dart:async';
 import 'dart:html';
 import 'dart:collection' show LinkedHashMap;
 
 // Technically, we shouldn't need any @MirrorsUsed, since this is for
 // development only, but our test bots don't yet run pub-build. See more details
 // on the comments of the mirrors import in `lib/polymer.dart`.
 @MirrorsUsed(metaTargets:
     const [CustomTag, InitMethodAnnotation],
     override: const ['smoke.mirrors', 'polymer.src.mirror_loader'])
 import 'dart:mirrors';
 
 import 'package:logging/logging.dart' show Logger;
+import 'package:observe/src/dirty_check.dart';
+import 'package:polymer/boot.dart' show discoverScripts, ScriptInfo;

[Siggi Cherem (dart-lang), 2014/05/22 21:10:41]

Q: I wonder if I should just move the code that is left in boot.dart here. I
kept it there to make the review simpler too.

[Jennifer Messerly, 2014/05/23 07:20:06]

I guess the main reason is if you want it to be public? Otherwise I would move
it here.

[Siggi Cherem (dart-lang), 2014/05/23 17:56:33]

Done. Moved it here. At least now it is structured in a way that it can be
imported more easily if we rewrite the boot.js as boot.dart in the future.

 import 'package:polymer/polymer.dart';
-import 'package:observe/src/dirty_check.dart';
 
 
+/// Used by code generated from the experimental polymer bootstrap in boot.js.
 void startPolymerInDevelopment(List<String> librariesToLoad) {
   dirtyCheckZone()..run(() {
     startPolymer(discoverInitializers(librariesToLoad), false);
   });
 }
 
 /// Set of initializers that are invoked by `initPolymer`.  This is computed the
 /// list by crawling HTML imports, searching for script tags, and including an
 /// initializer for each type tagged with a [CustomTag] annotation and for each
 /// top-level method annotated with [initMethod].
+List<Function> initializers = discoverInitializers(
+    discoverLibrariesToLoad(document, window.location.href));
+
+/// True if we're in deployment mode.
+bool deployMode = false;
 
 /// Discovers what script tags are loaded from HTML pages and collects the
 /// initializers of their corresponding libraries.
 // Visible for testing only.
-List<Function> discoverInitializers(List<String> librariesToLoad) {
+List<Function> discoverInitializers(Iterable<String> librariesToLoad) {
   var initializers = [];
   for (var lib in librariesToLoad) {
     try {
       _loadLibrary(lib, initializers);
     } catch (e, s) {
       // Deliver errors async, so if a single library fails it doesn't prevent
       // other things from loading.
       new Completer().completeError(e, s);
     }
   }
   return initializers;
 }
 
+Iterable<String> discoverLibrariesToLoad(Document doc, String baseUri) =>
+    discoverScripts(doc, baseUri).map(
+        (info) => _hasPackageUrl(info) ? info.packageUrl : info.resolvedUrl);
+
+bool _hasPackageUrl(ScriptInfo info) =>
+    info.isPackage && _libs[Uri.parse(info.packageUrl)] != null;
+
 /// All libraries in the current isolate.
 final _libs = currentMirrorSystem().libraries;
 
 final Logger _loaderLog = new Logger('polymer.src.mirror_loader');
 
 /// Reads the library at [uriString] (which can be an absolute URI or a relative
 /// URI from the root library), and:
 ///
 ///   * If present, invokes any top-level and static functions marked
 ///     with the [initMethod] annotation (in the order they appear).
@@ skipping 86 matching lines @@
         " ${method.simpleName} is not.");
     return;
   }
   if (!method.parameters.where((p) => !p.isOptional).isEmpty) {
     print("warning: methods marked with @initMethod should take no "
         "arguments, ${method.simpleName} expects some.");
     return;
   }
   initializers.add(() => obj.invoke(method.simpleName, const []));
 }