Add a system trace scope for event region building
authorsimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 2 Mar 2019 00:23:56 +0000 (00:23 +0000)
committersimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 2 Mar 2019 00:23:56 +0000 (00:23 +0000)
https://bugs.webkit.org/show_bug.cgi?id=195226

Reviewed by Jon Lee.

This trace scope measures the time spend converting element rects into Region objects,
which can be large on some pages.

The value for "Display Refresh Dispatch to main thread" was wrong and I fixed it.

Source/WTF:

* wtf/SystemTracing.h:

Tools:

* Tracing/SystemTracePoints.plist:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@242294 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WTF/ChangeLog
Source/WTF/wtf/SystemTracing.h
Tools/ChangeLog
Tools/Tracing/SystemTracePoints.plist

index 6adf6cc..48b2608 100644 (file)
@@ -1,3 +1,17 @@
+2019-03-01  Simon Fraser  <simon.fraser@apple.com>
+
+        Add a system trace scope for event region building
+        https://bugs.webkit.org/show_bug.cgi?id=195226
+
+        Reviewed by Jon Lee.
+
+        This trace scope measures the time spend converting element rects into Region objects,
+        which can be large on some pages.
+
+        The value for "Display Refresh Dispatch to main thread" was wrong and I fixed it.
+
+        * wtf/SystemTracing.h:
+
 2019-02-28  Yusuke Suzuki  <ysuzuki@apple.com>
 
         [JSC] sizeof(JSString) should be 16
index 105196e..ac9fda0 100644 (file)
@@ -74,6 +74,8 @@ enum TracePointCode {
     DisplayListRecordStart,
     DisplayListRecordEnd,
     DisplayRefreshDispatchingToMainThread,
+    ComputeEventRegionsStart,
+    ComputeEventRegionsEnd,
 
     WebKitRange = 10000,
     WebHTMLViewPaintStart,
index 6674833..ec4f305 100644 (file)
@@ -1,3 +1,17 @@
+2019-03-01  Simon Fraser  <simon.fraser@apple.com>
+
+        Add a system trace scope for event region building
+        https://bugs.webkit.org/show_bug.cgi?id=195226
+
+        Reviewed by Jon Lee.
+
+        This trace scope measures the time spend converting element rects into Region objects,
+        which can be large on some pages.
+
+        The value for "Display Refresh Dispatch to main thread" was wrong and I fixed it.
+
+        * Tracing/SystemTracePoints.plist:
+
 2019-03-01  Ross Kirsling  <ross.kirsling@sony.com>
 
         EnvironmentUtilities::stripValuesEndingWithString isn't thread-safe
index 0e1a678..940140a 100644 (file)
              </dict>
              <dict>
                  <key>Name</key>
-                 <string>Update touch event regions</string>
+                 <string>Update event rects</string>
                  <key>Type</key>
                  <string>Interval</string>
                  <key>Component</key>
                  <key>Component</key>
                  <string>47</string>
                  <key>Code</key>
+                 <string>5025</string>
+             </dict>
+             <dict>
+                 <key>Name</key>
+                 <string>Build event regions</string>
+                 <key>Type</key>
+                 <string>Interval</string>
+                 <key>Component</key>
+                 <string>47</string>
+                 <key>CodeBegin</key>
                  <string>5026</string>
+                 <key>CodeEnd</key>
+                 <string>5027</string>
              </dict>
              <dict>
                  <key>Name</key>