Title: [119997] trunk/Source
Revision
119997
Author
pilg...@chromium.org
Date
2012-06-11 11:54:52 -0700 (Mon, 11 Jun 2012)

Log Message

[Chromium] Call shared timer functions directly
https://bugs.webkit.org/show_bug.cgi?id=88781

Reviewed by Adam Barth.

Part of a refactoring series. See tracking bug 82948.

Source/WebCore:

* platform/chromium/PlatformSupport.h:
(PlatformSupport):
* platform/chromium/SharedTimerChromium.cpp:
(WebCore::setSharedTimerFiredFunction):
(WebCore::setSharedTimerFireInterval):

Source/WebKit/chromium:

* src/PlatformSupport.cpp:
(WebCore):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (119996 => 119997)


--- trunk/Source/WebCore/ChangeLog	2012-06-11 18:48:47 UTC (rev 119996)
+++ trunk/Source/WebCore/ChangeLog	2012-06-11 18:54:52 UTC (rev 119997)
@@ -1,3 +1,18 @@
+2012-06-11  Mark Pilgrim  <pilg...@chromium.org>
+
+        [Chromium] Call shared timer functions directly
+        https://bugs.webkit.org/show_bug.cgi?id=88781
+
+        Reviewed by Adam Barth.
+
+        Part of a refactoring series. See tracking bug 82948.
+
+        * platform/chromium/PlatformSupport.h:
+        (PlatformSupport):
+        * platform/chromium/SharedTimerChromium.cpp:
+        (WebCore::setSharedTimerFiredFunction):
+        (WebCore::setSharedTimerFireInterval):
+
 2012-06-11  Pravin D  <pravind....@gmail.com>
 
         Relative pos. input fields in columns vanish when you start typing in them

Modified: trunk/Source/WebCore/platform/chromium/PlatformSupport.h (119996 => 119997)


--- trunk/Source/WebCore/platform/chromium/PlatformSupport.h	2012-06-11 18:48:47 UTC (rev 119996)
+++ trunk/Source/WebCore/platform/chromium/PlatformSupport.h	2012-06-11 18:54:52 UTC (rev 119997)
@@ -167,10 +167,6 @@
     static IntRect screenRect(Widget*);
     static IntRect screenAvailableRect(Widget*);
 
-    // SharedTimers -------------------------------------------------------
-    static void setSharedTimerFiredFunction(void (*func)());
-    static void setSharedTimerFireInterval(double);
-
     // Returns private and shared usage, in bytes. Private bytes is the amount of
     // memory currently allocated to this process that cannot be shared. Returns
     // false on platform specific error conditions.

Modified: trunk/Source/WebCore/platform/chromium/SharedTimerChromium.cpp (119996 => 119997)


--- trunk/Source/WebCore/platform/chromium/SharedTimerChromium.cpp	2012-06-11 18:48:47 UTC (rev 119996)
+++ trunk/Source/WebCore/platform/chromium/SharedTimerChromium.cpp	2012-06-11 18:54:52 UTC (rev 119997)
@@ -20,26 +20,24 @@
  * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
  * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
 #include "SharedTimer.h"
 
-#include "PlatformSupport.h"
-
 #include <public/Platform.h>
 
 namespace WebCore {
 
 void setSharedTimerFiredFunction(void (*f)())
-{                   
-    PlatformSupport::setSharedTimerFiredFunction(f);
+{
+    WebKit::Platform::current()->setSharedTimerFiredFunction(f);
 }
 
 void setSharedTimerFireInterval(double fireTime)
 {
-    PlatformSupport::setSharedTimerFireInterval(fireTime);
+    WebKit::Platform::current()->setSharedTimerFireInterval(fireTime);
 }
 
 void stopSharedTimer()

Modified: trunk/Source/WebKit/chromium/ChangeLog (119996 => 119997)


--- trunk/Source/WebKit/chromium/ChangeLog	2012-06-11 18:48:47 UTC (rev 119996)
+++ trunk/Source/WebKit/chromium/ChangeLog	2012-06-11 18:54:52 UTC (rev 119997)
@@ -1,3 +1,15 @@
+2012-06-11  Mark Pilgrim  <pilg...@chromium.org>
+
+        [Chromium] Call shared timer functions directly
+        https://bugs.webkit.org/show_bug.cgi?id=88781
+
+        Reviewed by Adam Barth.
+
+        Part of a refactoring series. See tracking bug 82948.
+
+        * src/PlatformSupport.cpp:
+        (WebCore):
+
 2012-06-11  Amy Ousterhout  <aoust...@chromium.org>
 
         [Chromium] Removing long WebDeviceOrientation constructor

Modified: trunk/Source/WebKit/chromium/src/PlatformSupport.cpp (119996 => 119997)


--- trunk/Source/WebKit/chromium/src/PlatformSupport.cpp	2012-06-11 18:48:47 UTC (rev 119996)
+++ trunk/Source/WebKit/chromium/src/PlatformSupport.cpp	2012-06-11 18:54:52 UTC (rev 119997)
@@ -371,18 +371,6 @@
 
 #endif // ENABLE(WEB_AUDIO)
 
-// SharedTimers ---------------------------------------------------------------
-
-void PlatformSupport::setSharedTimerFiredFunction(void (*func)())
-{
-    webKitPlatformSupport()->setSharedTimerFiredFunction(func);
-}
-
-void PlatformSupport::setSharedTimerFireInterval(double interval)
-{
-    webKitPlatformSupport()->setSharedTimerFireInterval(interval);
-}
-
 // Theming --------------------------------------------------------------------
 
 #if OS(WINDOWS)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to