Include HTTP header bytes in GetTotalSent/ReceivedBytes for QUIC.

Include HTTP header bytes in GetTotalSent/ReceivedBytes for QUIC.

This change includes the size of the SPDY request/response header frames
sent/received for a QuicHttpStream in
QuicHttpStream.GetTotalSent/ReceivedBytes.

BUG=518894
Committed: https://crrev.com/c4dc1a3ab9a0ff90ec68110c0c18688d98c63446
Cr-Commit-Position: refs/heads/master@{#350404}

[sclittle, 2015-09-23 01:46:44]

Patchset #1 (id:1) has been deleted

[sclittle, 2015-09-23 01:48:01]

sclittle@chromium.org changed reviewers:
+ bengr@chromium.org, rch@chromium.org

[sclittle, 2015-09-23 01:48:01]

PTAL

[Ryan Hamilton, 2015-09-23 02:53:26]

LGTM, module one tweak.

https://codereview.chromium.org/1360063002/diff/20001/net/quic/quic_http_stre...
File net/quic/quic_http_stream.h (right):

https://codereview.chromium.org/1360063002/diff/20001/net/quic/quic_http_stre...
net/quic/quic_http_stream.h:151: int64_t headers_stream_sent_bytes_;
nit: I would name these: headers_bytes_sent_ and headers_bytes_received_. The
"headers stream" is a well defined thing, of course, but it will have
sent/received more bytes than those for just this stream

[sclittle, 2015-09-23 20:59:27]

The CQ bit was checked by sclittle@chromium.org

[sclittle, 2015-09-23 20:59:28]

The patchset sent to the CQ was uploaded after l-g-t-m from rch@chromium.org
Link to the patchset: https://codereview.chromium.org/1360063002/#ps40001
(title: "Addressed nits")

[commit-bot: I haz the power, 2015-09-23 21:00:56]

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

[commit-bot: I haz the power, 2015-09-23 21:25:57]

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

[commit-bot: I haz the power, 2015-09-23 21:25:58]

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

[sclittle, 2015-09-23 21:26:44]

The CQ bit was checked by sclittle@chromium.org

[commit-bot: I haz the power, 2015-09-23 21:28:00]

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

[commit-bot: I haz the power, 2015-09-23 22:01:55]

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

[commit-bot: I haz the power, 2015-09-23 22:01:56]

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

[sclittle, 2015-09-23 22:04:03]

The CQ bit was checked by sclittle@chromium.org

[commit-bot: I haz the power, 2015-09-23 22:06:27]

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

[commit-bot: I haz the power, 2015-09-23 22:36:53]

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

[commit-bot: I haz the power, 2015-09-23 22:36:53]

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

[sclittle, 2015-09-23 23:02:51]

The CQ bit was checked by sclittle@chromium.org

[commit-bot: I haz the power, 2015-09-23 23:07:51]

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

[commit-bot: I haz the power, 2015-09-24 00:15:53]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-09-24 00:35:01]

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