Title: [140617] trunk/Source/WebCore
- Revision
- 140617
- Author
- [email protected]
- Date
- 2013-01-23 17:18:37 -0800 (Wed, 23 Jan 2013)
Log Message
Have scrollperf logging log information about wheel event handlers
https://bugs.webkit.org/show_bug.cgi?id=107761
<rdar://problem/12281015>
Reviewed by Tim Horton.
Log when the wheel event handler count of a ScrollingTreeScrollingNodeMac changes.
* page/scrolling/mac/ScrollingTreeScrollingNodeMac.mm:
(WebCore::ScrollingTreeScrollingNodeMac::update):
(WebCore::logWheelEventHandlerCountChanged):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (140616 => 140617)
--- trunk/Source/WebCore/ChangeLog 2013-01-24 01:01:40 UTC (rev 140616)
+++ trunk/Source/WebCore/ChangeLog 2013-01-24 01:18:37 UTC (rev 140617)
@@ -1,3 +1,17 @@
+2013-01-23 Simon Fraser <[email protected]>
+
+ Have scrollperf logging log information about wheel event handlers
+ https://bugs.webkit.org/show_bug.cgi?id=107761
+ <rdar://problem/12281015>
+
+ Reviewed by Tim Horton.
+
+ Log when the wheel event handler count of a ScrollingTreeScrollingNodeMac changes.
+
+ * page/scrolling/mac/ScrollingTreeScrollingNodeMac.mm:
+ (WebCore::ScrollingTreeScrollingNodeMac::update):
+ (WebCore::logWheelEventHandlerCountChanged):
+
2013-01-23 Kentaro Hara <[email protected]>
[V8] Add an optional Isolate parameter to GetTemplate() and GetRawTemplate()
Modified: trunk/Source/WebCore/page/scrolling/mac/ScrollingTreeScrollingNodeMac.mm (140616 => 140617)
--- trunk/Source/WebCore/page/scrolling/mac/ScrollingTreeScrollingNodeMac.mm 2013-01-24 01:01:40 UTC (rev 140616)
+++ trunk/Source/WebCore/page/scrolling/mac/ScrollingTreeScrollingNodeMac.mm 2013-01-24 01:18:37 UTC (rev 140617)
@@ -44,7 +44,9 @@
namespace WebCore {
static void logThreadedScrollingMode(unsigned mainThreadScrollingReasons);
+static void logWheelEventHandlerCountChanged(unsigned);
+
PassOwnPtr<ScrollingTreeScrollingNode> ScrollingTreeScrollingNode::create(ScrollingTree* scrollingTree)
{
return adoptPtr(new ScrollingTreeScrollingNodeMac(scrollingTree));
@@ -97,6 +99,11 @@
if (scrollingTree()->scrollingPerformanceLoggingEnabled())
logThreadedScrollingMode(mainThreadScrollingReasons);
}
+
+ if ((state->changedProperties() & ScrollingStateScrollingNode::WheelEventHandlerCount)) {
+ if (scrollingTree()->scrollingPerformanceLoggingEnabled())
+ logWheelEventHandlerCountChanged(state->wheelEventHandlerCount());
+ }
}
void ScrollingTreeScrollingNodeMac::handleWheelEvent(const PlatformWheelEvent& wheelEvent)
@@ -396,6 +403,11 @@
WTFLogAlways("SCROLLING: Switching to threaded scrolling mode. Time: %f\n", WTF::monotonicallyIncreasingTime());
}
+void logWheelEventHandlerCountChanged(unsigned count)
+{
+ WTFLogAlways("SCROLLING: Wheel event handler count changed. Time: %f Count: %u\n", WTF::monotonicallyIncreasingTime(), count);
+}
+
} // namespace WebCore
#endif // ENABLE(THREADED_SCROLLING)
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes
