Report data reduction proxy related daily content lengths even if zero

Reporting data reduction proxy related daily content lengths if even they are zeros.

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

[bolian, 2014-06-06 20:20:15]

Hi Ben,

This CL is ready for review. Thanks!

[Matt Welsh, 2014-06-06 20:23:19]

lgtm

[bengr, 2014-06-06 20:41:34]

lgtm, with nit and change the title to "Report data reduction proxy related
daily content lengths even if zero"

https://codereview.chromium.org/324503003/diff/1/components/data_reduction_pr...
File components/data_reduction_proxy/browser/data_reduction_proxy_metrics.cc
(right):

https://codereview.chromium.org/324503003/diff/1/components/data_reduction_pr...
components/data_reduction_proxy/browser/data_reduction_proxy_metrics.cc:151:
DCHECK_GE(received_length_via_data_reduction_proxy, 0);
Move this down to just before the UMA that refers to the variable.

[bolian, 2014-06-06 20:56:18]

Title changed. Thanks.

https://codereview.chromium.org/324503003/diff/1/components/data_reduction_pr...
File components/data_reduction_proxy/browser/data_reduction_proxy_metrics.cc
(right):

https://codereview.chromium.org/324503003/diff/1/components/data_reduction_pr...
components/data_reduction_proxy/browser/data_reduction_proxy_metrics.cc:151:
DCHECK_GE(received_length_via_data_reduction_proxy, 0);
On 2014/06/06 20:41:34, bengr1 wrote:
> Move this down to just before the UMA that refers to the variable.

Done.

[bolian, 2014-06-06 20:56:45]

The CQ bit was checked by bolian@chromium.org

[commit-bot: I haz the power, 2014-06-06 20:58:16]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/bolian@chromium.org/324503003/20001

[commit-bot: I haz the power, 2014-06-07 04:21:30]

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

[commit-bot: I haz the power, 2014-06-07 06:08:03]

Change committed as 275631