Bug 106165

Summary: REGRESSION(r138834): http/tests/security/contentSecurityPolicy/report-uri-from-javascript.html is flaky
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: Page LoadingAssignee: Mike West <mkwst>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, ap, gyuyoung.kim, mkwst, mkwst+watchlist, rakuco, tdanderson, webkit.review.bot
Priority: P1 Keywords: LayoutTestFailure, Regression
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 105374    
Attachments:
Description Flags
Patch none

Comment 1 Ryosuke Niwa 2013-01-04 19:56:52 PST
Added a flaky test expectation in http://trac.webkit.org/changeset/138885.
Comment 2 Ryosuke Niwa 2013-01-04 19:58:32 PST
*** Bug 106153 has been marked as a duplicate of this bug. ***
Comment 3 Ryosuke Niwa 2013-01-04 20:00:02 PST
One more update: http://trac.webkit.org/changeset/138886
Comment 4 Mike West 2013-01-04 23:09:46 PST
Hrm. The line numbers are incorrect fairly often. That's no good.

I'll leave the test running for a bit locally to see if I can track down the error. Thanks for raising this, and for skipping the test.
Comment 5 Mike West 2013-01-04 23:56:29 PST
Looks like I chose a poor injection mechanism; the existing file injects two script tags, which generates two violation reports, delivered in an indeterminate order.

I'll create a more deterministic test, and upload it shortly for review.
Comment 6 Mike West 2013-01-05 00:04:49 PST
Created attachment 181430 [details]
Patch
Comment 7 Mike West 2013-01-05 00:05:39 PST
The attached patch should make the bots happier. Let's see what EWS thinks.
Comment 8 WebKit Review Bot 2013-01-05 13:22:14 PST
Comment on attachment 181430 [details]
Patch

Clearing flags on attachment: 181430

Committed r138907: <http://trac.webkit.org/changeset/138907>
Comment 9 WebKit Review Bot 2013-01-05 13:22:19 PST
All reviewed patches have been landed.  Closing bug.