compositing/iframes/iframe-src-change.html test fails on Windows

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Simon Fraser (smfr)]

This test fails on Windows (when building with compositing).

The reason is that the test sets the iframe source, and immediately dumps layers. On windows, it seems that resource loading is timer-based, so the iframe load doesn't happen until after the test completes.

We can fix this with a setTimeout in the test.

[Adam Roben (:aroben)]

(In reply to comment #0)
> On windows, it seems that resource loading is timer-based, so the iframe load doesn't happen until after the test completes.

How does this work on Mac? Are we just getting lucky?

[Simon Fraser (smfr)]

Created attachment 67215 [details]
Patch

[Simon Fraser (smfr)]

I see the "about:blank" load happening via

 	WebKit.dll!WebCore::FrameLoader::loadWithDocumentLoader(WebCore::DocumentLoader * loader=0x046959d8, WebCore::FrameLoadType type=FrameLoadTypeRedirectWithLockedBackForwardList, WTF::PassRefPtr<WebCore::FormState> prpFormState={...})  Line 1488	C++
 	WebKit.dll!WebCore::FrameLoader::loadWithNavigationAction(const WebCore::ResourceRequest & request={...}, const WebCore::NavigationAction & action={...}, bool lockHistory=false, WebCore::FrameLoadType type=FrameLoadTypeRedirectWithLockedBackForwardList, WTF::PassRefPtr<WebCore::FormState> formState={...})  Line 1411	C++
 	WebKit.dll!WebCore::FrameLoader::loadURL(const WebCore::KURL & newURL={...}, const WTF::String & referrer={...}, const WTF::String & frameName={...}, bool lockHistory=false, WebCore::FrameLoadType newLoadType=FrameLoadTypeRedirectWithLockedBackForwardList, WTF::PassRefPtr<WebCore::Event> event={...}, WTF::PassRefPtr<WebCore::FormState> prpFormState={...})  Line 1354	C++
 	WebKit.dll!WebCore::FrameLoader::loadFrameRequest(const WebCore::FrameLoadRequest & request={...}, bool lockHistory=false, bool lockBackForwardList=true, WTF::PassRefPtr<WebCore::Event> event={...}, WTF::PassRefPtr<WebCore::FormState> formState={...}, WebCore::ReferrerPolicy referrerPolicy=SendReferrer)  Line 1293	C++
 	WebKit.dll!WebCore::FrameLoader::urlSelected(const WebCore::ResourceRequest & request={...}, const WTF::String & passedTarget={...}, WTF::PassRefPtr<WebCore::Event> triggeringEvent={...}, bool lockHistory=false, bool lockBackForwardList=true, bool userGesture=false, WebCore::ReferrerPolicy referrerPolicy=SendReferrer, WebCore::ShouldReplaceDocumentIfJavaScriptURL shouldReplaceDocumentIfJavaScriptURL=ReplaceDocumentIfJavaScriptURL)  Line 304	C++
 	WebKit.dll!WebCore::FrameLoader::changeLocation(const WebCore::KURL & url={...}, const WTF::String & referrer={...}, bool lockHistory=false, bool lockBackForwardList=true, bool userGesture=false, bool refresh=false)  Line 273 + 0x40 bytes	C++
>	WebKit.dll!WebCore::ScheduledURLNavigation::fire(WebCore::Frame * frame=0x046640e0)  Line 98 + 0x43 bytes	C++
 	WebKit.dll!WebCore::RedirectScheduler::timerFired(WebCore::Timer<WebCore::RedirectScheduler> * __formal=0x04664390)  Line 360 + 0x20 bytes	C++
 	WebKit.dll!WebCore::Timer<WebCore::RedirectScheduler>::fired()  Line 98 + 0x23 bytes	C++
 	WebKit.dll!WebCore::ThreadTimers::sharedTimerFiredInternal()  Line 112 + 0xf bytes	C++

This doesn't seem synchronous.

[Adam Roben (:aroben)]

I guess about:blank only loads synchronously for new frames. I still wonder how this works on Mac.

[Simon Fraser (smfr)]

Comment on attachment 67215 [details]
Patch

The test is wrong; I'll come up with a better one.

[Simon Fraser (smfr)]

Created attachment 67237 [details]
Patch

[Adam Roben (:aroben)]

Comment on attachment 67237 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=67237&action=prettypatch

> LayoutTests/compositing/iframes/iframe-src-change.html:52
> +                        results.innerText = "FAIL: should have initially had layers, and did not."
I would say "but did not"

r=me

[Simon Fraser (smfr)]

http://trac.webkit.org/changeset/67232