Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove workaround for bug in HTML imports #19650

Closed
sigmundch opened this issue Jun 25, 2014 · 8 comments
Closed

remove workaround for bug in HTML imports #19650

sigmundch opened this issue Jun 25, 2014 · 8 comments
Labels
area-pkg Used for miscellaneous pkg/ packages not associated with specific area- teams.

Comments

@sigmundch
Copy link
Member

There is a bug in html imports that causes the browser to block in Chrome 36.

This happens when you have a DAG in your graph of imports, where the node in the merge of the DAG doesn't have a script tag.

See attached repro, which has a graph like this:

  index.html* -> n1 -> n2*
       | |
       | v
       +-------------> n3 -> n4*

where files with * have a script tag. The page doesn't load unless you add a script in n3, or you remove the one in n2.

This happens in polymer apps: polymer.html is a funnel, similar to n3 in the repro.


Attachment:
repro.tar (10 KB)

@sigmundch
Copy link
Member Author

FYI the bug seems to be fixed in chrome canary.

@sigmundch
Copy link
Member Author

Added this to the Later milestone.

@sigmundch
Copy link
Member Author

Added Waiting label.

@sigmundch
Copy link
Member Author

Added Polymer-P-2 label.

@sigmundch
Copy link
Member Author

Removed this from the Later milestone.

@sigmundch
Copy link
Member Author

Removed Polymer-P-2 label.
Added Polymer-Milestone-Later label.

@sigmundch
Copy link
Member Author

Removed Polymer-Milestone-Later label.
Added PolymerMilestone-Later label.

@sigmundch sigmundch added Type-Defect area-pkg Used for miscellaneous pkg/ packages not associated with specific area- teams. labels Jul 9, 2014
@DartBot
Copy link

DartBot commented Jun 5, 2015

This issue has been moved to dart-archive/polymer-dart#218.

@DartBot DartBot closed this as completed Jun 5, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-pkg Used for miscellaneous pkg/ packages not associated with specific area- teams.
Projects
None yet
Development

No branches or pull requests

2 participants