Backup and restore data reduction state

Backup and restore data reduction state

Save the data reduction proxy preferences in backups. Both the Java
and the C++ preferences are needed, since the Java pref is used during
startup.

BUG=705501
Review-Url: https://codereview.chromium.org/2843963002
Cr-Commit-Position: refs/heads/master@{#467624}
Committed: https://chromium.googlesource.com/chromium/src/+/07cbcdc2a6293bba57b711d5d449c91a0a0541bc

[aberent, 2017-04-26 13:53:46]

aberent@chromium.org changed reviewers:
+ bauerb@chromium.org

[aberent, 2017-04-26 13:53:47]

https://codereview.chromium.org/2843963002/diff/1/chrome/android/java/src/org...
File chrome/android/java/src/org/chromium/chrome/browser/ChromeBackupAgent.java
(right):

https://codereview.chromium.org/2843963002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/ChromeBackupAgent.java:83:
FirstRunGlueImpl.CACHED_TOS_ACCEPTED_PREF,
FirstRunStatus.FIRST_RUN_FLOW_COMPLETE,
Formatting change done by git cl format. Pre-submit objects if I don't run git
cl format.

[aberent, 2017-04-26 14:15:33]

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

[commit-bot: I haz the power, 2017-04-26 14:15:53]

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-26 14:49:16]

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

[commit-bot: I haz the power, 2017-04-26 14:49:17]

Dry run: This issue passed the CQ dry run.

[Bernhard Bauer, 2017-04-27 08:55:17]

lgtm

https://codereview.chromium.org/2843963002/diff/1/chrome/android/java/src/org...
File chrome/android/java/src/org/chromium/chrome/browser/ChromeBackupAgent.java
(right):

https://codereview.chromium.org/2843963002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/ChromeBackupAgent.java:83:
FirstRunGlueImpl.CACHED_TOS_ACCEPTED_PREF,
FirstRunStatus.FIRST_RUN_FLOW_COMPLETE,
On 2017/04/26 13:53:47, aberent wrote:
> Formatting change done by git cl format. Pre-submit objects if I don't run git
> cl format.

You could try adding a comment at the end of a line you don't want to be broken.
Otherwise I don't care too much.

[aberent, 2017-04-27 09:00:17]

The CQ bit was checked by aberent@chromium.org

[aberent, 2017-04-27 09:00:30]

https://codereview.chromium.org/2843963002/diff/1/chrome/android/java/src/org...
File chrome/android/java/src/org/chromium/chrome/browser/ChromeBackupAgent.java
(right):

https://codereview.chromium.org/2843963002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/ChromeBackupAgent.java:83:
FirstRunGlueImpl.CACHED_TOS_ACCEPTED_PREF,
FirstRunStatus.FIRST_RUN_FLOW_COMPLETE,
On 2017/04/27 08:55:17, Bernhard Bauer wrote:
> On 2017/04/26 13:53:47, aberent wrote:
> > Formatting change done by git cl format. Pre-submit objects if I don't run
git
> > cl format.
> 
> You could try adding a comment at the end of a line you don't want to be
broken.
> Otherwise I don't care too much.

Landing as is. May report a bug on git cl format if I can be bothered.

[commit-bot: I haz the power, 2017-04-27 09:00:38]

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-27 09:04:32]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493283617547570, "parent_rev":
"4a97922bec93566369465c2e3d7facc45d165005", "commit_rev":
"07cbcdc2a6293bba57b711d5d449c91a0a0541bc"}

[commit-bot: I haz the power, 2017-04-27 09:04:42]

Description was changed from

==========
Backup and restore data reduction state

Save the data reduction proxy preferences in backups. Both the Java
and the C++ preferences are needed, since the Java pref is used during
startup.

BUG=705501
==========

to

==========
Backup and restore data reduction state

Save the data reduction proxy preferences in backups. Both the Java
and the C++ preferences are needed, since the Java pref is used during
startup.

BUG=705501

Review-Url: https://codereview.chromium.org/2843963002
Cr-Commit-Position: refs/heads/master@{#467624}
Committed:
https://chromium.googlesource.com/chromium/src/+/07cbcdc2a6293bba57b711d5d449...
==========

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

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/07cbcdc2a6293bba57b711d5d449...