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

Unified Diff: chrome/browser/chromeos/policy/device_local_account_browsertest.cc

Issue 22986010: Add policy for fullscreen mode; disallow fullscreen in public sessions (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 4 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/chromeos/policy/device_local_account_browsertest.cc
diff --git a/chrome/browser/chromeos/policy/device_local_account_browsertest.cc b/chrome/browser/chromeos/policy/device_local_account_browsertest.cc
index 77951ff6718ecda0212cc500b07458f949ac32f4..7837d27f7ca7f5ff0ea6d057e1a1d71043758bc4 100644
--- a/chrome/browser/chromeos/policy/device_local_account_browsertest.cc
+++ b/chrome/browser/chromeos/policy/device_local_account_browsertest.cc
@@ -33,9 +33,13 @@
#include "chrome/browser/policy/proto/chromeos/chrome_device_policy.pb.h"
#include "chrome/browser/policy/test/local_policy_test_server.h"
#include "chrome/browser/prefs/session_startup_pref.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/profiles/profile_manager.h"
#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/browser_commands.h"
#include "chrome/browser/ui/browser_finder.h"
#include "chrome/browser/ui/browser_list.h"
+#include "chrome/browser/ui/browser_window.h"
#include "chrome/browser/ui/host_desktop.h"
#include "chrome/browser/ui/tabs/tab_strip_model.h"
#include "chrome/common/chrome_switches.h"
@@ -302,6 +306,7 @@ IN_PROC_BROWSER_TEST_F(DeviceLocalAccountTest, StartSession) {
chrome::NOTIFICATION_USER_LIST_CHANGED,
base::Bind(&DisplayNameMatches, user_id_1_, kDisplayName)).Wait();
+ // Start login into the device-local account.
chromeos::LoginDisplayHost* host =
chromeos::LoginDisplayHostImpl::default_host();
ASSERT_TRUE(host);
@@ -377,4 +382,51 @@ IN_PROC_BROWSER_TEST_F(DeviceLocalAccountTest, TermsOfService) {
wizard_controller->current_screen()->GetName());
}
+IN_PROC_BROWSER_TEST_F(DeviceLocalAccountTest, FullscreenDisabled) {
+ InstallDeviceLocalAccountPolicy();
+ AddPublicSessionToDevicePolicy(kAccountId1);
+
+ // This observes the display name becoming available as this indicates
+ // device-local account policy is fully loaded, which is a prerequisite for
+ // successful login.
+ content::WindowedNotificationObserver(
+ chrome::NOTIFICATION_USER_LIST_CHANGED,
+ base::Bind(&DisplayNameMatches, user_id_1_, kDisplayName)).Wait();
+
+ // Ensure that the browser stays alive, even though no windows are opened
+ // during session start.
+ chrome::StartKeepAlive();
+
+ // Start login into the device-local account.
+ chromeos::LoginDisplayHost* host =
+ chromeos::LoginDisplayHostImpl::default_host();
+ ASSERT_TRUE(host);
+ host->StartSignInScreen();
+ chromeos::ExistingUserController* controller =
+ chromeos::ExistingUserController::current_controller();
+ ASSERT_TRUE(controller);
+ controller->LoginAsPublicAccount(user_id_1_);
+
+ // Wait for the session to start.
+ content::WindowedNotificationObserver(chrome::NOTIFICATION_SESSION_STARTED,
+ base::Bind(IsSessionStarted)).Wait();
+
+ // Open a browser window.
+ chrome::NewEmptyWindow(ProfileManager::GetDefaultProfile(),
+ chrome::HOST_DESKTOP_TYPE_ASH);
+ BrowserList* browser_list =
+ BrowserList::GetInstance(chrome::HOST_DESKTOP_TYPE_ASH);
+ EXPECT_EQ(1U, browser_list->size());
+ Browser* browser = browser_list->get(0);
+ ASSERT_TRUE(browser);
+ BrowserWindow* browser_window = browser->window();
+ ASSERT_TRUE(browser_window);
+ chrome::EndKeepAlive();
+
+ // Verify that an attempt to enter fullscreen mode is denied.
+ EXPECT_FALSE(browser_window->IsFullscreen());
+ chrome::ToggleFullscreenMode(browser);
+ EXPECT_FALSE(browser_window->IsFullscreen());
+}
+
} // namespace policy

Powered by Google App Engine
This is Rietveld 408576698