Add data usage tracking for data reduction proxy, precache services

Add data usage tracking for data reduction proxy, precache services

BUG=527304
Committed: https://crrev.com/e74a4dcf845e3302a6c65d03fb3490e3fd3626ef
Cr-Commit-Position: refs/heads/master@{#421130}

[Raj, 2016-08-24 18:45:14]

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

[bengr, 2016-08-26 16:53:00]

https://codereview.chromium.org/2274243002/diff/1/components/data_use_measure...
File components/data_use_measurement/core/data_use_user_data.h (right):

https://codereview.chromium.org/2274243002/diff/1/components/data_use_measure...
components/data_use_measurement/core/data_use_user_data.h:43:
DATA_REDUCTION_PROXY,
Please update DataUseServices in histograms.xml

[Raj, 2016-09-22 18:59:30]

rajendrant@chromium.org changed reviewers:
+ rkaplow@chromium.org

[Raj, 2016-09-22 18:59:30]

https://codereview.chromium.org/2274243002/diff/1/components/data_use_measure...
File components/data_use_measurement/core/data_use_user_data.h (right):

https://codereview.chromium.org/2274243002/diff/1/components/data_use_measure...
components/data_use_measurement/core/data_use_user_data.h:43:
DATA_REDUCTION_PROXY,
On 2016/08/26 16:53:00, bengr wrote:
> Please update DataUseServices in histograms.xml

Done.

[rkaplow, 2016-09-22 19:01:34]

Description was changed from

==========
Add data usage tracking for data reduction proxy, precache services

BUG=527304
==========

to

==========
Add data usage tracking for data reduction proxy, precache services

BUG=527304
==========

[rkaplow, 2016-09-22 19:01:35]

rkaplow@chromium.org changed reviewers:
+ gayane@chromium.org
- rkaplow@chromium.org

[bengr, 2016-09-23 20:14:06]

lgtm. But this would be better if URLFetcher could be constructed with a label,
so that all services that use URLFetcher would be tracked.

[gayane -on leave until 09-2017, 2016-09-26 16:06:05]

lgtm

[Raj, 2016-09-26 16:45:57]

The CQ bit was checked by rajendrant@chromium.org

[commit-bot: I haz the power, 2016-09-26 16:46:26]

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-26 16:57:06]

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

[commit-bot: I haz the power, 2016-09-26 16:57:07]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)

[Raj, 2016-09-26 18:39:27]

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

[commit-bot: I haz the power, 2016-09-26 18:40:18]

Dry run: 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-26 18:47:49]

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

[commit-bot: I haz the power, 2016-09-26 18:47:50]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Raj, 2016-09-26 20:39:39]

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

[commit-bot: I haz the power, 2016-09-26 20:40:35]

Dry run: 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-26 20:47:18]

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

[commit-bot: I haz the power, 2016-09-26 20:47:19]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[Raj, 2016-09-26 21:04:48]

Patchset #4 (id:60001) has been deleted

[Raj, 2016-09-26 21:21:17]

Patchset #3 (id:40001) has been deleted

[Raj, 2016-09-26 21:23:25]

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

[commit-bot: I haz the power, 2016-09-26 21:23:54]

Dry run: 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-26 21:33:28]

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

[commit-bot: I haz the power, 2016-09-26 21:33:29]

Dry run: Try jobs failed on following builders:
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[Raj, 2016-09-26 21:34:09]

Patchset #3 (id:80001) has been deleted

[Raj, 2016-09-26 21:35:44]

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

[commit-bot: I haz the power, 2016-09-26 21:36:36]

Dry run: 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-26 21:43:59]

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

[commit-bot: I haz the power, 2016-09-26 21:44:00]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Raj, 2016-09-26 22:16:57]

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

[commit-bot: I haz the power, 2016-09-26 22:17:48]

Dry run: 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-26 23:25:27]

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

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

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

[Raj, 2016-09-27 04:19:35]

Patchset #4 (id:120001) has been deleted

[Raj, 2016-09-27 04:19:44]

Patchset #3 (id:100001) has been deleted

[Raj, 2016-09-27 04:20:16]

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

[commit-bot: I haz the power, 2016-09-27 04:20:43]

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

[Raj, 2016-09-27 05:41:34]

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

[commit-bot: I haz the power, 2016-09-27 05:41:58]

Dry run: 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-27 06:29:50]

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

[commit-bot: I haz the power, 2016-09-27 06:29:51]

Dry run: This issue passed the CQ dry run.

[Raj, 2016-09-27 06:33:18]

The CQ bit was checked by rajendrant@chromium.org

[Raj, 2016-09-27 06:33:18]

The patchset sent to the CQ was uploaded after l-g-t-m from bengr@chromium.org,
gayane@chromium.org
Link to the patchset: https://codereview.chromium.org/2274243002/#ps160001
(title: "rebased")

[commit-bot: I haz the power, 2016-09-27 06:33:27]

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-27 06:38:43]

Description was changed from

==========
Add data usage tracking for data reduction proxy, precache services

BUG=527304
==========

to

==========
Add data usage tracking for data reduction proxy, precache services

BUG=527304
==========

[commit-bot: I haz the power, 2016-09-27 06:38:44]

Committed patchset #4 (id:160001)

[commit-bot: I haz the power, 2016-09-27 06:40:07]

Description was changed from

==========
Add data usage tracking for data reduction proxy, precache services

BUG=527304
==========

to

==========
Add data usage tracking for data reduction proxy, precache services

BUG=527304

Committed: https://crrev.com/e74a4dcf845e3302a6c65d03fb3490e3fd3626ef
Cr-Commit-Position: refs/heads/master@{#421130}
==========

[commit-bot: I haz the power, 2016-09-27 06:40:09]

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