Title: [86985] trunk/Source/WebKit2
Revision
86985
Author
mdela...@apple.com
Date
2011-05-20 14:40:36 -0700 (Fri, 20 May 2011)

Log Message

2011-05-20  Matthew Delaney  <mdela...@apple.com>

        Reviewed by Steve Falkenburg.

        Default min DOM Timer interval is not set soon enough for first page to pick it up in WK2
        https://bugs.webkit.org/show_bug.cgi?id=61215 

        * WebProcess/WebPage/WebPage.cpp: Simply moving the DOM min timer interval setting to before the first page creation.

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (86984 => 86985)


--- trunk/Source/WebKit2/ChangeLog	2011-05-20 21:28:00 UTC (rev 86984)
+++ trunk/Source/WebKit2/ChangeLog	2011-05-20 21:40:36 UTC (rev 86985)
@@ -1,3 +1,12 @@
+2011-05-20  Matthew Delaney  <mdela...@apple.com>
+
+        Reviewed by Steve Falkenburg.
+
+        Default min DOM Timer interval is not set soon enough for first page to pick it up in WK2
+        https://bugs.webkit.org/show_bug.cgi?id=61215 
+
+        * WebProcess/WebPage/WebPage.cpp: Simply moving the DOM min timer interval setting to before the first page creation.
+
 2011-05-20  Simon Fraser  <simon.fra...@apple.com>
 
         Reviewed by Sam Weinig.

Modified: trunk/Source/WebKit2/WebProcess/WebPage/WebPage.cpp (86984 => 86985)


--- trunk/Source/WebKit2/WebProcess/WebPage/WebPage.cpp	2011-05-20 21:28:00 UTC (rev 86984)
+++ trunk/Source/WebKit2/WebProcess/WebPage/WebPage.cpp	2011-05-20 21:40:36 UTC (rev 86985)
@@ -194,6 +194,9 @@
 #endif
 {
     ASSERT(m_pageID);
+    // FIXME: This is a non-ideal location for this Setting and
+    // 4ms should be adopted project-wide now, https://bugs.webkit.org/show_bug.cgi?id=61214
+    Settings::setDefaultMinDOMTimerInterval(0.004);
 
     Page::PageClients pageClients;
     pageClients.chromeClient = new WebChromeClient(this);
@@ -223,7 +226,6 @@
     m_page->setGroupName(m_pageGroup->identifier());
 
     platformInitialize();
-    Settings::setDefaultMinDOMTimerInterval(0.004);
 
     m_drawingArea = DrawingArea::create(this, parameters);
     m_mainFrame = WebFrame::createMainFrame(this);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to