[Smart Lock, UI] Use correct string for the auto sign-in first run.

[Smart Lock, UI] Use correct string for the auto sign-in first run.

Method which check is settings are synced should check PRIORITY_PREFERENCE instead of PREFERENCE, because only they are synced on mobile devices.

BUG=590756
Committed: https://crrev.com/d2329b1a69b53d8f4384c60d0cd31cd03908cde3
Cr-Commit-Position: refs/heads/master@{#378238}

[melandory, 2016-02-29 16:23:55]

Description was changed from

==========
[Smart Lock, UI] Use correct string for the auto sign-in first run.

BUG=590756
==========

to

==========
[Smart Lock, UI] Use correct string for the auto sign-in first run.

Method which check is settings are synced should check PRIORITY_PREFERENCE
instead of PREFERENCE, because only they are synced on mobile devices.

BUG=590756
==========

[melandory, 2016-02-29 16:24:13]

melandory@chromium.org changed reviewers:
+ vasilii@chromium.org

[melandory, 2016-02-29 16:24:13]

PTAL, thanks!

[melandory, 2016-02-29 17:00:00]

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

[commit-bot: I haz the power, 2016-02-29 17:00:17]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1750603002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1750603002/1

[vasilii, 2016-02-29 17:20:48]

https://codereview.chromium.org/1750603002/diff/1/chrome/browser/ui/passwords...
File chrome/browser/ui/passwords/manage_passwords_view_utils.cc (right):

https://codereview.chromium.org/1750603002/diff/1/chrome/browser/ui/passwords...
chrome/browser/ui/passwords/manage_passwords_view_utils.cc:176: bool
IsSyncingSettings(Profile* profile) {
Make the method name more descriptive (like IsSyncingAutosignSetting) and add a
comment.

[melandory, 2016-02-29 17:37:52]

https://codereview.chromium.org/1750603002/diff/1/chrome/browser/ui/passwords...
File chrome/browser/ui/passwords/manage_passwords_view_utils.cc (right):

https://codereview.chromium.org/1750603002/diff/1/chrome/browser/ui/passwords...
chrome/browser/ui/passwords/manage_passwords_view_utils.cc:176: bool
IsSyncingSettings(Profile* profile) {
On 2016/02/29 17:20:48, vasilii wrote:
> Make the method name more descriptive (like IsSyncingAutosignSetting) and add
a
> comment.

Done.

[vasilii, 2016-02-29 17:42:38]

lgtm

[melandory, 2016-02-29 17:49:48]

The CQ bit was checked by melandory@chromium.org

[commit-bot: I haz the power, 2016-02-29 17:50:04]

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

[commit-bot: I haz the power, 2016-02-29 18:34:48]

Description was changed from

==========
[Smart Lock, UI] Use correct string for the auto sign-in first run.

Method which check is settings are synced should check PRIORITY_PREFERENCE
instead of PREFERENCE, because only they are synced on mobile devices.

BUG=590756
==========

to

==========
[Smart Lock, UI] Use correct string for the auto sign-in first run.

Method which check is settings are synced should check PRIORITY_PREFERENCE
instead of PREFERENCE, because only they are synced on mobile devices.

BUG=590756
==========

[commit-bot: I haz the power, 2016-02-29 18:34:49]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-02-29 18:36:50]

Description was changed from

==========
[Smart Lock, UI] Use correct string for the auto sign-in first run.

Method which check is settings are synced should check PRIORITY_PREFERENCE
instead of PREFERENCE, because only they are synced on mobile devices.

BUG=590756
==========

to

==========
[Smart Lock, UI] Use correct string for the auto sign-in first run.

Method which check is settings are synced should check PRIORITY_PREFERENCE
instead of PREFERENCE, because only they are synced on mobile devices.

BUG=590756

Committed: https://crrev.com/d2329b1a69b53d8f4384c60d0cd31cd03908cde3
Cr-Commit-Position: refs/heads/master@{#378238}
==========

[commit-bot: I haz the power, 2016-02-29 18:36:51]

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