Chromium Code Reviews| Index: tools/metrics/histograms/histograms.xml |
| diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml |
| index dbadde1624dc6ee83e21482b872d9e356507fd66..03c0d72326dcaf153d4273ee82d2a37777f92630 100644 |
| --- a/tools/metrics/histograms/histograms.xml |
| +++ b/tools/metrics/histograms/histograms.xml |
| @@ -13040,6 +13040,17 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries. |
| </summary> |
| </histogram> |
| +<histogram name="Event.DownEventCount.PerDestination" |
| + enum="DownEventDestination"> |
| + <owner>xiaoyinh@chromium.org</owner> |
|
bruthig
2016/09/30 00:30:25
nit: Might be a good idea to put a PM here as well
xiaoyinh(OOO Sep 11-29)
2016/09/30 16:53:05
Done.
|
| + <summary> |
| + The number of down events received per destination. Every down event that is |
| + targeted to each destination will be counted, including those that don't |
| + have an effect. For example: Tapping on a disabled button inside the browser |
| + frame will be treated as down events on browser window. |
| + </summary> |
| +</histogram> |
| + |
| <histogram name="Event.DownEventCount.PerFormFactor" enum="DownEventFormFactor"> |
| <owner>xiaoyinh@chromium.org</owner> |
| <summary> |
| @@ -76730,6 +76741,14 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries. |
| <int value="2" label="GET_NON_CACHABLE"/> |
| </enum> |
| +<enum name="DownEventDestination" type="int"> |
| + <int value="0" label="Others, everything except browser and apps"/> |
| + <int value="1" label="Inside the browser frame"/> |
| + <int value="2" label="Regular chrome app, except default note-taking app"/> |
| + <int value="3" label="ARC++ app (android app on Chrome)"/> |
|
Daniel Erat
2016/09/29 22:58:46
nit: i think we usually use "ARC" instead of "ARC+
xiaoyinh(OOO Sep 11-29)
2016/09/30 00:25:40
Done.
|
| + <int value="4" label="Default note-taking app"/> |
| +</enum> |
| + |
| <enum name="DownEventFormFactor" type="int"> |
| <int value="0" label="Clamshell"/> |
| <int value="1" label="Touchview"/> |