[MD Settings] Add feature to enable md-settings by default.

chrome/browser/profiles/profile_window.cc

 // 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/browser/profiles/profile_window.h"
 
 #include <stddef.h>
 
 #include <string>
 #include <vector>
@@ skipping 14 matching lines @@
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/signin/account_reconcilor_factory.h"
 #include "chrome/browser/signin/account_tracker_service_factory.h"
 #include "chrome/browser/signin/signin_manager_factory.h"
 #include "chrome/browser/signin/signin_ui_util.h"
 #include "chrome/browser/sync/profile_sync_service_factory.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_dialogs.h"
 #include "chrome/browser/ui/profile_chooser_constants.h"
 #include "chrome/browser/ui/user_manager.h"
+#include "chrome/common/chrome_features.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/url_constants.h"
 #include "components/browser_sync/browser/profile_sync_service.h"
 #include "components/flags_ui/pref_service_flags_storage.h"
 #include "components/prefs/pref_service.h"
 #include "components/signin/core/browser/account_reconcilor.h"
 #include "components/signin/core/browser/account_tracker_service.h"
 #include "components/signin/core/browser/signin_manager.h"
 #include "components/signin/core/common/profile_management_switches.h"
 #include "components/signin/core/common/signin_pref_names.h"
@@ skipping 80 matching lines @@
 void OnUserManagerSystemProfileCreated(
     const base::FilePath& profile_path_to_focus,
     profiles::UserManagerTutorialMode tutorial_mode,
     profiles::UserManagerProfileSelected profile_open_action,
     const base::Callback<void(Profile*, const std::string&)>& callback,
     Profile* system_profile,
     Profile::CreateStatus status) {
   if (status != Profile::CREATE_STATUS_INITIALIZED || callback.is_null())
     return;
 
+  // Force Material Design user manager on when Material Design settings are on.
+  bool useMaterialDesignUserManager =

[Dan Beam, 2016/06/07 00:25:08]

use_material_design_user_manager

this is not cocoa or js ;)

also, super optional nit: i think use_md_user_manager would probably be equally
explanatory and more succinct

[groby-ooo-7-16, 2016/06/09 00:18:24]

Done.

+      switches::IsMaterialDesignUserManager() ||
+      base::FeatureList::IsEnabled(features::kMaterialDesignSettingsFeature);
+
   // Tell the webui which user should be focused.
-  std::string page = switches::IsMaterialDesignUserManager() ?
-      chrome::kChromeUIMdUserManagerUrl : chrome::kChromeUIUserManagerURL;
+  std::string page = useMaterialDesignUserManager
+                         ? chrome::kChromeUIMdUserManagerUrl
+                         : chrome::kChromeUIUserManagerURL;
 
   if (tutorial_mode == profiles::USER_MANAGER_TUTORIAL_OVERVIEW) {
     page += profiles::kUserManagerDisplayTutorial;
   } else if (!profile_path_to_focus.empty()) {
     // The file path is processed in the same way as base::CreateFilePathValue
     // (i.e. convert to std::string with AsUTF8Unsafe()), and then URI encoded.
     page += "#";
     page += net::EscapeUrlEncodedData(profile_path_to_focus.AsUTF8Unsafe(),
                                       false);
   } else if (profile_open_action ==
@@ skipping 343 matching lines @@
   PrefService* local_state = g_browser_process->local_state();
   const bool dismissed = local_state->GetBoolean(
       prefs::kProfileAvatarRightClickTutorialDismissed);
 
   // Don't show the tutorial if it's already been dismissed or if right-clicking
   // wouldn't show any targets.
   return !dismissed && HasProfileSwitchTargets(profile);
 }
 
 }  // namespace profiles