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

Unified Diff: chrome/browser/about_flags.cc

Issue 2946693002: Use GN buildflags and features to configure account consistency (Closed)
Patch Set: Created 3 years, 6 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: chrome/browser/about_flags.cc
diff --git a/chrome/browser/about_flags.cc b/chrome/browser/about_flags.cc
index d302cd553312e8b2ce83ae6432b77e1f0829b087..98e1e2233bb3d4b876952d24af004794388efc4b 100644
--- a/chrome/browser/about_flags.cc
+++ b/chrome/browser/about_flags.cc
@@ -66,6 +66,7 @@
#include "components/proximity_auth/switches.h"
#include "components/security_state/core/security_state.h"
#include "components/security_state/core/switches.h"
+#include "components/signin/core/common/signin_features.h"
#include "components/signin/core/common/signin_switches.h"
#include "components/spellcheck/common/spellcheck_features.h"
#include "components/spellcheck/common/spellcheck_switches.h"
@@ -305,15 +306,17 @@ const FeatureEntry::Choice kDefaultTileHeightChoices[] = {
{flag_descriptions::kDefaultTileHeightVenti, switches::kDefaultTileHeight,
"1024"}};
-#if !defined(OS_ANDROID)
+#if !BUILDFLAG(ENABLE_MIRROR)
const FeatureEntry::Choice kAccountConsistencyChoices[] = {
{flags_ui::kGenericExperimentChoiceDefault, "", ""},
{flag_descriptions::kAccountConsistencyChoiceMirror,
switches::kAccountConsistency, switches ::kAccountConsistencyMirror},
+#if BUILDFLAG(ENABLE_DICE_SUPPORT)
{flag_descriptions::kAccountConsistencyChoiceDice,
switches::kAccountConsistency, switches::kAccountConsistencyDice},
+#endif // BUILDFLAG(ENABLE_DICE_SUPPORT)
};
-#endif
+#endif // !BUILDFLAG(ENABLE_MIRROR)
const FeatureEntry::Choice kSimpleCacheBackendChoices[] = {
{flags_ui::kGenericExperimentChoiceDefault, "", ""},
@@ -1679,10 +1682,10 @@ const FeatureEntry kFeatureEntries[] = {
flag_descriptions::kWebglDraftExtensionsName,
flag_descriptions::kWebglDraftExtensionsDescription, kOsAll,
SINGLE_VALUE_TYPE(switches::kEnableWebGLDraftExtensions)},
-#if !defined(OS_ANDROID)
+#if !BUILDFLAG(ENABLE_MIRROR)
{"account-consistency", flag_descriptions::kAccountConsistencyName,
- flag_descriptions::kAccountConsistencyDescription,
- kOsWin | kOsLinux | kOsMac, MULTI_VALUE_TYPE(kAccountConsistencyChoices)},
+ flag_descriptions::kAccountConsistencyDescription, kOsAll,
+ MULTI_VALUE_TYPE(kAccountConsistencyChoices)},
#endif
#if BUILDFLAG(ENABLE_APP_LIST)
{"reset-app-list-install-state",

Powered by Google App Engine
This is Rietveld 408576698