Display on signin-internals the status of account consistency.

Display on signin-internals the status of account consistency.

BUG=326577
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=278220

[Mike Lerman, 2014-06-18 17:44:11]

Hi Roger,

Adding the new flag to signin-internals. PTAL.

Thanks!

[Roger Tawa OOO till Jul 10th, 2014-06-18 18:22:44]

lgtm

Thanks Mike.

Thanks,
Roger

-


On Wed, Jun 18, 2014 at 1:44 PM, <mlerman@chromium.org> wrote:

> Reviewers: Roger Tawa,
>
> Message:
> Hi Roger,
>
> Adding the new flag to signin-internals. PTAL.
>
> Thanks!
>
> Description:
> Display on signin-internals the status of account consistency.
>
> BUG=326577
>
> Please review this at https://codereview.chromium.org/340773004/
>
> SVN Base: https://chromium.googlesource.com/chromium/src.git@master
>
> Affected files (+2, -0 lines):
>   M components/signin/core/browser/about_signin_internals.cc
>
>
> Index: components/signin/core/browser/about_signin_internals.cc
> diff --git a/components/signin/core/browser/about_signin_internals.cc
> b/components/signin/core/browser/about_signin_internals.cc
> index 86b94e76414e447456871a52e748b8eb456b3699..
> d05f57cd0f27e70f7a5b8f7a3f30a54e76087ff0 100644
> --- a/components/signin/core/browser/about_signin_internals.cc
> +++ b/components/signin/core/browser/about_signin_internals.cc
> @@ -362,6 +362,8 @@ scoped_ptr<base::DictionaryValue>
> AboutSigninInternals::SigninStatus::ToValue(
>        CommandLine::ForCurrentProcess()->HasSwitch(switches::
> kNewAvatarMenu);
>    AddSectionEntry(basic_info, "New Avatar Menu Flag Set?",
>        new_avatar_menu_flag ? "True" : "False");
> +  AddSectionEntry(basic_info, "Account Consistency Enabled?",
> +      switches::IsEnableAccountConsistency() == true ? "True" : "False");
>
>    // Only add username.  SID and LSID have moved to tokens section.
>    const std::string field =
>
>
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Roger Tawa OOO till Jul 10th, 2014-06-18 18:23:23]

lgtm

[Mike Lerman, 2014-06-18 18:33:20]

The CQ bit was checked by mlerman@chromium.org

[commit-bot: I haz the power, 2014-06-18 18:35:14]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/mlerman@chromium.org/340773004/1

[commit-bot: I haz the power, 2014-06-19 01:18:25]

Change committed as 278220