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

Two tests started failing on csp in r[33397:33409] #17326

Closed
ricowind opened this issue Mar 7, 2014 · 13 comments
Closed

Two tests started failing on csp in r[33397:33409] #17326

ricowind opened this issue Mar 7, 2014 · 13 comments
Assignees
Labels
area-pkg Used for miscellaneous pkg/ packages not associated with specific area- teams.

Comments

@ricowind
Copy link
Contributor

ricowind commented Mar 7, 2014

pkg/polymer/test/noscript_test
pkg/polymer/test/js_interop_test

See
http://build.chromium.org/p/client.dart/builders/dart2js-drt-linux-csp-be/builds/979/steps/dart2js-drt%20tests%20--csp%20failures/logs/stdio

Assigning to polymer, but feel free to reasign

@jmesserly
Copy link

Added Pkg-Polymer label.

@jmesserly
Copy link

Removed Area-Polymer label.
Added Area-Pkg label.

@sigmundch
Copy link
Member

Added this to the 1.6 milestone.

@sigmundch
Copy link
Member

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

@sigmundch
Copy link
Member

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

@sigmundch
Copy link
Member

Added PolymerMilestone-Next label.

@sigmundch
Copy link
Member

Removed Polymer-Milestone-Next label.

@kasperl
Copy link

kasperl commented Jul 10, 2014

Added this to the 1.6 milestone.

@sigmundch
Copy link
Member

Removed this from the 1.6 milestone.

@sethladd
Copy link
Contributor

Removed Priority-High label.
Added Priority-Medium label.

@sigmundch
Copy link
Member

pkg/polymer/test/noscript_test is fixed now that we rolled the latest version of polymer.js, we removed the last bit of non-CSP compliant code they had.

The other tests are failing because they contain inline JS code. I'm working on a fix for that too.


Set owner to @sigmundch.
Added Started label.

@sigmundch
Copy link
Member

submitted fix in r38707


Added Fixed label.

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

DartBot commented Jun 5, 2015

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

This issue was closed.
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

6 participants