Fixed chrome://settings/accounts availability for MP.

chrome/browser/ui/webui/options/chromeos/accounts_options_browsertest.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 "base/basictypes.h"
+#include "base/compiler_specific.h"
+#include "base/prefs/pref_service.h"
+#include "chrome/browser/chromeos/login/login_manager_test.h"
+#include "chrome/browser/chromeos/login/startup_utils.h"
+#include "chrome/browser/chromeos/login/user_adding_screen.h"
+#include "chrome/browser/chromeos/login/user_manager.h"
+#include "chrome/browser/chromeos/settings/cros_settings.h"
+#include "chrome/browser/chromeos/settings/stub_cros_settings_provider.h"
+#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/tabs/tab_strip_model.h"
+#include "chrome/common/chrome_switches.h"
+#include "chrome/common/pref_names.h"
+#include "chrome/test/base/ui_test_utils.h"
+#include "chromeos/chromeos_switches.h"
+#include "chromeos/settings/cros_settings_names.h"
+#include "content/public/browser/web_contents.h"
+#include "content/public/test/browser_test_utils.h"
+#include "content/public/test/test_utils.h"
+
+namespace chromeos {
+
+namespace {
+
+const char* kTestUsers[] = { "test-user1@gmail.com", "test-user2@gmail.com" };
+
+}  // namespace
+
+class AccountsOptionsTest : public LoginManagerTest {
+ public:
+  AccountsOptionsTest()
+    : LoginManagerTest(false),
+      device_settings_provider_(NULL) {
+    stub_settings_provider_.Set(kDeviceOwner, base::StringValue(kTestUsers[0]));
+  }
+
+  virtual ~AccountsOptionsTest() {
+  }
+
+  virtual void SetUpOnMainThread() OVERRIDE {
+    LoginManagerTest::SetUpOnMainThread();
+    CrosSettings* settings = CrosSettings::Get();
+    device_settings_provider_ = settings->GetProvider(kDeviceOwner);
+    settings->RemoveSettingsProvider(device_settings_provider_);
+    settings->AddSettingsProvider(&stub_settings_provider_);
+  }
+
+  virtual void CleanUpOnMainThread() OVERRIDE {
+    CrosSettings* settings = CrosSettings::Get();
+    settings->RemoveSettingsProvider(&stub_settings_provider_);
+    settings->AddSettingsProvider(device_settings_provider_);
+    LoginManagerTest::CleanUpOnMainThread();
+  }
+
+  virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
+    LoginManagerTest::SetUpCommandLine(command_line);
+    command_line->AppendSwitch(::switches::kMultiProfiles);
+    command_line->AppendSwitch(switches::kForceMultiProfileInTests);
+  }
+
+ protected:
+  void CheckAccountsUI(const User* user, bool is_owner) {
+    Profile* profile = UserManager::Get()->GetProfileByUser(user);
+    profile->GetPrefs()->SetString(prefs::kGoogleServicesUsername,
+                                   user->email());
+
+    ui_test_utils::BrowserAddedObserver observer;
+    Browser* browser = CreateBrowser(profile);
+    observer.WaitForSingleNewBrowser();
+
+    ui_test_utils::NavigateToURL(browser,

[Nikita (slow), 2014/01/28 12:24:30]

nit: Does this method guarantee that Settings WebUI is fully initialized?

[ygorshenin1, 2014/01/28 13:32:54]

This method blocks until navigation is finished.

On 2014/01/28 12:24:30, Nikita Kostylev wrote:

[ygorshenin1, 2014/01/28 13:47:58]

+ also it follows the way how
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ui/...
is implemented.

On 2014/01/28 13:32:54, ygorshenin1 wrote:

+                                 GURL("chrome://settings-frame/accounts"));
+    content::WebContents* contents =
+        browser->tab_strip_model()->GetActiveWebContents();
+
+    bool warning_visible;
+    ASSERT_TRUE(content::ExecuteScriptAndExtractBool(
+        contents,
+        "var e = document.getElementById('ownerOnlyWarning');"
+        "var visible = e.offsetWidth > 0 && e.offsetHeight > 0;"
+        "window.domAutomationController.send(visible);",
+        &warning_visible));
+    EXPECT_EQ(is_owner, !warning_visible);
+
+    bool guest_option_enabled;
+    ASSERT_TRUE(content::ExecuteScriptAndExtractBool(
+        contents,
+        "var e = document.getElementById('allowBwsiCheck');"
+        "window.domAutomationController.send(!e.disabled);",
+        &guest_option_enabled));
+    ASSERT_EQ(is_owner, guest_option_enabled);
+
+    bool user_pods_enabled;
+    ASSERT_TRUE(content::ExecuteScriptAndExtractBool(
+        contents,
+        "var e = document.getElementById('showUserNamesCheck');"
+        "window.domAutomationController.send(!e.disabled);",
+        &user_pods_enabled));
+    ASSERT_EQ(is_owner, user_pods_enabled);
+
+    bool whitelist_enabled;
+    ASSERT_TRUE(content::ExecuteScriptAndExtractBool(
+        contents,
+        "var e = document.getElementById('useWhitelistCheck');"
+        "window.domAutomationController.send(!e.disabled);",
+        &whitelist_enabled));
+    ASSERT_EQ(is_owner, whitelist_enabled);
+  }
+
+  StubCrosSettingsProvider stub_settings_provider_;
+  CrosSettingsProvider* device_settings_provider_;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(AccountsOptionsTest);
+};
+
+IN_PROC_BROWSER_TEST_F(AccountsOptionsTest, PRE_MultiProfilesAccountsOptions) {
+  RegisterUser(kTestUsers[0]);
+  RegisterUser(kTestUsers[1]);
+  StartupUtils::MarkOobeCompleted();
+}
+
+IN_PROC_BROWSER_TEST_F(AccountsOptionsTest, MultiProfilesAccountsOptions) {
+  LoginUser(kTestUsers[0]);
+  UserAddingScreen::Get()->Start();
+  content::RunAllPendingInMessageLoop();
+  AddUser(kTestUsers[1]);
+
+  UserManager* manager = UserManager::Get();
+  ASSERT_EQ(2u, manager->GetLoggedInUsers().size());
+
+  CheckAccountsUI(manager->FindUser(kTestUsers[0]), true /* is_owner */);
+  CheckAccountsUI(manager->FindUser(kTestUsers[1]), false /* is_owner */);
+}
+
+}  // namespace chromeos