2011-02-18 Kenneth Russell <kbr@google.com>
authorkbr@google.com <kbr@google.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 18 Feb 2011 23:20:12 +0000 (23:20 +0000)
committerkbr@google.com <kbr@google.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 18 Feb 2011 23:20:12 +0000 (23:20 +0000)
        Reviewed by James Robinson.

        [chromium] Clean up initialization of minimum timer interval
        https://bugs.webkit.org/show_bug.cgi?id=54772

        Rolled forward Chromium DEPS to pick up new webkit_support entry
        point. Removed setting of default minimum timer interval, as this
        is now done every time WebPreferences are applied.

        Tested with Chromium DRT port built from WebKit workspace; ran
        fast/dom/ layout tests.

        * DEPS:
        * src/WebKit.cpp:
        (WebKit::initialize):
2011-02-18  Kenneth Russell  <kbr@google.com>

        Reviewed by James Robinson.

        [chromium] Clean up initialization of minimum timer interval
        https://bugs.webkit.org/show_bug.cgi?id=54772

        Use new GetForegroundTabTimerInterval function in webkit_support
        to reset the minimum timer interval before each test.

        * DumpRenderTree/chromium/WebViewHost.cpp:
        (WebViewHost::reset):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@79038 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit/chromium/ChangeLog
Source/WebKit/chromium/DEPS
Source/WebKit/chromium/src/WebKit.cpp
Tools/ChangeLog
Tools/DumpRenderTree/chromium/WebViewHost.cpp

index 0ce9ed2..2cf282c 100644 (file)
@@ -1,3 +1,21 @@
+2011-02-18  Kenneth Russell  <kbr@google.com>
+
+        Reviewed by James Robinson.
+
+        [chromium] Clean up initialization of minimum timer interval
+        https://bugs.webkit.org/show_bug.cgi?id=54772
+
+        Rolled forward Chromium DEPS to pick up new webkit_support entry
+        point. Removed setting of default minimum timer interval, as this
+        is now done every time WebPreferences are applied.
+
+        Tested with Chromium DRT port built from WebKit workspace; ran
+        fast/dom/ layout tests.
+
+        * DEPS:
+        * src/WebKit.cpp:
+        (WebKit::initialize):
+
 2011-02-18  Ben Vanik  <benvanik@google.com>
 
         Reviewed by Kenneth Russell.
index bb5524d..020da9a 100644 (file)
@@ -32,7 +32,7 @@
 
 vars = {
   'chromium_svn': 'http://src.chromium.org/svn/trunk/src',
-  'chromium_rev': '75190'
+  'chromium_rev': '75443'
 }
 
 deps = {
index 471fd39..d130dd5 100644 (file)
@@ -66,15 +66,6 @@ void initialize(WebKitClient* webKitClient)
     WTF::initializeMainThread();
     WTF::AtomicString::init();
 
-    // Chromium sets the minimum interval timeout to 4ms, overriding the
-    // default of 10ms.  We'd like to go lower, however there are poorly
-    // coded websites out there which do create CPU-spinning loops.  Using
-    // 4ms prevents the CPU from spinning too busily and provides a balance
-    // between CPU spinning and the smallest possible interval timer.
-    // FIXME: remove this call once Chromium has been changed to set
-    // this for all pages using the new WebSettings API.
-    WebCore::Settings::setDefaultMinDOMTimerInterval(0.004);
-
     // There are some code paths (for example, running WebKit in the browser
     // process and calling into LocalStorage before anything else) where the
     // UTF8 string encoding tables are used on a background thread before
index fb1ee17..622c64d 100644 (file)
@@ -1,3 +1,16 @@
+2011-02-18  Kenneth Russell  <kbr@google.com>
+
+        Reviewed by James Robinson.
+
+        [chromium] Clean up initialization of minimum timer interval
+        https://bugs.webkit.org/show_bug.cgi?id=54772
+
+        Use new GetForegroundTabTimerInterval function in webkit_support
+        to reset the minimum timer interval before each test.
+
+        * DumpRenderTree/chromium/WebViewHost.cpp:
+        (WebViewHost::reset):
+
 2011-02-18  Dirk Pranke  <dpranke@chromium.org>
 
         Reviewed by Adam Roben.
index 5ba0dcb..a0ed6da 100644 (file)
@@ -1210,8 +1210,7 @@ void WebViewHost::reset()
 
     if (m_webWidget) {
         webView()->mainFrame()->setName(WebString());
-        // FIXME: change to use webkit_support::ForegroundTabTimerInterval once that lands.
-        webView()->settings()->setMinimumTimerInterval(0.004);
+        webView()->settings()->setMinimumTimerInterval(webkit_support::GetForegroundTabTimerInterval());
     }
 }