Added new BypassedBytes.Current histograms.

Added new BypassedBytes.Current histograms.

Added new histograms to break down Current bypasses (i.e. "Chrome-Proxy: block-once") by audio/video content types, as well as "content-type: application/octet-stream".

BUG=473763
Committed: https://crrev.com/6fef54c59bcb8192521b0ba0c94c9230e7a93520
Cr-Commit-Position: refs/heads/master@{#324075}

[sclittle, 2015-04-03 21:04:29]

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

[sclittle, 2015-04-03 21:04:30]

PTAL

[bengr, 2015-04-03 22:09:55]

lgtm, modulo nit. Please be more specific in the CL description.

https://codereview.chromium.org/1048303004/diff/1/components/data_reduction_p...
File
components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats_unittest.cc
(right):

https://codereview.chromium.org/1048303004/diff/1/components/data_reduction_p...
components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats_unittest.cc:775:
{"DataReductionProxy.BypassedBytes.Current", "HTTP/1.1 200 OK\r\n\r\n"},
use git cl format

[sclittle, 2015-04-03 22:22:45]

https://codereview.chromium.org/1048303004/diff/1/components/data_reduction_p...
File
components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats_unittest.cc
(right):

https://codereview.chromium.org/1048303004/diff/1/components/data_reduction_p...
components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats_unittest.cc:775:
{"DataReductionProxy.BypassedBytes.Current", "HTTP/1.1 200 OK\r\n\r\n"},
On 2015/04/03 22:09:55, bengr wrote:
> use git cl format

Done.

[sclittle, 2015-04-03 22:22:56]

The CQ bit was checked by sclittle@chromium.org

[sclittle, 2015-04-03 22:22:57]

The patchset sent to the CQ was uploaded after l-g-t-m from bengr@chromium.org
Link to the patchset: https://codereview.chromium.org/1048303004/#ps20001
(title: "Addressed comments")

[commit-bot: I haz the power, 2015-04-03 22:23:18]

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

[commit-bot: I haz the power, 2015-04-03 22:35:58]

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

[commit-bot: I haz the power, 2015-04-03 22:35:59]

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...)

[sclittle, 2015-04-03 23:55:07]

sclittle@chromium.org changed reviewers:
+ asvitkine@google.com

[sclittle, 2015-04-03 23:55:07]

asvitkine: histograms.xml

Thanks in advance!

[Alexei Svitkine (slow), 2015-04-07 15:31:06]

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

[Alexei Svitkine (slow), 2015-04-07 15:31:07]

lgtm

[sclittle, 2015-04-07 17:29:47]

The CQ bit was checked by sclittle@chromium.org

[commit-bot: I haz the power, 2015-04-07 17:30:16]

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

[commit-bot: I haz the power, 2015-04-07 18:29:19]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-04-07 18:30:32]

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