DataReductionProxy.BypassedBytes.ShortAudioVideo UMA reports collateral damage

Fixing DataReductionProxy.BypassedBytes.ShortAudioVideo UMA.
Currently the UMA tells us whenever audio/video is bypassed as collateral damage of a short bypass. 
We want to know when audio or video is the cause of a bypass so we now report to 
DataReductionProxy.BypassedBytes.ShortTriggeringRequestAudioVideo when audio or video triggers a bypass.

BUG=401257
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=289569

[megjablon, 2014-08-06 21:57:27]

asvitkine: histograms.xml
bengr: data_reduction_proxy_usage_stats.cc

[Alexei Svitkine (slow), 2014-08-08 15:34:52]

https://codereview.chromium.org/447933002/diff/20001/tools/metrics/histograms...
File tools/metrics/histograms/histograms.xml (left):

https://codereview.chromium.org/447933002/diff/20001/tools/metrics/histograms...
tools/metrics/histograms/histograms.xml:49426: -  <suffix name="ShortAudioVideo"
Was any data collected yet using this suffix? If yes, then please keep the old
one (and say it's obsolete in the label).

[megjablon, 2014-08-12 01:02:49]

https://codereview.chromium.org/447933002/diff/20001/tools/metrics/histograms...
File tools/metrics/histograms/histograms.xml (left):

https://codereview.chromium.org/447933002/diff/20001/tools/metrics/histograms...
tools/metrics/histograms/histograms.xml:49426: -  <suffix name="ShortAudioVideo"
On 2014/08/08 15:34:52, Alexei Svitkine wrote:
> Was any data collected yet using this suffix? If yes, then please keep the old
> one (and say it's obsolete in the label).

I'll just keep the name the same and switch the description.

[Alexei Svitkine (slow), 2014-08-12 14:32:37]

https://codereview.chromium.org/447933002/diff/40001/components/data_reductio...
File components/data_reduction_proxy/browser/data_reduction_proxy_usage_stats.cc
(right):

https://codereview.chromium.org/447933002/diff/40001/components/data_reductio...
components/data_reduction_proxy/browser/data_reduction_proxy_usage_stats.cc:196:
"DataReductionProxy.BypassedBytes.ShortTriggeringRequestAudioVideo",
Did you mean to not rename this?

[megjablon, 2014-08-12 16:50:09]

On 2014/08/12 14:32:37, Alexei Svitkine wrote:
>
https://codereview.chromium.org/447933002/diff/40001/components/data_reductio...
> File
components/data_reduction_proxy/browser/data_reduction_proxy_usage_stats.cc
> (right):
> 
>
https://codereview.chromium.org/447933002/diff/40001/components/data_reductio...
>
components/data_reduction_proxy/browser/data_reduction_proxy_usage_stats.cc:196:
> "DataReductionProxy.BypassedBytes.ShortTriggeringRequestAudioVideo",
> Did you mean to not rename this?

Yes. I thought the new name was too wordy so I went back to the old and switched
the description.

[megjablon, 2014-08-12 16:53:59]

https://codereview.chromium.org/447933002/diff/40001/components/data_reductio...
File components/data_reduction_proxy/browser/data_reduction_proxy_usage_stats.cc
(right):

https://codereview.chromium.org/447933002/diff/40001/components/data_reductio...
components/data_reduction_proxy/browser/data_reduction_proxy_usage_stats.cc:196:
"DataReductionProxy.BypassedBytes.ShortTriggeringRequestAudioVideo",
On 2014/08/12 14:32:37, Alexei Svitkine wrote:
> Did you mean to not rename this?
Ah sorry didn't mean to send that message yet. I thought this comment was in a
different location. It's fixed now.

[Alexei Svitkine (slow), 2014-08-12 18:21:31]

lgtm

[bengr, 2014-08-14 01:34:07]

On 2014/08/12 18:21:31, Alexei Svitkine wrote:
> lgtm

lgtm.

[megjablon, 2014-08-14 01:35:02]

The CQ bit was checked by megjablon@chromium.org

[commit-bot: I haz the power, 2014-08-14 01:37:41]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/megjablon@chromium.org/447933002/60001

[megjablon, 2014-08-14 06:24:48]

The CQ bit was checked by megjablon@chromium.org

[commit-bot: I haz the power, 2014-08-14 06:27:53]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/megjablon@chromium.org/447933002/80001

[commit-bot: I haz the power, 2014-08-14 12:31:26]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_chromium_chromeos_rel_swarming on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[commit-bot: I haz the power, 2014-08-14 14:43:50]

Committed patchset #4 (80001) as 289569