Bug 42777 – WebKit2 needs layoutTestController.setDeferMainResourceDataLoad

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit2

[Sam Weinig]

WebKit2 needs layoutTestController.setDeferMainResourceDataLoad

[Sam Weinig]

<rdar://problem/8218168>

[Jonathan Bedard]

Created attachment 286339 [details]
Patch

[Jonathan Bedard]

Note that only one test actually called setDeferMainResourceDataLoad, and even in DumpRenderTree, this test would pass if the function was not called.

[Darin Adler]

Comment on attachment 286339 [details]
Patch

This does not seem right. We should fix the test so it tests what it intended to, rather than removing it. I see no rationale here for removing the test other than “it never worked”, but that is not sufficient.

[Darin Adler]

One path here is to come up for a better rationale for why it's OK to remove this test.

Unfortunately, the fact that the test passes when setDeferMainResourceDataLoad does nothing does *not* mean the test had no value. It's entirely possible that it can detect bugs that other tests cannot. It's also possible, though, that it is true that the test no longer has value.

[Jonathan Bedard]

Another possibility is to keep the tests but remove calls to setDeferMainResourceDataLoad, but keep the test.  Looking at the test, it seems that deferring load is the default behavior.

[Jonathan Bedard]

Created attachment 286639 [details]
Patch

[Darin Adler]

Comment on attachment 286639 [details]
Patch

I looked back at when we originally added the test:

https://bugs.webkit.org/show_bug.cgi?id=30879

I read it carefully, and I see that this was a test specific to a peculiar way that WebKit was used by the now-long-obsolete Qt port. Given that, I changed my mind and I think that we *can* remove the test.

[Jonathan Bedard]

Created attachment 286864 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 286864 [details]
Patch

Clearing flags on attachment: 286864

Committed r204918: <http://trac.webkit.org/changeset/204918>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Ryan Haddad]

Attempted Windows build fix in http://trac.webkit.org/projects/webkit/changeset/204933