Adjust URL during inlining more selectively

[DartBot]

<img src="https://avatars.githubusercontent.com/u/2049220?v=3" align="left" width="96" height="96"hspace="10"> Issue by sigmundch
Originally opened as dart-lang/sdk#19770

---

The list indicates not just which attributes to filter, but on what elements:
https://code.google.com/p/dart/codesearch#dart/trunk/dart/pkg/polymer/lib/src/build/import_inliner.dart&amp;l=406

The inliner works fine with normal DOM elements because those attributes are not used for other purposes, but if some custom element happens to use the same attribute names, we shouldn't try to fix it.

See discussion here:

https://groups.google.com/a/dartlang.org/d/msgid/web/6b798af4-d5d3-4825-8397-1fe8d6f786eb%40dartlang.org?utm_medium=email&amp;utm_source=footer

[DartBot]

<img src="https://avatars.githubusercontent.com/u/2049220?v=3" align="left" width="48" height="48"hspace="10"> Comment by sigmundch

---

Removed Pkg-Polymer label.
Added Pkg-PolymerBuild label.

[DartBot]

<img src="https://avatars.githubusercontent.com/u/2049220?v=3" align="left" width="48" height="48"hspace="10"> Comment by sigmundch

---

Issue #167 has been merged into this issue.

[DartBot]

<img src="https://avatars.githubusercontent.com/u/2049220?v=3" align="left" width="48" height="48"hspace="10"> Comment by sigmundch

---

Set owner to @jakemac53.

[DartBot]

<img src="https://avatars.githubusercontent.com/u/2049220?v=3" align="left" width="48" height="48"hspace="10"> Comment by sigmundch

---

Removed this from the 1.6 milestone.
Added Polymer-P-1 label.

[DartBot]

<img src="https://avatars.githubusercontent.com/u/2049220?v=3" align="left" width="48" height="48"hspace="10"> Comment by sigmundch

---

Removed Polymer-P-1 label.
Added Polymer-Milestone-Next label.

[DartBot]

<img src="https://avatars.githubusercontent.com/u/2049220?v=3" align="left" width="48" height="48"hspace="10"> Comment by sigmundch

---

Added PolymerMilestone-Next label.

[DartBot]

<img src="https://avatars.githubusercontent.com/u/2049220?v=3" align="left" width="48" height="48"hspace="10"> Comment by sigmundch

---

Removed Polymer-Milestone-Next label.

[DartBot]

<img src="https://avatars.githubusercontent.com/u/2049220?v=3" align="left" width="48" height="48"hspace="10"> Comment by sigmundch

---

Issue #227 has been merged into this issue.

[DartBot]

<img src="https://avatars.githubusercontent.com/u/984921?v=3" align="left" width="48" height="48"hspace="10"> Comment by jakemac53

---

dart-lang/sdk@4b85b6a changes this so it will only affect attributes on standard html elements.

---

Added Fixed label.

[DartBot]

This comment was originally written by jan.johanss...@gmail.com

---

I was experiencing the same exception when on polymer version >=0.11.0 <0.12.0. When using the version >=0.12.0 it works fine in Dartium, but throws a new exception in Chrome: "Uncaught TypeError: Cannot read property 'element' of undefined"

That is, after the fix, the code runs good in dartium but blows up in chrome after compiling the solution to java script.

By inspecting the element in chrome, I got:

'Resource interpreted as Script but transferred with MIME type application/dart'

With entry point (after a chain of anonymous functions):

  'isolate_helper.dart:97'

and the blow up at:

  'isolate_helper.dart:438'

I also got this message:

'flushing 26 elements' at 'polymer.js:26' (I don't have a clue to what that means)

Some more info:

If I create my polymer element (that itself depends on core icons) in the web-root directory, then everything works fine and out of the box with polymer version >=0.11.0 <0.12.0. If I move my element to a directory in the web-root directory (i.e. components), then the element blows up (as described by this thread). Upgrading to the new polymer version >=0.12.0 fixes the problem in dartium, but now it blows up in chrome (when compiled to java script).