Bug 94060

Summary: [Chromium] fast/dom/Window/dom-access-from-closure-window.html is flaky on Linux
Product: WebKit Reporter: Yuta Kitamura <yutak>
Component: Tools / TestsAssignee: Adam Barth <abarth>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, eric, haraken, japhet, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Need to check JSC
none
Patch none

Description Yuta Kitamura 2012-08-14 19:29:57 PDT
The following test is flaky on Chromium-Linux bots:

fast/dom/Window/dom-access-from-closure-window.html

Looks like http://trac.webkit.org/changeset/125592 fixed the issue which caused this test to fail consistently, but it introduced a new flakiness (or revealed the hidden flakiness).
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=fast%2Fdom%2FWindow%2Fdom-access-from-closure-window.html

It seems Mac and Windows are okay at this moment.

Diffs look like the following:

--- /b/build/slave/Webkit_Linux__dbg_/build/layout-test-results/fast/dom/Window/dom-access-from-closure-window-expected.txt 
+++ /b/build/slave/Webkit_Linux__dbg_/build/layout-test-results/fast/dom/Window/dom-access-from-closure-window-actual.txt 
@@ -1,5 +1,4 @@
-document.URL: LayoutTests/fast/dom/Window/resources/dom-access-from-closure-window-child.html
-window.document.URL: LayoutTests/fast/dom/Window/resources/notify-opener-done.html
-name: child
-window.name: child
+CONSOLE MESSAGE: Unsafe JavaScript attempt to access frame with URL notify-opener-done.html. Domains, protocols and ports must match.
 
+An exception was thrown: Cannot read property 'URL' of undefined
+
Comment 1 Adam Barth 2012-08-14 20:22:57 PDT
Thanks.  This does sound related to my patch.
Comment 2 Adam Barth 2012-08-14 22:11:24 PDT
The flakiness seems to be gc-dependent.  If you call gc() as the first statement in done(), you can repro 100% of the time.
Comment 3 Adam Barth 2012-08-14 23:45:26 PDT
Created attachment 158509 [details]
Need to check JSC
Comment 4 Adam Barth 2012-08-15 00:17:40 PDT
Created attachment 158515 [details]
Patch
Comment 5 Kentaro Hara 2012-08-15 00:24:53 PDT
Comment on attachment 158515 [details]
Patch

Looks reasonable.
Comment 6 WebKit Review Bot 2012-08-15 01:26:22 PDT
Comment on attachment 158515 [details]
Patch

Clearing flags on attachment: 158515

Committed r125656: <http://trac.webkit.org/changeset/125656>
Comment 7 WebKit Review Bot 2012-08-15 01:26:26 PDT
All reviewed patches have been landed.  Closing bug.