Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1552)

Unified Diff: components/signin/core/browser/about_signin_internals.cc

Issue 431083003: Change new-avatar-menu to an enable-disable flag. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 54b86bb01b4346f0466155bf6eac40b2ad93feb3..d3056a64ee7b3ea36fb9a21c18078ec28732c368 100644
--- a/components/signin/core/browser/about_signin_internals.cc
+++ b/components/signin/core/browser/about_signin_internals.cc
@@ -369,10 +369,11 @@ scoped_ptr<base::DictionaryValue> AboutSigninInternals::SigninStatus::ToValue(
switches::IsNewProfileManagement() == true ? "True" : "False");
AddSectionEntry(basic_info, "New Avatar Menu Enabled?",
switches::IsNewAvatarMenu() == true ? "True" : "False");
- bool new_avatar_menu_flag =
- CommandLine::ForCurrentProcess()->HasSwitch(switches::kNewAvatarMenu);
- AddSectionEntry(basic_info, "New Avatar Menu Flag Set?",
- new_avatar_menu_flag ? "True" : "False");
+ bool old_avatar_menu_flag =
+ CommandLine::ForCurrentProcess()->HasSwitch(
+ switches::kDisableNewAvatarMenu);
+ AddSectionEntry(basic_info, "Disable Avatar Menu?",
+ old_avatar_menu_flag ? "Disabled" : "Flag Not Set");
AddSectionEntry(basic_info, "Account Consistency Enabled?",
switches::IsEnableAccountConsistency() == true ? "True" : "False");

Powered by Google App Engine
This is Rietveld 408576698