Title: [127840] trunk/Source/WebCore
Revision
127840
Author
timothy_hor...@apple.com
Date
2012-09-07 01:08:37 -0700 (Fri, 07 Sep 2012)

Log Message

Scrolling performance logging should use WTFLogAlways instead of printf
https://bugs.webkit.org/show_bug.cgi?id=96080

Reviewed by Dan Bernstein.

Use WTF logging facilities instead of printf() for scrolling performance logging.

* page/scrolling/mac/ScrollingTreeNodeMac.mm:
(WebCore::ScrollingTreeNodeMac::logExposedUnfilledArea):
* platform/graphics/ca/mac/WebTileLayer.mm:
(-[WebTileLayer logFilledFreshTile]):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (127839 => 127840)


--- trunk/Source/WebCore/ChangeLog	2012-09-07 07:56:07 UTC (rev 127839)
+++ trunk/Source/WebCore/ChangeLog	2012-09-07 08:08:37 UTC (rev 127840)
@@ -1,3 +1,17 @@
+2012-09-07  Tim Horton  <timothy_hor...@apple.com>
+
+        Scrolling performance logging should use WTFLogAlways instead of printf
+        https://bugs.webkit.org/show_bug.cgi?id=96080
+
+        Reviewed by Dan Bernstein.
+
+        Use WTF logging facilities instead of printf() for scrolling performance logging.
+
+        * page/scrolling/mac/ScrollingTreeNodeMac.mm:
+        (WebCore::ScrollingTreeNodeMac::logExposedUnfilledArea):
+        * platform/graphics/ca/mac/WebTileLayer.mm:
+        (-[WebTileLayer logFilledFreshTile]):
+
 2012-09-07  Luke Macpherson   <macpher...@chromium.org>
 
         Make CSSPrimitiveValue::cleanup() handle all UnitTypes, fixing memory leak in the process.

Modified: trunk/Source/WebCore/page/scrolling/mac/ScrollingTreeNodeMac.mm (127839 => 127840)


--- trunk/Source/WebCore/page/scrolling/mac/ScrollingTreeNodeMac.mm	2012-09-07 07:56:07 UTC (rev 127839)
+++ trunk/Source/WebCore/page/scrolling/mac/ScrollingTreeNodeMac.mm	2012-09-07 08:08:37 UTC (rev 127840)
@@ -338,7 +338,7 @@
     unsigned unfilledArea = TileCache::blankPixelCountForTiles(tiles, viewportRect(), IntPoint(-scrollPosition.x(), -scrollPosition.y()));
 
     if (unfilledArea)
-        printf("SCROLLING: Exposed tileless area. Time: %f Unfilled Pixels: %u\n", WTF::monotonicallyIncreasingTime(), unfilledArea);
+        WTFLogAlways("SCROLLING: Exposed tileless area. Time: %f Unfilled Pixels: %u\n", WTF::monotonicallyIncreasingTime(), unfilledArea);
 }
 
 static void logThreadedScrollingMode(unsigned reasonsForUpdatingScrollLayerPositionOnMainThread)

Modified: trunk/Source/WebCore/platform/graphics/ca/mac/WebTileLayer.mm (127839 => 127840)


--- trunk/Source/WebCore/platform/graphics/ca/mac/WebTileLayer.mm	2012-09-07 07:56:07 UTC (rev 127839)
+++ trunk/Source/WebCore/platform/graphics/ca/mac/WebTileLayer.mm	2012-09-07 08:08:37 UTC (rev 127840)
@@ -77,7 +77,7 @@
     visiblePart.intersect(_tileCache->visibleRect());
 
     if ([self repaintCount] == 1 && !visiblePart.isEmpty())
-        printf("SCROLLING: Filled visible fresh tile. Time: %f Unfilled Pixels: %u\n", WTF::monotonicallyIncreasingTime(), _tileCache->blankPixelCount());
+        WTFLogAlways("SCROLLING: Filled visible fresh tile. Time: %f Unfilled Pixels: %u\n", WTF::monotonicallyIncreasingTime(), _tileCache->blankPixelCount());
 }
 
 @end
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to