Bug 61215 – Default min DOM Timer interval is not set soon enough for first page to pick it up in WK2

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit Misc.

[Matthew Delaney]

The min DOM timer interval is set just after the first page is created. When pages are created, they use the currently set timer interval. Thus, we should be setting the min DOM timer interval sooner - ideally, just before page creation.

As a follow up, this DOM timer setting code could be cleaned up better very soon since it appears that all (or at least most) of the ports are now agreeing upon default values. I've filed this bug for that followup cleanup: https://bugs.webkit.org/show_bug.cgi?id=61214

[Matthew Delaney]

Created attachment 94267 [details]
Patch

[Darin Adler]

Comment on attachment 94267 [details]
Patch

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

> Source/WebKit2/WebProcess/WebPage/WebPage.cpp:210
> +    Settings::setDefaultMinDOMTimerInterval(0.004);

Moving this early seems good, but it’s a bit strange to tuck this in between setting up the page clients and calling new Page, since the pageClients structure is really just an argument to the Page constructor. I suggest calling this even before the definition of the pageClients local variable, right after the assert.

[Matthew Delaney]

Created attachment 94268 [details]
Patch

[Matthew Delaney]

Committed r86985: <http://trac.webkit.org/changeset/86985>