[HttpCache] Add cache metrics for audio/video behavior

[HttpCache] Add cache metrics for audio/video behavior

Cleans up the content-type metrics in http_cache_transaction and adds metrics
for two new content types, audio and video.

BUG=617620
Committed: https://crrev.com/81c04644f176ac5873c84db8f1cc8193762f88f3
Cr-Commit-Position: refs/heads/master@{#420661}

[jkarlin, 2016-09-19 17:03:16]

The CQ bit was checked by jkarlin@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-19 17:03:43]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jkarlin, 2016-09-19 17:04:25]

jkarlin@chromium.org changed reviewers:
+ gavinp@chromium.org, rkaplow@chromium.org

[jkarlin, 2016-09-19 17:04:26]

gavinp@ PTAL at everything
rkaplow@ PTAL at histograms.xml

Thanks!

[commit-bot: I haz the power, 2016-09-19 17:57:41]

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

[commit-bot: I haz the power, 2016-09-19 17:57:42]

Dry run: This issue passed the CQ dry run.

[rkaplow, 2016-09-19 18:23:03]

lgtm

[jkarlin, 2016-09-19 18:48:59]

Thanks rkaplow. gavinp@ PTAL, thanks!

[jkarlin, 2016-09-22 15:25:09]

gavinp@ PTAL, thank you.

[jkarlin, 2016-09-23 14:19:41]

gavinp@ Ping!

[gavinp, 2016-09-23 15:41:48]

lgtm.

That macro was a bit overdue. Thanks.

https://codereview.chromium.org/2350183002/diff/1/net/http/http_cache_transac...
File net/http/http_cache_transaction.cc (right):

https://codereview.chromium.org/2350183002/diff/1/net/http/http_cache_transac...
net/http/http_cache_transaction.cc:2873: CACHE_STATUS_HISTOGRAMS("");
Does this have to be "", or would () work too? I believe "", right?

[gavinp, 2016-09-23 15:41:49]

lgtm.

That macro was a bit overdue. Thanks.

https://codereview.chromium.org/2350183002/diff/1/net/http/http_cache_transac...
File net/http/http_cache_transaction.cc (right):

https://codereview.chromium.org/2350183002/diff/1/net/http/http_cache_transac...
net/http/http_cache_transaction.cc:2873: CACHE_STATUS_HISTOGRAMS("");
Does this have to be "", or would () work too? I believe "", right?

[jkarlin, 2016-09-23 15:53:02]

Thanks!

https://codereview.chromium.org/2350183002/diff/1/net/http/http_cache_transac...
File net/http/http_cache_transaction.cc (right):

https://codereview.chromium.org/2350183002/diff/1/net/http/http_cache_transac...
net/http/http_cache_transaction.cc:2873: CACHE_STATUS_HISTOGRAMS("");
On 2016/09/23 15:41:49, gavinp wrote:
> Does this have to be "", or would () work too? I believe "", right?
> 

() works. WIll commit with that.

[jkarlin, 2016-09-23 15:54:09]

The CQ bit was checked by jkarlin@chromium.org

[jkarlin, 2016-09-23 15:54:09]

The patchset sent to the CQ was uploaded after l-g-t-m from gavinp@chromium.org,
rkaplow@chromium.org
Link to the patchset: https://codereview.chromium.org/2350183002/#ps20001
(title: "macro nit")

[commit-bot: I haz the power, 2016-09-23 15:54:30]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-23 16:45:00]

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

[commit-bot: I haz the power, 2016-09-23 16:45:01]

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

[jkarlin, 2016-09-23 17:05:22]

https://codereview.chromium.org/2350183002/diff/1/net/http/http_cache_transac...
File net/http/http_cache_transaction.cc (right):

https://codereview.chromium.org/2350183002/diff/1/net/http/http_cache_transac...
net/http/http_cache_transaction.cc:2873: CACHE_STATUS_HISTOGRAMS("");
On 2016/09/23 15:53:02, jkarlin wrote:
> On 2016/09/23 15:41:49, gavinp wrote:
> > Does this have to be "", or would () work too? I believe "", right?
> > 
> 
> () works. WIll commit with that.

Nope, Windows doesn't like it. Reverting.

[jkarlin, 2016-09-23 17:05:30]

Patchset #2 (id:20001) has been deleted

[jkarlin, 2016-09-23 17:05:35]

The CQ bit was checked by jkarlin@chromium.org

[commit-bot: I haz the power, 2016-09-23 17:06:07]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-23 18:09:25]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-23 18:11:48]

Description was changed from

==========
[HttpCache] Add cache metrics for audio/video behavior

Cleans up the content-type metrics in http_cache_transaction and adds metrics
for two new content types, audio and video.

BUG=617620
==========

to

==========
[HttpCache] Add cache metrics for audio/video behavior

Cleans up the content-type metrics in http_cache_transaction and adds metrics
for two new content types, audio and video.

BUG=617620

Committed: https://crrev.com/81c04644f176ac5873c84db8f1cc8193762f88f3
Cr-Commit-Position: refs/heads/master@{#420661}
==========

[commit-bot: I haz the power, 2016-09-23 18:11:49]

Patchset 1 (id:??) landed as
https://crrev.com/81c04644f176ac5873c84db8f1cc8193762f88f3
Cr-Commit-Position: refs/heads/master@{#420661}