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

Unified Diff: chrome/browser/ui/cocoa/exclusive_access_bubble_window_controller_unittest.mm

Issue 2001423002: Mac: Start removing Cocoa fullscreen code. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Respond to review. Created 4 years, 7 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/cocoa/exclusive_access_bubble_window_controller_unittest.mm
diff --git a/chrome/browser/ui/cocoa/exclusive_access_bubble_window_controller_unittest.mm b/chrome/browser/ui/cocoa/exclusive_access_bubble_window_controller_unittest.mm
index b3160e833e161ab13ec5e65d5303b73d02ca7ff1..ed6e6361297ae540a481c4afa8c8e103e36c770c 100644
--- a/chrome/browser/ui/cocoa/exclusive_access_bubble_window_controller_unittest.mm
+++ b/chrome/browser/ui/cocoa/exclusive_access_bubble_window_controller_unittest.mm
@@ -87,43 +87,6 @@ class ExclusiveAccessBubbleWindowControllerTest : public CocoaProfileTest {
base::scoped_nsobject<ExclusiveAccessBubbleWindowController> controller_;
};
-// http://crbug.com/103912
-TEST_F(ExclusiveAccessBubbleWindowControllerTest,
- DISABLED_DenyExitsFullscreen) {
- NSWindow* window = browser()->window()->GetNativeWindow();
- BrowserWindowController* bwc =
- [BrowserWindowController browserWindowControllerForWindow:window];
-
- [bwc showWindow:nil];
-
- AppendTabToStrip();
- WebContents* fullscreen_tab =
- browser()->tab_strip_model()->GetActiveWebContents();
- {
- base::mac::ScopedNSAutoreleasePool pool;
- content::WindowedNotificationObserver fullscreen_observer(
- chrome::NOTIFICATION_FULLSCREEN_CHANGED,
- content::NotificationService::AllSources());
- browser()->EnterFullscreenModeForTab(fullscreen_tab, GURL());
- fullscreen_observer.Wait();
- ASSERT_TRUE(browser()->window()->IsFullscreen());
- }
-
- ExclusiveAccessController* access_controller =
- [bwc exclusiveAccessController];
- EXPECT_TRUE(access_controller->cocoa_bubble());
- {
- content::WindowedNotificationObserver fullscreen_observer(
- chrome::NOTIFICATION_FULLSCREEN_CHANGED,
- content::NotificationService::AllSources());
- [access_controller->cocoa_bubble() deny:nil];
- fullscreen_observer.Wait();
- }
- EXPECT_FALSE(access_controller->cocoa_bubble());
- EXPECT_FALSE(browser()->window()->IsFullscreen());
- CloseBrowserWindow();
-}
-
TEST_F(ExclusiveAccessBubbleWindowControllerTest, LabelWasReplaced) {
EXPECT_FALSE([controller_ exitLabelPlaceholder]);
EXPECT_TRUE([controller_ exitLabel]);

Powered by Google App Engine
This is Rietveld 408576698