HttpServerProperties - Added a histogram to track the AlternateProtocolServers

HttpServerProperties - Added a histogram to track the AlternateProtocolServers
that are in memory.

R=asvitkine@chromium.org, rch@chromium.org

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

[ramant (doing other things), 2015-12-10 19:51:13]

Description was changed from

==========
HttpServerProperties - Added a histogram to track the AlternateProtocolServers
that are in memory.

R=asvitkine@chromium.org, rch@chromium.org
==========

to

==========
HttpServerProperties - Added a histogram to track the AlternateProtocolServers
that are in memory.

R=asvitkine@chromium.org, rch@chromium.org
==========

[Ryan Hamilton, 2015-12-10 19:54:45]

lgtm

[Alexei Svitkine (slow), 2015-12-10 20:35:33]

lgtm

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

https://codereview.chromium.org/1506353008/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:22722: +<histogram
name="Net.CountOfAlternateProtocolServers.Memory">
Add units="servers".

https://codereview.chromium.org/1506353008/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:22727: +    The total number of severs
(in memory) to which alternative protocol was
servers?

[ramant (doing other things), 2015-12-10 22:54:45]

Hi Alexei,
  Would appreciate if you could take another look.

thanks
raman

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

https://codereview.chromium.org/1506353008/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:22722: +<histogram
name="Net.CountOfAlternateProtocolServers.Memory">
On 2015/12/10 20:35:32, Alexei Svitkine (slow) wrote:
> Add units="servers".

Done.

https://codereview.chromium.org/1506353008/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:22727: +    The total number of severs
(in memory) to which alternative protocol was
On 2015/12/10 20:35:32, Alexei Svitkine (slow) wrote:
> servers?

It is HostPortPairs. Added that to the description.

Done.

[Alexei Svitkine (slow), 2015-12-11 16:28:24]

lgtm

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

https://codereview.chromium.org/1506353008/diff/20001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:22727: +    The total number of severs
(HostPortPairs in memory) to which alternative
Nit: severs -> servers

Fix typo throughout.

[ramant (doing other things), 2015-12-11 16:55:54]

Thanks Alexei.

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

https://codereview.chromium.org/1506353008/diff/20001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:22727: +    The total number of severs
(HostPortPairs in memory) to which alternative
On 2015/12/11 16:28:24, Alexei Svitkine (slow) wrote:
> Nit: severs -> servers
> 
> Fix typo throughout.

Done.

[ramant (doing other things), 2015-12-11 16:56:36]

The CQ bit was checked by rtenneti@chromium.org

[ramant (doing other things), 2015-12-11 16:56:37]

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/1506353008/#ps40001
(title: "fixed typos")

[commit-bot: I haz the power, 2015-12-11 16:58:01]

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

[commit-bot: I haz the power, 2015-12-11 18:30:39]

Description was changed from

==========
HttpServerProperties - Added a histogram to track the AlternateProtocolServers
that are in memory.

R=asvitkine@chromium.org, rch@chromium.org
==========

to

==========
HttpServerProperties - Added a histogram to track the AlternateProtocolServers
that are in memory.

R=asvitkine@chromium.org, rch@chromium.org
==========

[commit-bot: I haz the power, 2015-12-11 18:30:39]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-12-11 18:31:24]

Description was changed from

==========
HttpServerProperties - Added a histogram to track the AlternateProtocolServers
that are in memory.

R=asvitkine@chromium.org, rch@chromium.org
==========

to

==========
HttpServerProperties - Added a histogram to track the AlternateProtocolServers
that are in memory.

R=asvitkine@chromium.org, rch@chromium.org

Committed: https://crrev.com/f635e8507d82be6c5e82ab3d145ca0004510f919
Cr-Commit-Position: refs/heads/master@{#364744}
==========

[commit-bot: I haz the power, 2015-12-11 18:31:24]

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