Use the 30 day history stats for the Data Saver menu item

Use the 30 day history stats for the Data Saver menu item

The Data Saver menu item should display the savings for the last 30 days
rather than lifetime savings.

BUG=702799
Review-Url: https://codereview.chromium.org/2808653004
Cr-Commit-Position: refs/heads/master@{#463699}
Committed: https://chromium.googlesource.com/chromium/src/+/dc6f2e8e9f1c909c48d403c8e5a183a8769f2c03

[megjablon, 2017-04-11 01:55:26]

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

[commit-bot: I haz the power, 2017-04-11 01:56:14]

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

[megjablon, 2017-04-11 01:56:40]

megjablon@chromium.org changed reviewers:
+ dtrainor@chromium.org

[megjablon, 2017-04-11 01:56:41]

PTAL, thanks!

[commit-bot: I haz the power, 2017-04-11 02:47:39]

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

[commit-bot: I haz the power, 2017-04-11 02:47:40]

Dry run: This issue passed the CQ dry run.

[David Trainor- moved to gerrit, 2017-04-11 05:04:03]

lgtm % nit

https://codereview.chromium.org/2808653004/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/net/spdyproxy/DataReductionProxySettings.java
(right):

https://codereview.chromium.org/2808653004/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/net/spdyproxy/DataReductionProxySettings.java:215:
savings = length.getOriginal() - length.getReceived();
long savings = std::max(length.getOriginal() - length.getReceived(), 0)?

[megjablon, 2017-04-11 17:29:48]

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

[megjablon, 2017-04-11 17:30:03]

https://codereview.chromium.org/2808653004/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/net/spdyproxy/DataReductionProxySettings.java
(right):

https://codereview.chromium.org/2808653004/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/net/spdyproxy/DataReductionProxySettings.java:215:
savings = length.getOriginal() - length.getReceived();
On 2017/04/11 05:04:03, David Trainor-ping if over 24h wrote:
> long savings = std::max(length.getOriginal() - length.getReceived(), 0)?

Done.

[commit-bot: I haz the power, 2017-04-11 17:30:42]

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, 2017-04-11 18:14:48]

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

[commit-bot: I haz the power, 2017-04-11 18:14:49]

Dry run: This issue passed the CQ dry run.

[megjablon, 2017-04-11 18:30:53]

The CQ bit was checked by megjablon@chromium.org

[megjablon, 2017-04-11 18:30:53]

The patchset sent to the CQ was uploaded after l-g-t-m from
dtrainor@chromium.org
Link to the patchset: https://codereview.chromium.org/2808653004/#ps20001
(title: "dtrainor comment")

[commit-bot: I haz the power, 2017-04-11 18:31:41]

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-04-11 18:41:22]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1491935453410470,
"parent_rev": "ca6e72620f092e950e6ce788bde57455a4e73eb1", "commit_rev":
"dc6f2e8e9f1c909c48d403c8e5a183a8769f2c03"}

[commit-bot: I haz the power, 2017-04-11 18:42:20]

Description was changed from

==========
Use the 30 day history stats for the Data Saver menu item

The Data Saver menu item should display the savings for the last 30 days
rather than lifetime savings.

BUG=702799
==========

to

==========
Use the 30 day history stats for the Data Saver menu item

The Data Saver menu item should display the savings for the last 30 days
rather than lifetime savings.

BUG=702799

Review-Url: https://codereview.chromium.org/2808653004
Cr-Commit-Position: refs/heads/master@{#463699}
Committed:
https://chromium.googlesource.com/chromium/src/+/dc6f2e8e9f1c909c48d403c8e5a1...
==========

[commit-bot: I haz the power, 2017-04-11 18:42:21]

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