Do not sync the ARC++ opt-in preference.

Do not sync the ARC++ opt-in preference.

This is a re-implementation of 1c7841e8e0322b8c3599710e017ff0fb38ab056f,
which made ARC opt-in preference not synced.  That change also stopped
sending UMA stats due to not calling the OnSyncedPrefChanged method.

BUG=633017
TEST=browser_tests --gtest_filter=ArcAuthServiceTest.*
Additionally manually tested that the normal opt-in/opt-out process
worked as expected.
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2206883002
Cr-Commit-Position: refs/heads/master@{#410994}
(cherry picked from commit 176f6733ab5bdf6de91f08e840b814cf76b910bb)

[dspaid, 2016-08-12 05:47:21]

dspaid@chromium.org changed reviewers:
+ hidehiko@chromium.org

[dspaid, 2016-08-12 05:47:21]

This is the CP for the patch you reviewed the other day.

[hidehiko, 2016-08-12 12:08:15]

lgtm

[dspaid, 2016-08-14 22:53:42]

The CQ bit was checked by dspaid@chromium.org

[commit-bot: I haz the power, 2016-08-14 22:53:50]

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-14 22:53:51]

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

[commit-bot: I haz the power, 2016-08-14 22:53:52]

CLs for remote refs other than refs/pending/heads/master must contain NOTRY=true
and NOPRESUBMIT=true in order for the CQ to process them

[dspaid, 2016-08-14 22:59:57]

Description was changed from

==========
Do not sync the ARC++ opt-in preference.

This is a re-implementation of 1c7841e8e0322b8c3599710e017ff0fb38ab056f,
which made ARC opt-in preference not synced.  That change also stopped
sending UMA stats due to not calling the OnSyncedPrefChanged method.

BUG=633017
TEST=browser_tests --gtest_filter=ArcAuthServiceTest.*
Additionally manually tested that the normal opt-in/opt-out process
worked as expected.

Review-Url: https://codereview.chromium.org/2206883002
Cr-Commit-Position: refs/heads/master@{#410994}
(cherry picked from commit 176f6733ab5bdf6de91f08e840b814cf76b910bb)
==========

to

==========
Do not sync the ARC++ opt-in preference.

This is a re-implementation of 1c7841e8e0322b8c3599710e017ff0fb38ab056f,
which made ARC opt-in preference not synced.  That change also stopped
sending UMA stats due to not calling the OnSyncedPrefChanged method.

BUG=633017
TEST=browser_tests --gtest_filter=ArcAuthServiceTest.*
Additionally manually tested that the normal opt-in/opt-out process
worked as expected.
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2206883002
Cr-Commit-Position: refs/heads/master@{#410994}
(cherry picked from commit 176f6733ab5bdf6de91f08e840b814cf76b910bb)
==========

[dspaid, 2016-08-14 23:00:07]

The CQ bit was checked by dspaid@chromium.org

[commit-bot: I haz the power, 2016-08-14 23:00:23]

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-14 23:03:16]

Description was changed from

==========
Do not sync the ARC++ opt-in preference.

This is a re-implementation of 1c7841e8e0322b8c3599710e017ff0fb38ab056f,
which made ARC opt-in preference not synced.  That change also stopped
sending UMA stats due to not calling the OnSyncedPrefChanged method.

BUG=633017
TEST=browser_tests --gtest_filter=ArcAuthServiceTest.*
Additionally manually tested that the normal opt-in/opt-out process
worked as expected.
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2206883002
Cr-Commit-Position: refs/heads/master@{#410994}
(cherry picked from commit 176f6733ab5bdf6de91f08e840b814cf76b910bb)
==========

to

==========
Do not sync the ARC++ opt-in preference.

This is a re-implementation of 1c7841e8e0322b8c3599710e017ff0fb38ab056f,
which made ARC opt-in preference not synced.  That change also stopped
sending UMA stats due to not calling the OnSyncedPrefChanged method.

BUG=633017
TEST=browser_tests --gtest_filter=ArcAuthServiceTest.*
Additionally manually tested that the normal opt-in/opt-out process
worked as expected.
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2206883002
Cr-Commit-Position: refs/heads/master@{#410994}
(cherry picked from commit 176f6733ab5bdf6de91f08e840b814cf76b910bb)
==========

[commit-bot: I haz the power, 2016-08-14 23:03:16]

Committed patchset #1 (id:1)