Record forced alternative service histogram.

Record forced alternative service histogram.

This is to track force-alt-protocols command line flag usage to be able to
decide whether it should be kept.

BUG=452514
Committed: https://crrev.com/a0d2ac651f2ccfd3daaa9f9b7bd593e95da04808
Cr-Commit-Position: refs/heads/master@{#321246}

[Bence, 2015-03-17 19:31:27]

bnc@chromium.org changed reviewers:
+ rch@chromium.org

[Bence, 2015-03-17 19:31:28]

Ryan, PTAL.  It seems like https://crrev.com/879863004 does not work at all, I
will revert it, and I propose this CL instead.  It seems to work, at least
locally.

[Ryan Hamilton, 2015-03-17 19:40:41]

lgtm

https://codereview.chromium.org/1016873002/diff/1/tools/metrics/histograms/hi...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/1016873002/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:17723: +  <summary>Track
force-alt-protocols command line flag usage.</summary>
Might want to mention that this is recorded for each request.

[Bence, 2015-03-17 19:46:31]

Thanks.

https://codereview.chromium.org/1016873002/diff/1/tools/metrics/histograms/hi...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/1016873002/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:17723: +  <summary>Track
force-alt-protocols command line flag usage.</summary>
On 2015/03/17 19:40:41, Ryan Hamilton wrote:
> Might want to mention that this is recorded for each request. 

Done.

[Bence, 2015-03-17 19:46:51]

bnc@chromium.org changed reviewers:
+ asvitkine@chromium.org

[Bence, 2015-03-17 19:46:51]

Alexei, PTAL.  Thank you.

[Alexei Svitkine (slow), 2015-03-17 23:12:19]

lgtm % comment

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

https://codereview.chromium.org/1016873002/diff/20001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:17721: +<histogram
name="Net.ForceAlternativeService" enum="Boolean">
Nit: enum="BooleanForced" - add one if it doesn't exist

[Bence, 2015-03-18 16:04:48]

Patchset #3 (id:40001) has been deleted

[Bence, 2015-03-18 16:05:44]

The CQ bit was checked by bnc@chromium.org

[Bence, 2015-03-18 16:05:44]

The patchset sent to the CQ was uploaded after l-g-t-m from rch@chromium.org,
asvitkine@chromium.org
Link to the patchset: https://codereview.chromium.org/1016873002/#ps60001
(title: "Add enum BooleanForced.")

[commit-bot: I haz the power, 2015-03-18 16:05:55]

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

[commit-bot: I haz the power, 2015-03-18 16:39:53]

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

[commit-bot: I haz the power, 2015-03-18 16:39:54]

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

[Bence, 2015-03-18 20:19:53]

The CQ bit was checked by bnc@chromium.org

[Bence, 2015-03-18 20:19:53]

The patchset sent to the CQ was uploaded after l-g-t-m from
asvitkine@chromium.org, rch@chromium.org
Link to the patchset: https://codereview.chromium.org/1016873002/#ps80001
(title: "MSVC doesn't like implicit pointer to bool conversion.")

[commit-bot: I haz the power, 2015-03-18 20:20:32]

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

[commit-bot: I haz the power, 2015-03-19 00:15:39]

Committed patchset #4 (id:80001)

[commit-bot: I haz the power, 2015-03-19 00:16:36]

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