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

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

Issue 12018007: Refactor BrowserWindow fullscreen and presentation on Mac to be consistent with other platforms. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge TOT Created 7 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/cocoa/browser_window_controller_unittest.mm
diff --git a/chrome/browser/ui/cocoa/browser_window_controller_unittest.mm b/chrome/browser/ui/cocoa/browser_window_controller_unittest.mm
index dad7acf3dd983119311fad9691f984b3c23e8e8a..f0ce63a5b76cbfc8cfb8b9fc8722504e773a7d5a 100644
--- a/chrome/browser/ui/cocoa/browser_window_controller_unittest.mm
+++ b/chrome/browser/ui/cocoa/browser_window_controller_unittest.mm
@@ -850,8 +850,7 @@ TEST_F(BrowserWindowFullScreenControllerTest, TestFullscreen) {
[controller_ showWindow:nil];
EXPECT_FALSE([controller_ isFullscreen]);
- [controller_ enterFullscreenForURL:GURL()
- bubbleType:FEB_TYPE_BROWSER_FULLSCREEN_EXIT_INSTRUCTION];
+ [controller_ enterFullscreen];
WaitForFullScreenTransition();
EXPECT_TRUE([controller_ isFullscreen]);
@@ -872,8 +871,7 @@ TEST_F(BrowserWindowFullScreenControllerTest, TestActivate) {
[controller_ activate];
EXPECT_TRUE(IsFrontWindow([controller_ window]));
- [controller_ enterFullscreenForURL:GURL()
- bubbleType:FEB_TYPE_BROWSER_FULLSCREEN_EXIT_INSTRUCTION];
+ [controller_ enterFullscreen];
WaitForFullScreenTransition();
[controller_ activate];
« no previous file with comments | « chrome/browser/ui/cocoa/browser_window_controller.mm ('k') | chrome/browser/ui/fullscreen/fullscreen_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698