[Histograms]: Add histograms for requestIdleCallback.

[Histograms]: Add histograms for requestIdleCallback.

Updates histograms.xml to include UMA metrics added to Blink in
https://codereview.chromium.org/1344053002.

Also update UseCounter histograms to add missing values (by running
update_use_counter_feature_enum.py) to include the following new values:
     - HeaderValueNotMatchingRFC7230
     - ClipPathOfPositionedElement
     - ClipCssOfPositionedElement
     - NetInfoType
     - NetInfoDownlinkMax
     - NetInfoOnChange
     - NetInfoOnTypeChange
     - V8Window_Alert_Method
     - V8Window_Confirm_Method
     - V8Window_Prompt_Method
     - V8Window_Print_Method
     - V8Window_RequestIdleCallback_Method

BUG=514651
Committed: https://crrev.com/1e9d97bcc03b5c3a62ca2a68491095350d1ee0ce
Cr-Commit-Position: refs/heads/master@{#349085}

[rmcilroy, 2015-09-15 17:21:41]

Patchset #1 (id:1) has been deleted

[rmcilroy, 2015-09-15 17:22:28]

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

[rmcilroy, 2015-09-15 17:22:29]

Ilya, could you please review this change, thanks.

[Ilya Sherman, 2015-09-15 21:56:22]

LGTM % nits

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

https://codereview.chromium.org/1342143002/diff/20001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:50851: +    The amount of time that a
requestIdleCallback callback is allocated before
Optional nit: I think this sentence would be clearer with "is allocated"
replaced by "allocates"

https://codereview.chromium.org/1342143002/diff/20001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:50860: +    The amount of time which an
requestIdleCallback callback overran its
nit: "which an" -> "by which a"

[rmcilroy, 2015-09-15 23:18:59]

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

https://codereview.chromium.org/1342143002/diff/20001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:50851: +    The amount of time that a
requestIdleCallback callback is allocated before
That wouldn't be quite right since it isn't the callback which allocates the
time (it get's given a deadline to run within).  Changed to:
"The amount of time allotted to a requestIdleCallback callback, i.e., the
difference between the time when it is called and when its deadline expires."

Is this clearer?

https://codereview.chromium.org/1342143002/diff/20001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:50860: +    The amount of time which an
requestIdleCallback callback overran its
On 2015/09/15 21:56:22, Ilya Sherman wrote:
> nit: "which an" -> "by which a"

Done.

[rmcilroy, 2015-09-15 23:58:51]

The CQ bit was checked by rmcilroy@chromium.org

[rmcilroy, 2015-09-15 23:58:51]

The patchset sent to the CQ was uploaded after l-g-t-m from
isherman@chromium.org
Link to the patchset: https://codereview.chromium.org/1342143002/#ps40001
(title: "Address Ilya's comments.")

[commit-bot: I haz the power, 2015-09-16 00:01:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1342143002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1342143002/40001

[Ilya Sherman, 2015-09-16 00:55:09]

(Still LGTM)

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

https://codereview.chromium.org/1342143002/diff/20001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:50851: +    The amount of time that a
requestIdleCallback callback is allocated before
On 2015/09/15 23:18:59, rmcilroy wrote:
> That wouldn't be quite right since it isn't the callback which allocates the
> time (it get's given a deadline to run within).  Changed to:
> "The amount of time allotted to a requestIdleCallback callback, i.e., the
> difference between the time when it is called and when its deadline expires."
> 
> Is this clearer?

Yep, that's clearer -- thanks!

[commit-bot: I haz the power, 2015-09-16 01:33:25]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-09-16 01:33:25]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[rmcilroy, 2015-09-16 08:19:13]

The CQ bit was checked by rmcilroy@chromium.org

[commit-bot: I haz the power, 2015-09-16 08:19:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1342143002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1342143002/40001

[commit-bot: I haz the power, 2015-09-16 09:59:04]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-09-16 10:00:01]

Patchset 2 (id:??) landed as
https://crrev.com/1e9d97bcc03b5c3a62ca2a68491095350d1ee0ce
Cr-Commit-Position: refs/heads/master@{#349085}

[commit-bot: I haz the power, 2015-09-23 12:54:02]

Patchset 2 (id:??) landed as
https://crrev.com/1e9d97bcc03b5c3a62ca2a68491095350d1ee0ce
Cr-Commit-Position: refs/heads/master@{#349085}