Removing unused PreviewsDataSavings

Removing unused PreviewsDataSavings

PreviewsDataSavings is an unused class that is unnecessary. Initially,
this was intended to be used by previews to record savings for various
previews and hook it into the data_reduction_proxy savings logic. This
design doesn't make sense right now for the previews considered, and
previews should track their own savings.

As previews progresses, the data savings can be moved from d_r_p if
necessary into previews, but the dependency from previews to d_r_p is
largely backwards.

BUG=699264
Review-Url: https://codereview.chromium.org/2736953003
Cr-Commit-Position: refs/heads/master@{#455388}
Committed: https://chromium.googlesource.com/chromium/src/+/3f2aebea4889eee1f1d960590605a4544da63f51

[RyanSturm, 2017-03-07 22:21:05]

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

[commit-bot: I haz the power, 2017-03-07 22:21:45]

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

[RyanSturm, 2017-03-07 23:35:19]

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

[commit-bot: I haz the power, 2017-03-07 23:36:19]

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

[RyanSturm, 2017-03-07 23:37:07]

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

[RyanSturm, 2017-03-07 23:37:07]

sclittle: PTAL, make sure you don't need the extra d_r_p paths I am removing. If
you do need them, I can move back to patch set #1.

[sclittle, 2017-03-07 23:40:39]

LGTM! Hooray for code deletion!

[commit-bot: I haz the power, 2017-03-08 00:58:20]

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

[commit-bot: I haz the power, 2017-03-08 00:58:21]

Dry run: This issue passed the CQ dry run.

[RyanSturm, 2017-03-08 05:49:39]

The CQ bit was checked by ryansturm@chromium.org

[commit-bot: I haz the power, 2017-03-08 05:49:55]

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, 2017-03-08 05:56:58]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1488952179295520,
"parent_rev": "b86c2dc8b581ec650806e80b7c112682078ffb0b", "commit_rev":
"3f2aebea4889eee1f1d960590605a4544da63f51"}

[commit-bot: I haz the power, 2017-03-08 05:57:38]

Description was changed from

==========
Removing unused PreviewsDataSavings

PreviewsDataSavings is an unused class that is unnecessary. Initially,
this was intended to be used by previews to record savings for various
previews and hook it into the data_reduction_proxy savings logic. This
design doesn't make sense right now for the previews considered, and
previews should track their own savings.

As previews progresses, the data savings can be moved from d_r_p if
necessary into previews, but the dependency from previews to d_r_p is
largely backwards.

BUG=699264
==========

to

==========
Removing unused PreviewsDataSavings

PreviewsDataSavings is an unused class that is unnecessary. Initially,
this was intended to be used by previews to record savings for various
previews and hook it into the data_reduction_proxy savings logic. This
design doesn't make sense right now for the previews considered, and
previews should track their own savings.

As previews progresses, the data savings can be moved from d_r_p if
necessary into previews, but the dependency from previews to d_r_p is
largely backwards.

BUG=699264

Review-Url: https://codereview.chromium.org/2736953003
Cr-Commit-Position: refs/heads/master@{#455388}
Committed:
https://chromium.googlesource.com/chromium/src/+/3f2aebea4889eee1f1d960590605...
==========

[commit-bot: I haz the power, 2017-03-08 05:57:39]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/3f2aebea4889eee1f1d960590605...