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

Unified Diff: chrome/common/profile_management_switches.cc

Issue 235813002: Add profile-switches to signin-internals; move switches to component (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix gyp(i) files Created 6 years, 8 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/common/profile_management_switches.cc
diff --git a/chrome/common/profile_management_switches.cc b/chrome/common/profile_management_switches.cc
deleted file mode 100644
index e6af27935a32c15c3404c8206258b56181fda6a2..0000000000000000000000000000000000000000
--- a/chrome/common/profile_management_switches.cc
+++ /dev/null
@@ -1,61 +0,0 @@
-// Copyright 2013 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "chrome/common/profile_management_switches.h"
-
-#include "base/command_line.h"
-#include "base/metrics/field_trial.h"
-#include "chrome/common/chrome_switches.h"
-
-namespace {
-
-const char kNewProfileManagementFieldTrialName[] = "NewProfileManagement";
-
-bool CheckProfileManagementFlag(std::string command_switch, bool active_state) {
- // Individiual flag settings take precedence.
- if (CommandLine::ForCurrentProcess()->HasSwitch(command_switch)) {
- return true;
- }
-
- // --new-profile-management flag always affects all switches.
- if (CommandLine::ForCurrentProcess()->HasSwitch(
- switches::kNewProfileManagement)) {
- return active_state;
- }
-
- // NewProfileManagement experiment acts like above flag.
- std::string trial_type =
- base::FieldTrialList::FindFullName(kNewProfileManagementFieldTrialName);
- if (!trial_type.empty()) {
- if (trial_type == "Enabled")
- return active_state;
- if (trial_type == "Disabled")
- return !active_state;
- }
-
- return false;
-}
-
-} // namespace
-
-namespace switches {
-
-bool IsEnableWebBasedSignin() {
- return CheckProfileManagementFlag(switches::kEnableWebBasedSignin, false);
-}
-
-bool IsGoogleProfileInfo() {
- return CheckProfileManagementFlag(switches::kGoogleProfileInfo, true);
-}
-
-bool IsNewProfileManagement() {
- return CheckProfileManagementFlag(switches::kNewProfileManagement, true);
-}
-
-bool IsFastUserSwitching() {
- return CommandLine::ForCurrentProcess()->HasSwitch(
- switches::kFastUserSwitching);
-}
Roger Tawa OOO till Jul 10th 2014/04/11 20:57:43 Put in same order as .h file.
Mike Lerman 2014/04/14 19:39:53 Done.
-
-} // namespace switches

Powered by Google App Engine
This is Rietveld 408576698