Add a new histogram Net.QuicSession.HeadersStreamEarlyFramesReceived

Add a new histogram Net.QuicSession.HeadersStreamEarlyFramesReceived
for tracking the frames received on the headers stream which arrived early.

Committed: https://crrev.com/7cde50e9d748c56c8c2b1a99d00816df810a6df3
Cr-Commit-Position: refs/heads/master@{#316916}

[Ryan Hamilton, 2015-02-18 18:58:33]

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

[ramant (doing other things), 2015-02-18 19:08:20]

lgtm

[Ryan Hamilton, 2015-02-18 19:26:54]

The CQ bit was checked by rch@chromium.org

[commit-bot: I haz the power, 2015-02-18 19:27:36]

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

[commit-bot: I haz the power, 2015-02-18 19:35:34]

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

[commit-bot: I haz the power, 2015-02-18 19:35:35]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Ryan Hamilton, 2015-02-18 19:41:11]

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

[Ryan Hamilton, 2015-02-18 19:41:12]

+asvitkine: histograms.xml

[Alexei Svitkine (slow), 2015-02-18 19:53:55]

lgtm

https://codereview.chromium.org/937803002/diff/1/net/quic/quic_connection_log...
File net/quic/quic_connection_logger.cc (right):

https://codereview.chromium.org/937803002/diff/1/net/quic/quic_connection_log...
net/quic/quic_connection_logger.cc:372:
UMA_HISTOGRAM_COUNTS("Net.QuicSession.HeadersStreamEarlyFramesReceived",
Optional nit: Maybe more readable if you add another . - i.e.:

Net.QuicSession.HeadersStreamEarly.FramesReceived

[Ryan Hamilton, 2015-02-18 19:56:25]

New patchsets have been uploaded after l-g-t-m from
rtenneti@chromium.org,asvitkine@chromium.org

[Ryan Hamilton, 2015-02-18 19:57:35]

Thanks

https://codereview.chromium.org/937803002/diff/1/net/quic/quic_connection_log...
File net/quic/quic_connection_logger.cc (right):

https://codereview.chromium.org/937803002/diff/1/net/quic/quic_connection_log...
net/quic/quic_connection_logger.cc:372:
UMA_HISTOGRAM_COUNTS("Net.QuicSession.HeadersStreamEarlyFramesReceived",
On 2015/02/18 19:53:55, Alexei Svitkine wrote:
> Optional nit: Maybe more readable if you add another . - i.e.:
> 
> Net.QuicSession.HeadersStreamEarly.FramesReceived

Done. Albeit as: Net.QuicSession.HeadersStream.EarlyFramesReceived

[Ryan Hamilton, 2015-02-18 19:58:13]

The CQ bit was checked by rch@chromium.org

[commit-bot: I haz the power, 2015-02-18 19:58:43]

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

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

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

[commit-bot: I haz the power, 2015-02-18 20:19:33]

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

[Ryan Hamilton, 2015-02-18 21:59:21]

The CQ bit was checked by rch@chromium.org

[commit-bot: I haz the power, 2015-02-18 22:00:31]

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

[commit-bot: I haz the power, 2015-02-18 23:33:16]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-02-18 23:33:48]

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