Improves how to initialize polymer apps and fixes polymer build and linter to

Improves how to initialize polymer apps and fixes polymer build and linter to
match Dartium semantics (Issue 14238). In particular:
  - only one script tag is allowed per document
  - initPolymer can be call from init.dart or the users main method, the build
    process knows how to call the user's main given the new Dartium semantics.

R=jmesserly@google.com

Committed: https://code.google.com/p/dart/source/detail?r=28971

[Siggi Cherem (dart-lang), 2013-10-19 00:01:14]

FYI - I just did one more upload, the only change is in pkg.status - I'm still
investigating prop_attr_reflection_test. It seems now to always be failing for
me in ff/chrome.

I think it's related to what you saw in your change.

[Jennifer Messerly, 2013-10-19 00:15:47]

the transformer changes are fine... but I'm worried about the test changes. The
single-script-tag-per-document messes up the Polymer programming model a bit
more than I'd expected. Let's follow up offline on that issue.

https://codereview.chromium.org/27903006/diff/30001/pkg/polymer/lib/src/build...
File pkg/polymer/lib/src/build/import_inliner.dart (right):

https://codereview.chromium.org/27903006/diff/30001/pkg/polymer/lib/src/build...
pkg/polymer/lib/src/build/import_inliner.dart:63: // Split Dart script tags from
all the other elements
maybe expand on the comment here about why we're doing this?
(your explanation in person was awesome :) )

[Jennifer Messerly, 2013-10-19 00:28:51]

fwiw, code changes lgtm. Up to you if you think this is an improvement or not
over the current polymer/init.dart story.

[Siggi Cherem (dart-lang), 2013-10-19 00:33:14]

+vsm

totally fair. I have similar concerns.

Vijay - I think we are going to need to meet up again on Monday and chat a bit
more about single-script tag vs multiple-script tags on the page.

[Siggi Cherem (dart-lang), 2013-10-21 20:43:14]

just to write here what our conclusion was: we decided to go ahead with this
approach for 1.0 and keep iterating afterwards. There are a lot of design
questions re: multiple tags in Dartium that we will want to address in the near
future.

https://codereview.chromium.org/27903006/diff/30001/pkg/polymer/lib/src/build...
File pkg/polymer/lib/src/build/import_inliner.dart (right):

https://codereview.chromium.org/27903006/diff/30001/pkg/polymer/lib/src/build...
pkg/polymer/lib/src/build/import_inliner.dart:63: // Split Dart script tags from
all the other elements
On 2013/10/19 00:15:47, John Messerly wrote:
> maybe expand on the comment here about why we're doing this?
> (your explanation in person was awesome :) )

Done.

[Siggi Cherem (dart-lang), 2013-10-21 23:29:39]

John - I just merged your latest changes and will be submitting shortly.

The only noticeable change from the previous iteration is that instead of using
a setter for 'librariesToLoad' I added a method 'configureForDeployment' that
additionally sets an internal flag to not do dirty checking.

[Siggi Cherem (dart-lang), 2013-10-21 23:31:00]

Committed patchset #5 manually as r28971 (presubmit successful).