Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(44)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: tools/metrics/histograms/histograms.xml

Issue 2233543002: Make first TouchStart and first TouchMove events on a flinging layer non-blocking (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fling intervetion Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
Download patch
« third_party/WebKit/Source/core/input/TouchEventManager.cpp ('K') | « third_party/WebKit/public/platform/WebInputEvent.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 28968b1a570d6e7b780e809a2cda0d43da9750b9..1ffd90f138fe20bc3e65d94a03591b7da64a76fd 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -13228,6 +13228,10 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<histogram name="Event.PassiveListeners.ForcedNonBlockingLatency"
units="microseconds">
+ <obsolete>
+ Deprecated 08/2016 in Issue 2233543002, and replaced by
dtapuska 2016/08/15 14:41:29 ditto
+ Event.PassiveListeners.ForcedNonBlockingLatencyDueToFling.
+ </obsolete>
<owner>dtapuska@chromium.org</owner>
<summary>
Time between when a forced non-blocking event was generated and the event
@@ -13236,6 +13240,17 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram name="Event.PassiveListeners.ForcedNonBlockingLatencyDueToFling"
+ units="microseconds">
+ <owner>dtapuska@chromium.org</owner>
+ <summary>
+ Time between when a forced non-blocking touchstart or first touchmove event
+ per scroll was generated and the event processed during fling. This
+ histogram tracks the benefit of forcing non-blocking events listeners during
+ fling.
+ </summary>
+</histogram>
+
<histogram name="Event.PassiveListeners.Latency" units="microseconds">
<owner>dtapuska@chromium.org</owner>
<summary>
@@ -13317,7 +13332,7 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
enum="TouchEventDispatchResultType">
<owner>tdresser@chromium.org</owner>
<summary>
- Records the disposition (handled or not handled) of touch start events and
+ Records the disposition (handled or not handled) of touchstart events and
the first touchmove events per scroll. Only recorded after the page is fully
loaded.
</summary>
@@ -13327,7 +13342,7 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
enum="TouchEventDispatchResultType">
<owner>tdresser@chromium.org</owner>
<summary>
- Records the disposition (handled or not handled) of touch start events and
+ Records the disposition (handled or not handled) of touchstart events and
the first touchmove events per scroll. Only recorded before the page is
fully loaded.
</summary>
@@ -13335,27 +13350,55 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<histogram name="Event.Touch.TouchDispositionsDuringFling"
enum="TouchEventDispatchResultType">
+ <obsolete>
+ Deprecated 08/2016 in Issue 2233543002, and replaced by
+ Event.Touch.TouchDispositionsDuringFling2.
+ </obsolete>
<owner>tdresser@chromium.org</owner>
<summary>
- Records the disposition (handled or not handled) of touch start events. Only
+ Records the disposition (handled or not handled) of touchstart events. Only
recorded while there is an active fling animation.
</summary>
</histogram>
+<histogram name="Event.Touch.TouchDispositionsDuringFling2"
+ enum="TouchEventDispatchResultType">
+ <owner>tdresser@chromium.org</owner>
+ <summary>
+ Records the disposition (handled or not handled) of touchstart and first
+ touchmove events per scroll. Only recorded while there is an active fling
+ animation.
+ </summary>
+</histogram>
+
<histogram name="Event.Touch.TouchDispositionsOutsideFling"
enum="TouchEventDispatchResultType">
+ <obsolete>
+ Deprecated 08/2016 in Issue 2233543002, and replaced by
+ Event.Touch.TouchDispositionsOutsideFling2.
+ </obsolete>
<owner>tdresser@chromium.org</owner>
<summary>
- Records the disposition (handled or not handled) of touch start events. Only
+ Records the disposition (handled or not handled) of touchstart events. Only
recorded while there is no active fling animation.
</summary>
</histogram>
+<histogram name="Event.Touch.TouchDispositionsOutsideFling2"
+ enum="TouchEventDispatchResultType">
+ <owner>tdresser@chromium.org</owner>
+ <summary>
+ Records the disposition (handled or not handled) of touchstart and first
+ touchmove events per scroll events. Only recorded while there is no active
+ fling animation.
+ </summary>
+</histogram>
+
<histogram name="Event.Touch.TouchLatencyAfterPageLoad" units="microseconds">
<owner>tdresser@chromium.org</owner>
<summary>
Time between when a touch event was generated and the event was processed.
- Recorded only for touch start events and the first touchmove events per
+ Recorded only for touchstart events and the first touchmove events per
scroll that occur after the page is fully loaded.
</summary>
</histogram>
@@ -13364,16 +13407,29 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<owner>tdresser@chromium.org</owner>
<summary>
Time between when a touch event was generated and the event was processed.
- Recorded only for touch start events and the first touchmove events per
+ Recorded only for touchstart events and the first touchmove events per
scroll that occur before the page is fully loaded. This histogram tracks the
benefit of forcing passive event listeners before the page is fully loaded.
</summary>
</histogram>
+<histogram name="Event.Touch.TouchLatencyOutsideFling" units="microseconds">
+ <owner>tdresser@chromium.org</owner>
+ <summary>
+ Time between when a touch event was generated and the event was processed.
+ Recorded only for touchstart events and the first touchmove events per
+ scroll when there was no active fling animation.
+ </summary>
+</histogram>
+
<histogram name="Event.Touch.TouchStartLatencyDuringFling" units="microseconds">
+ <obsolete>
+ Deprecated 08/2016 in Issue 2233543002, and replaced by
dtapuska 2016/08/15 14:41:29 its kind of weird to call a review an issue #... d
lanwei 2016/08/17 01:22:35 Done.
+ Event.PassiveListeners.ForcedNonBlockingLatencyDueToFling.
+ </obsolete>
<owner>tdresser@chromium.org</owner>
<summary>
- Time between when a touch start event was generated and the event was
+ Time between when a touchstart event was generated and the event was
processed. Recorded only when there was an active fling animation. This
histogram tracks the benefit of forcing passive event listeners during
fling.
@@ -13382,9 +13438,13 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<histogram name="Event.Touch.TouchStartLatencyOutsideFling"
units="microseconds">
+ <obsolete>
dtapuska 2016/08/15 14:41:29 ditto
+ Deprecated 08/2016 in Issue 2233543002, and replaced by
+ Event.Touch.TouchLatencyOutsideFling.
+ </obsolete>
<owner>tdresser@chromium.org</owner>
<summary>
- Time between when a touch start event was generated and the event was
+ Time between when a touchstart event was generated and the event was
processed. Recorded only when there was no active fling animation.
</summary>
</histogram>
« third_party/WebKit/Source/core/input/TouchEventManager.cpp ('K') | « third_party/WebKit/public/platform/WebInputEvent.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698