Title: [113034] trunk/Source
Revision
113034
Author
[email protected]
Date
2012-04-03 08:03:29 -0700 (Tue, 03 Apr 2012)

Log Message

Call incrementStatsCounter directly
https://bugs.webkit.org/show_bug.cgi?id=83023

Patch by Mark Pilgrim <[email protected]> on 2012-04-03
Reviewed by Kentaro Hara.

Source/WebCore:

* bindings/v8/V8Proxy.h:
* platform/chromium/PlatformSupport.h:
(PlatformSupport):

Source/WebKit/chromium:

* src/PlatformSupport.cpp:
(WebCore):
* src/WebFrameImpl.cpp:
(WebKit::WebFrameImpl::WebFrameImpl):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (113033 => 113034)


--- trunk/Source/WebCore/ChangeLog	2012-04-03 14:48:33 UTC (rev 113033)
+++ trunk/Source/WebCore/ChangeLog	2012-04-03 15:03:29 UTC (rev 113034)
@@ -1,3 +1,14 @@
+2012-04-03  Mark Pilgrim  <[email protected]>
+
+        Call incrementStatsCounter directly
+        https://bugs.webkit.org/show_bug.cgi?id=83023
+
+        Reviewed by Kentaro Hara.
+
+        * bindings/v8/V8Proxy.h:
+        * platform/chromium/PlatformSupport.h:
+        (PlatformSupport):
+
 2012-04-03  Sami Kyostila  <[email protected]>
 
         [chromium] Canvas2DLayerChromium::updateCompositorResources should flush after copying

Modified: trunk/Source/WebCore/bindings/v8/V8Proxy.h (113033 => 113034)


--- trunk/Source/WebCore/bindings/v8/V8Proxy.h	2012-04-03 14:48:33 UTC (rev 113033)
+++ trunk/Source/WebCore/bindings/v8/V8Proxy.h	2012-04-03 15:03:29 UTC (rev 113034)
@@ -46,7 +46,7 @@
 #include <wtf/text/TextPosition.h>
 
 #if defined(ENABLE_DOM_STATS_COUNTERS) && PLATFORM(CHROMIUM)
-#define INC_STATS(name) PlatformSupport::incrementStatsCounter(name)
+#define INC_STATS(name) webkitPlatformSupport()->incrementStatsCounter(name)
 #else
 #define INC_STATS(name)
 #endif

Modified: trunk/Source/WebCore/platform/chromium/PlatformSupport.h (113033 => 113034)


--- trunk/Source/WebCore/platform/chromium/PlatformSupport.h	2012-04-03 14:48:33 UTC (rev 113033)
+++ trunk/Source/WebCore/platform/chromium/PlatformSupport.h	2012-04-03 15:03:29 UTC (rev 113034)
@@ -260,7 +260,6 @@
     static void stopSharedTimer();
 
     // StatsCounters ------------------------------------------------------
-    static void incrementStatsCounter(const char* name);
     static void histogramCustomCounts(const char* name, int sample, int min, int max, int bucketCount);
     static void histogramEnumeration(const char* name, int sample, int boundaryValue);
 

Modified: trunk/Source/WebKit/chromium/ChangeLog (113033 => 113034)


--- trunk/Source/WebKit/chromium/ChangeLog	2012-04-03 14:48:33 UTC (rev 113033)
+++ trunk/Source/WebKit/chromium/ChangeLog	2012-04-03 15:03:29 UTC (rev 113034)
@@ -1,3 +1,15 @@
+2012-04-03  Mark Pilgrim  <[email protected]>
+
+        Call incrementStatsCounter directly
+        https://bugs.webkit.org/show_bug.cgi?id=83023
+
+        Reviewed by Kentaro Hara.
+
+        * src/PlatformSupport.cpp:
+        (WebCore):
+        * src/WebFrameImpl.cpp:
+        (WebKit::WebFrameImpl::WebFrameImpl):
+
 2012-04-03  Sami Kyostila  <[email protected]>
 
         [chromium] Canvas2DLayerChromium::updateCompositorResources should flush after copying

Modified: trunk/Source/WebKit/chromium/src/PlatformSupport.cpp (113033 => 113034)


--- trunk/Source/WebKit/chromium/src/PlatformSupport.cpp	2012-04-03 14:48:33 UTC (rev 113033)
+++ trunk/Source/WebKit/chromium/src/PlatformSupport.cpp	2012-04-03 15:03:29 UTC (rev 113034)
@@ -713,11 +713,6 @@
 
 // StatsCounters --------------------------------------------------------------
 
-void PlatformSupport::incrementStatsCounter(const char* name)
-{
-    webKitPlatformSupport()->incrementStatsCounter(name);
-}
-
 void PlatformSupport::histogramCustomCounts(const char* name, int sample, int min, int max, int bucketCount)
 {
     webKitPlatformSupport()->histogramCustomCounts(name, sample, min, max, bucketCount);

Modified: trunk/Source/WebKit/chromium/src/WebFrameImpl.cpp (113033 => 113034)


--- trunk/Source/WebKit/chromium/src/WebFrameImpl.cpp	2012-04-03 14:48:33 UTC (rev 113033)
+++ trunk/Source/WebKit/chromium/src/WebFrameImpl.cpp	2012-04-03 15:03:29 UTC (rev 113034)
@@ -1976,7 +1976,7 @@
     , m_identifier(generateFrameIdentifier())
     , m_inSameDocumentHistoryLoad(false)
 {
-    PlatformSupport::incrementStatsCounter(webFrameActiveCount);
+    WebKit::Platform::current()->incrementStatsCounter(webFrameActiveCount);
     frameCount++;
 }
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to