Disable multi-profile UI in managed mode.

chrome/browser/ui/webui/options2/browser_options_handler2.cc

 // Copyright (c) 2012 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/ui/webui/options2/browser_options_handler2.h"
 
 #include "base/basictypes.h"
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
@@ skipping 634 matching lines @@
     entry->SetString("url", match.destination_url.spec());
     suggestions.Append(entry);
   }
 
   web_ui()->CallJavascriptFunction(
       "BrowserOptions.updateAutocompleteSuggestions", suggestions);
 }
 
 void BrowserOptionsHandler::SendProfilesInfo() {
   // Set profile creation text and button if multi-profiles switch is on.
-  scoped_ptr<Value> visible(Value::CreateBooleanValue(multiprofile_));
+  bool profiles_section_visible = multiprofile_ &&
+      !g_browser_process->local_state()->GetBoolean(prefs::kInManagedMode);
+
+  scoped_ptr<Value> visible(
+      Value::CreateBooleanValue(profiles_section_visible));
   web_ui()->CallJavascriptFunction("BrowserOptions.setProfilesSectionVisible",
                                    *visible);
 
-  if (!multiprofile_)
+  if (!profiles_section_visible)
     return;
 
   ProfileInfoCache& cache =
       g_browser_process->profile_manager()->GetProfileInfoCache();
   ListValue profile_info_list;
   FilePath current_profile_path =
       web_ui()->GetWebContents()->GetBrowserContext()->GetPath();
   for (size_t i = 0, e = cache.GetNumberOfProfiles(); i < e; ++i) {
     DictionaryValue* profile_value = new DictionaryValue();
     FilePath profile_path = cache.GetPathOfProfileAtIndex(i);
@@ skipping 16 matching lines @@
     }
 
     profile_info_list.Append(profile_value);
   }
 
   web_ui()->CallJavascriptFunction("BrowserOptions.setProfilesInfo",
                                    profile_info_list);
 }
 
 void BrowserOptionsHandler::CreateProfile(const ListValue* args) {
+  if (g_browser_process->local_state()->GetBoolean(prefs::kInManagedMode)) {
+    NOTREACHED();
+    return;

[James Hawkins, 2012/03/01 12:42:33]

Don't 'handle' NOTREACHEDs by returning.

[Bernhard Bauer, 2012/03/01 16:08:32]

Hm, what do you propose? Silently returning or CHECK? FTR, I see both
`NOTREACHED(); return;` as well as `CHECK` in this file.

[James Hawkins, 2012/03/01 19:51:36]

If it really should not fail, use NOTREACHED and don't return at all.

http://www.chromium.org/developers/coding-style#TOC-CHECK-vs-DCHECK-and-NOTRE...

[Bernhard Bauer, 2012/03/01 23:02:16]

It can happen, in the case where a malicious user gets the chrome://settings
page to call this method when it shouldn't be called, so I don't want to just
DCHECK here.

I guess I can silently return (and add a comment why)?

+  }
   ProfileManager::CreateMultiProfileAsync();
 }
 
 void BrowserOptionsHandler::ObserveThemeChanged() {
   Profile* profile = Profile::FromWebUI(web_ui());
 #if defined(TOOLKIT_GTK)
   GtkThemeService* theme_service = GtkThemeService::GetFrom(profile);
   bool is_gtk_theme = theme_service->UsingNativeTheme();
   base::FundamentalValue gtk_enabled(!is_gtk_theme);
   web_ui()->CallJavascriptFunction("BrowserOptions.setGtkThemeButtonEnabled",
@@ skipping 74 matching lines @@
   sync_status->SetBoolean("hasUnrecoverableError",
                           service->unrecoverable_error_detected());
   sync_status->SetBoolean("autoLoginVisible",
       CommandLine::ForCurrentProcess()->HasSwitch(switches::kEnableAutologin) &&
       service->AreCredentialsAvailable());
 
   return sync_status;
 }
 
 }  // namespace options2