Title: [128828] trunk/Source
Revision
128828
Author
[email protected]
Date
2012-09-17 17:35:35 -0700 (Mon, 17 Sep 2012)

Log Message

[chromium] Add rendering commit statistics
https://bugs.webkit.org/show_bug.cgi?id=96938

Patch by Brian Anderson <[email protected]> on 2012-09-17
Reviewed by James Robinson.

Adds total commit time and total commit count to WebRenderingStats.
Allows us to caculate average commit time in performance tests.

Source/Platform:

* chromium/public/WebRenderingStats.h:
(WebRenderingStats):
(WebKit::WebRenderingStats::WebRenderingStats):
(WebKit::WebRenderingStats::enumerateFields):

Source/WebKit/chromium:

* src/WebLayerTreeViewImpl.cpp:
(WebKit::WebLayerTreeViewImpl::renderingStats):

Modified Paths

Diff

Modified: trunk/Source/Platform/ChangeLog (128827 => 128828)


--- trunk/Source/Platform/ChangeLog	2012-09-18 00:14:36 UTC (rev 128827)
+++ trunk/Source/Platform/ChangeLog	2012-09-18 00:35:35 UTC (rev 128828)
@@ -1,3 +1,18 @@
+2012-09-17  Brian Anderson  <[email protected]>
+
+        [chromium] Add rendering commit statistics
+        https://bugs.webkit.org/show_bug.cgi?id=96938
+
+        Reviewed by James Robinson.
+
+        Adds total commit time and total commit count to WebRenderingStats.
+        Allows us to caculate average commit time in performance tests.
+
+        * chromium/public/WebRenderingStats.h:
+        (WebRenderingStats):
+        (WebKit::WebRenderingStats::WebRenderingStats):
+        (WebKit::WebRenderingStats::enumerateFields):
+
 2012-09-17  Antoine Labour  <[email protected]>
 
         [chromium] Add onSendFrameToParentCompositorAck to WebCompositorOutputSurfaceClient

Modified: trunk/Source/Platform/chromium/public/WebRenderingStats.h (128827 => 128828)


--- trunk/Source/Platform/chromium/public/WebRenderingStats.h	2012-09-18 00:14:36 UTC (rev 128827)
+++ trunk/Source/Platform/chromium/public/WebRenderingStats.h	2012-09-18 00:35:35 UTC (rev 128828)
@@ -34,6 +34,8 @@
     int droppedFrameCount;
     double totalPaintTimeInSeconds;
     double totalRasterizeTimeInSeconds;
+    double totalCommitTimeInSeconds;
+    size_t totalCommitCount;
 
     WebRenderingStats()
         : numAnimationFrames(0)
@@ -41,6 +43,8 @@
         , droppedFrameCount(0)
         , totalPaintTimeInSeconds(0)
         , totalRasterizeTimeInSeconds(0)
+        , totalCommitTimeInSeconds(0)
+        , totalCommitCount(0)
     {
     }
 
@@ -64,6 +68,8 @@
         enumerator->addInt("droppedFrameCount", droppedFrameCount);
         enumerator->addDouble("totalPaintTimeInSeconds", totalPaintTimeInSeconds);
         enumerator->addDouble("totalRasterizeTimeInSeconds", totalRasterizeTimeInSeconds);
+        enumerator->addDouble("totalCommitTimeInSeconds", totalCommitTimeInSeconds);
+        enumerator->addInt("totalCommitCount", totalCommitCount);
     }
 };
 

Modified: trunk/Source/WebKit/chromium/ChangeLog (128827 => 128828)


--- trunk/Source/WebKit/chromium/ChangeLog	2012-09-18 00:14:36 UTC (rev 128827)
+++ trunk/Source/WebKit/chromium/ChangeLog	2012-09-18 00:35:35 UTC (rev 128828)
@@ -1,3 +1,16 @@
+2012-09-17  Brian Anderson  <[email protected]>
+
+        [chromium] Add rendering commit statistics
+        https://bugs.webkit.org/show_bug.cgi?id=96938
+
+        Reviewed by James Robinson.
+
+        Adds total commit time and total commit count to WebRenderingStats.
+        Allows us to caculate average commit time in performance tests.
+
+        * src/WebLayerTreeViewImpl.cpp:
+        (WebKit::WebLayerTreeViewImpl::renderingStats):
+
 2012-09-17  James Robinson  <[email protected]>
 
         [chromium] Move a FilterOperationsTest and WebInputEventConversionTest back to webkit_unittest_files variable

Modified: trunk/Source/WebKit/chromium/src/WebLayerTreeViewImpl.cpp (128827 => 128828)


--- trunk/Source/WebKit/chromium/src/WebLayerTreeViewImpl.cpp	2012-09-18 00:14:36 UTC (rev 128827)
+++ trunk/Source/WebKit/chromium/src/WebLayerTreeViewImpl.cpp	2012-09-18 00:35:35 UTC (rev 128828)
@@ -195,6 +195,8 @@
     stats.droppedFrameCount = ccStats.droppedFrameCount;
     stats.totalPaintTimeInSeconds = ccStats.totalPaintTimeInSeconds;
     stats.totalRasterizeTimeInSeconds = ccStats.totalRasterizeTimeInSeconds;
+    stats.totalCommitTimeInSeconds = ccStats.totalCommitTimeInSeconds;
+    stats.totalCommitCount = ccStats.totalCommitCount;
 }
 
 void WebLayerTreeViewImpl::setFontAtlas(SkBitmap bitmap, WebRect asciiToWebRectTable[128], int fontHeight)
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to