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

Unified Diff: chrome/browser/ui/webui/options/chromeos/accounts_options_browsertest.cc

Issue 144983002: Fixed chrome://settings/accounts availability for MP. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed proxy_settings_ui. Created 6 years, 11 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/ui/webui/options/chromeos/accounts_options_browsertest.cc
diff --git a/chrome/browser/ui/webui/options/chromeos/accounts_options_browsertest.cc b/chrome/browser/ui/webui/options/chromeos/accounts_options_browsertest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..6eefda29c0bd3f055f7287429d5d5f2cf4eb32bd
--- /dev/null
+++ b/chrome/browser/ui/webui/options/chromeos/accounts_options_browsertest.cc
@@ -0,0 +1,140 @@
+// 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,
+ 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
« no previous file with comments | « chrome/browser/ui/webui/components_ui.cc ('k') | chrome/browser/ui/webui/options/chromeos/accounts_options_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698