Adds Ash.StationaryTouchDuration to histograms.xml

Adds Ash.StationaryTouchDuration to histograms.xml

BUG=465751
TEST=None

Committed: https://crrev.com/bfe8469295007b95bf14004ae0b295430091f08a
Cr-Commit-Position: refs/heads/master@{#321204}

[pkotwicz, 2015-03-17 21:50:35]

Patchset #1 (id:1) has been deleted

[pkotwicz, 2015-03-17 21:51:50]

pkotwicz@chromium.org changed reviewers:
+ sadrul@chromium.org

[pkotwicz, 2015-03-17 21:51:50]

Sadrul, PTAL
This was the only Ash histogram which is missing from histograms.xml

[sadrul, 2015-03-17 21:55:26]

https://codereview.chromium.org/1017663003/diff/20001/tools/metrics/histogram...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/1017663003/diff/20001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:690: + 
<owner>sadrul@chromium.org</owner>
rbyers@

[pkotwicz, 2015-03-17 22:14:46]

Patchset #2 (id:40001) has been deleted

[pkotwicz, 2015-03-17 22:15:05]

Sadrul, can you please take another look?

https://codereview.chromium.org/1017663003/diff/20001/tools/metrics/histogram...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/1017663003/diff/20001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:690: + 
<owner>sadrul@chromium.org</owner>
On 2015/03/17 21:55:26, sadrul wrote:
> rbyers@

Done.

[sadrul, 2015-03-17 22:18:21]

lgtm (not sure why this needs my review though; rbyers@ would be more
appropriate)

[pkotwicz, 2015-03-18 17:40:07]

pkotwicz@chromium.org changed reviewers:
+ isherman@chromium.org

[pkotwicz, 2015-03-18 17:40:21]

isherman@ for OWNERS

[Ilya Sherman, 2015-03-18 18:26:39]

histograms.xml lgtm.  For future CLs, please include histograms.xml changes
along with the C++ changes, if possible.

https://codereview.chromium.org/1017663003/diff/60001/tools/metrics/histogram...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/1017663003/diff/60001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:691: +  <summary>The duration of mostly
stationary touches</summary>
nit: Please end the sentence with a period.

[Ilya Sherman, 2015-03-18 18:28:27]

From clicking through to the bug, I see that you're adding histograms that were
omitted from histograms.xml.  Thanks!

From reading through the code that logs the histogram, I have one more comment:

https://codereview.chromium.org/1017663003/diff/60001/tools/metrics/histogram...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/1017663003/diff/60001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:691: +  <summary>The duration of mostly
stationary touches</summary>
FWIW, it looks like this is only recorded for long touches.  Might be worth
mentioning that here as well.

[pkotwicz, 2015-03-18 18:54:16]

Patchset #3 (id:80001) has been deleted

[pkotwicz, 2015-03-18 18:54:39]

The CQ bit was checked by pkotwicz@chromium.org

[pkotwicz, 2015-03-18 18:54:40]

The patchset sent to the CQ was uploaded after l-g-t-m from sadrul@chromium.org,
isherman@chromium.org
Link to the patchset: https://codereview.chromium.org/1017663003/#ps100001
(title: " ")

[commit-bot: I haz the power, 2015-03-18 18:56:02]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1017663003/100001

[commit-bot: I haz the power, 2015-03-18 21:15:22]

Committed patchset #3 (id:100001)

[commit-bot: I haz the power, 2015-03-18 21:16:00]

Patchset 3 (id:??) landed as
https://crrev.com/bfe8469295007b95bf14004ae0b295430091f08a
Cr-Commit-Position: refs/heads/master@{#321204}