Bug 61215

Summary: Default min DOM Timer interval is not set soon enough for first page to pick it up in WK2
Product: WebKit Reporter: Matthew Delaney <mdelaney7>
Component: WebKit Misc.Assignee: Matthew Delaney <mdelaney7>
Severity: Normal CC: jberlin, sfalken
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Description Flags
Patch sfalken: review+

Description Matthew Delaney 2011-05-20 14:00:24 PDT
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
Comment 1 Matthew Delaney 2011-05-20 14:02:26 PDT
Created attachment 94267 [details]
Comment 2 Darin Adler 2011-05-20 14:04:44 PDT
Comment on attachment 94267 [details]

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.
Comment 3 Matthew Delaney 2011-05-20 14:07:25 PDT
Created attachment 94268 [details]
Comment 4 Matthew Delaney 2011-05-20 14:40:42 PDT
Committed r86985: <http://trac.webkit.org/changeset/86985>