Adding a DataSavingsRecorder interface

Adding a DataSavingsRecorder interface

This interface is implemented by DataReductionProxyCompressionStats and
provides a way for interventions that save data to report data savings
after the fact.

All URLRequests data are recorded, and when data saver is enabled, data
savings is also recorded.

BUG=615565
Committed: https://crrev.com/d6cf9b0695ef2f04cb7c3878caf45879d486d8de
Cr-Commit-Position: refs/heads/master@{#413264}

[RyanSturm, 2016-08-16 23:56:05]

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

[commit-bot: I haz the power, 2016-08-16 23:56:29]

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

[RyanSturm, 2016-08-16 23:59:38]

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

[commit-bot: I haz the power, 2016-08-17 00:00:04]

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

[RyanSturm, 2016-08-17 00:11:44]

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

[commit-bot: I haz the power, 2016-08-17 00:11: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-08-17 01:14:03]

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

[commit-bot: I haz the power, 2016-08-17 01:14:04]

Dry run: This issue passed the CQ dry run.

[RyanSturm, 2016-08-17 22:11:16]

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

[RyanSturm, 2016-08-17 22:11:16]

bengr: PTAL @ *

[RyanSturm, 2016-08-17 22:22:56]

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

[commit-bot: I haz the power, 2016-08-17 22:23:27]

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-08-17 23:28:00]

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

[commit-bot: I haz the power, 2016-08-17 23:28:01]

Dry run: This issue passed the CQ dry run.

[bengr, 2016-08-19 20:35:35]

Lgtm with a comment nit.

https://codereview.chromium.org/2249903004/diff/60001/components/data_reducti...
File
components/data_reduction_proxy/core/browser/data_reduction_proxy_compression_stats.cc
(right):

https://codereview.chromium.org/2249903004/diff/60001/components/data_reducti...
components/data_reduction_proxy/core/browser/data_reduction_proxy_compression_stats.cc:1018:
// Don't record any request level prefs.
Why not? Maybe more of a comment is necessary.

[RyanSturm, 2016-08-19 20:45:13]

The CQ bit was checked by ryansturm@chromium.org

[RyanSturm, 2016-08-19 20:45:14]

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/2249903004/#ps80001
(title: "bengr comments")

[commit-bot: I haz the power, 2016-08-19 20:45:48]

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-08-19 22:12:15]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-08-19 22:14:56]

Description was changed from

==========
Adding a DataSavingsRecorder interface

This interface is implemented by DataReductionProxyCompressionStats and
provides a way for interventions that save data to report data savings
after the fact.

All URLRequests data are recorded, and when data saver is enabled, data
savings is also recorded.

BUG=615565
==========

to

==========
Adding a DataSavingsRecorder interface

This interface is implemented by DataReductionProxyCompressionStats and
provides a way for interventions that save data to report data savings
after the fact.

All URLRequests data are recorded, and when data saver is enabled, data
savings is also recorded.

BUG=615565

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

[commit-bot: I haz the power, 2016-08-19 22:14:57]

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