🎾 Migrate more shared preferences to ContextUtils.

Migrate more shared preferences to ContextUtils.

First CL for chrome/: https://codereview.chromium.org/1874423002/

This way of accessing app-wide shared preferences will be required
in a follow-up presubmit CL.

BUG=599284
Committed: https://crrev.com/320db57a6fbcde9610944375d677d9b848c60d5f
Cr-Commit-Position: refs/heads/master@{#394408}

[Peter Wen, 2016-05-11 14:10:47]

Description was changed from

==========
f

t

BUG=
==========

to

==========
Migrate more shared preferences to ContextUtils.

First CL for chrome/: https://codereview.chromium.org/1874423002/

This way of accessing app-wide shared preferences will be required
in a follow-up presubmit CL.

BUG=599284
==========

[Peter Wen, 2016-05-11 14:13:23]

Description was changed from

==========
Migrate more shared preferences to ContextUtils.

First CL for chrome/: https://codereview.chromium.org/1874423002/

This way of accessing app-wide shared preferences will be required
in a follow-up presubmit CL.

BUG=599284
==========

to

==========
Migrate more shared preferences to ContextUtils.

First CL for chrome/: https://codereview.chromium.org/1874423002/

This way of accessing app-wide shared preferences will be required
in a follow-up presubmit CL.

BUG=599284
==========

[Peter Wen, 2016-05-11 14:16:57]

wnwen@chromium.org changed reviewers:
+ agrieve@chromium.org

[Peter Wen, 2016-05-11 14:16:58]

Hey Andrew,

Could you take a look at this CL? After this CL I plan to add a presubmit to
disallow calling PreferenceManager.getDefaultSharedPreferences. :)

Thanks, 🏁

Peter

[agrieve, 2016-05-11 17:57:13]

lgtm!

[Peter Wen, 2016-05-12 13:43:39]

wnwen@chromium.org changed reviewers:
+ nyquist@chromium.org, rkaplow@chromium.org, tedchoc@chromium.org,
tnagel@chromium.org

[Peter Wen, 2016-05-12 13:43:39]

+rkaplow - for components/variations/ OWNERS
+tnagel - for components/policy/ OWNERS
+tedchoc - for ui/android/ and chrome/android/ OWNERS
+nyquist - for blimp/ and components/invalidations and sync/android/ OWNERS

[Ted C, 2016-05-12 16:42:31]

On 2016/05/12 13:43:39, Peter Wen wrote:
> +rkaplow - for components/variations/ OWNERS
> +tnagel - for components/policy/ OWNERS
> +tedchoc - for ui/android/ and chrome/android/ OWNERS
> +nyquist - for blimp/ and components/invalidations and sync/android/ OWNERS

chrome/ ui/ - lgtm

[nyquist, 2016-05-12 18:01:18]

lgtm

[Peter Wen, 2016-05-16 18:46:50]

wnwen@chromium.org changed reviewers:
- tnagel@chromium.org

[Peter Wen, 2016-05-16 18:46:50]

-tnagel OOO.

[Peter Wen, 2016-05-16 18:49:37]

wnwen@chromium.org changed reviewers:
+ pastarmovj@chromium.org

[Peter Wen, 2016-05-16 18:49:38]

+pastarmovj for components/policy/ OWNERS.

@rkaplow - Friendly ping for components/variations/ OWNERS.

🏃

[rkaplow, 2016-05-16 19:22:38]

lgtm

[pastarmovj, 2016-05-17 22:09:06]

components/policy LGTM

[Peter Wen, 2016-05-18 12:34:45]

The CQ bit was checked by wnwen@chromium.org

[commit-bot: I haz the power, 2016-05-18 12:34:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1965093002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1965093002/20001

[commit-bot: I haz the power, 2016-05-18 12:36:33]

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

[commit-bot: I haz the power, 2016-05-18 12:36:34]

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

[Peter Wen, 2016-05-18 12:56:22]

The CQ bit was checked by wnwen@chromium.org

[Peter Wen, 2016-05-18 12:56:22]

The patchset sent to the CQ was uploaded after l-g-t-m from
tedchoc@chromium.org, nyquist@chromium.org, pastarmovj@chromium.org,
rkaplow@chromium.org, agrieve@chromium.org
Link to the patchset: https://codereview.chromium.org/1965093002/#ps40001
(title: "Rebase.")

[commit-bot: I haz the power, 2016-05-18 12:56:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1965093002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1965093002/40001

[commit-bot: I haz the power, 2016-05-18 13:52:03]

Description was changed from

==========
Migrate more shared preferences to ContextUtils.

First CL for chrome/: https://codereview.chromium.org/1874423002/

This way of accessing app-wide shared preferences will be required
in a follow-up presubmit CL.

BUG=599284
==========

to

==========
Migrate more shared preferences to ContextUtils.

First CL for chrome/: https://codereview.chromium.org/1874423002/

This way of accessing app-wide shared preferences will be required
in a follow-up presubmit CL.

BUG=599284
==========

[commit-bot: I haz the power, 2016-05-18 13:52:05]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-18 13:53:29]

Description was changed from

==========
Migrate more shared preferences to ContextUtils.

First CL for chrome/: https://codereview.chromium.org/1874423002/

This way of accessing app-wide shared preferences will be required
in a follow-up presubmit CL.

BUG=599284
==========

to

==========
Migrate more shared preferences to ContextUtils.

First CL for chrome/: https://codereview.chromium.org/1874423002/

This way of accessing app-wide shared preferences will be required
in a follow-up presubmit CL.

BUG=599284

Committed: https://crrev.com/320db57a6fbcde9610944375d677d9b848c60d5f
Cr-Commit-Position: refs/heads/master@{#394408}
==========

[commit-bot: I haz the power, 2016-05-18 13:53:31]

Patchset 3 (id:??) landed as
https://crrev.com/320db57a6fbcde9610944375d677d9b848c60d5f
Cr-Commit-Position: refs/heads/master@{#394408}