Hide the tab indicators and the shelf when in immersive + tab fullscreen

chrome/browser/ui/views/frame/immersive_mode_controller_ash_browsertest.cc

 // Copyright 2012 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 "chrome/browser/ui/views/frame/immersive_mode_controller_ash.h"
 
 #include "ash/ash_switches.h"
 #include "ash/root_window_controller.h"
 #include "ash/shelf/shelf_layout_manager.h"
 #include "ash/shelf/shelf_types.h"
 #include "ash/shell.h"
 #include "ash/wm/window_properties.h"
 #include "ash/wm/window_util.h"
 #include "base/command_line.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/bookmarks/bookmark_model.h"
 #include "chrome/browser/bookmarks/bookmark_model_factory.h"
 #include "chrome/browser/ui/app_modal_dialogs/app_modal_dialog_queue.h"
 #include "chrome/browser/ui/app_modal_dialogs/javascript_dialog_manager.h"
 #include "chrome/browser/ui/browser_commands.h"
+#include "chrome/browser/ui/fullscreen/fullscreen_controller.h"
+#include "chrome/browser/ui/fullscreen/fullscreen_controller_test.h"
 #include "chrome/browser/ui/views/bookmarks/bookmark_bar_view.h"
 #include "chrome/browser/ui/views/browser_dialogs.h"
 #include "chrome/browser/ui/views/frame/browser_view.h"
 #include "chrome/browser/ui/views/frame/top_container_view.h"
 #include "chrome/browser/ui/views/tabs/tab.h"
 #include "chrome/browser/ui/views/tabs/tab_strip.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "chrome/test/base/ui_test_utils.h"
 #include "content/public/browser/javascript_dialog_manager.h"
 #include "ui/compositor/layer_animator.h"
@@ skipping 125 matching lines @@
   ASSERT_FALSE(browser_view->IsFullscreen());
   EXPECT_FALSE(controller->IsEnabled());
   EXPECT_FALSE(controller->ShouldHideTopViews());
   EXPECT_FALSE(controller->IsRevealed());
   EXPECT_FALSE(browser_view->tabstrip()->IsImmersiveStyle());
   EXPECT_TRUE(browser_view->IsTabStripVisible());
   EXPECT_TRUE(browser_view->IsToolbarVisible());
 
   // When hiding the tab indicators, content is at the top of the browser view
   // both before and during reveal.
-  controller->SetHideTabIndicatorsForTest(true);
+  controller->SetForceHideTabIndicatorsForTest(true);
   chrome::ToggleFullscreenMode(browser());
   ASSERT_TRUE(browser_view->IsFullscreen());
   EXPECT_FALSE(browser_view->IsTabStripVisible());
   EXPECT_EQ(GetRectInWidget(browser_view).y(),
             GetRectInWidget(contents_view).y());
   controller->StartRevealForTest(true);
   EXPECT_TRUE(browser_view->IsTabStripVisible());
   // Shelf hide triggered by enabling immersive mode eventually changes the
   // widget bounds and causes a Layout(). Force it to happen early for test.
   browser_view->parent()->Layout();
   EXPECT_EQ(GetRectInWidget(browser_view).y(),
             GetRectInWidget(contents_view).y());
   chrome::ToggleFullscreenMode(browser());
   ASSERT_FALSE(browser_view->IsFullscreen());
-  controller->SetHideTabIndicatorsForTest(false);
+  controller->SetForceHideTabIndicatorsForTest(false);
 
   // Reveal ends when the mouse moves out of the reveal view.
   chrome::ToggleFullscreenMode(browser());
   ASSERT_TRUE(browser_view->IsFullscreen());
   EXPECT_TRUE(controller->IsEnabled());
   controller->StartRevealForTest(true);
   controller->SetMouseHoveredForTest(false);
   EXPECT_FALSE(controller->IsRevealed());
 
   // Window restore tracking is only implemented in the Aura port.
@@ skipping 406 matching lines @@
   chrome::ToggleFullscreenMode(browser());
   ASSERT_TRUE(browser_view->IsFullscreen());
   ASSERT_TRUE(immersive_controller->IsEnabled());
   EXPECT_EQ(ash::SHELF_AUTO_HIDE, shelf->visibility_state());
 
   // Disabling immersive mode maintains the user's auto-hide selection.
   chrome::ToggleFullscreenMode(browser());
   ASSERT_FALSE(browser_view->IsFullscreen());
   ASSERT_FALSE(immersive_controller->IsEnabled());
   EXPECT_EQ(ash::SHELF_AUTO_HIDE, shelf->visibility_state());
+}
 
-  // Setting the window property directly toggles immersive mode.
-  aura::Window* window = browser_view->GetWidget()->GetNativeWindow();
-  window->SetProperty(ash::internal::kImmersiveModeKey, true);
-  EXPECT_TRUE(immersive_controller->IsEnabled());
-  window->SetProperty(ash::internal::kImmersiveModeKey, false);
-  EXPECT_FALSE(immersive_controller->IsEnabled());
+// Test how being simultaneously in tab fullscreen and immersive fullscreen
+// affects the shelf visibility and whether the tab indicators are hidden.
+IN_PROC_BROWSER_TEST_F(ImmersiveModeControllerAshTest,
+                       TabAndBrowserFullscreen) {
+  ui::ScopedAnimationDurationScaleMode zero_duration_mode(
+      ui::ScopedAnimationDurationScaleMode::ZERO_DURATION);
+  ASSERT_TRUE(chrome::UseImmersiveFullscreen());
+
+  BrowserView* browser_view = static_cast<BrowserView*>(browser()->window());
+  ash::internal::ShelfLayoutManager* shelf =
+      ash::Shell::GetPrimaryRootWindowController()->GetShelfLayoutManager();
+  ImmersiveModeControllerAsh* controller =
+      static_cast<ImmersiveModeControllerAsh*>(
+          browser_view->immersive_mode_controller());
+
+  controller->SetForceHideTabIndicatorsForTest(false);
+
+  // The shelf should start out as visible.
+  ASSERT_EQ(ash::SHELF_VISIBLE, shelf->visibility_state());
+
+  // 1) Test that entering tab fullscreen from immersive mode hides the tab
+  // indicators and the shelf.
+  chrome::ToggleFullscreenMode(browser());
+  ASSERT_TRUE(controller->IsEnabled());
+  EXPECT_EQ(ash::SHELF_AUTO_HIDE, shelf->visibility_state());
+  EXPECT_FALSE(controller->ShouldHideTabIndicators());
+
+  // The shelf visibility and the tab indicator visibility are updated as a
+  // result of NOTIFICATION_FULLSCREEN_CHANGED which is asynchronous. Wait for
+  // the notification before testing visibility.
+  scoped_ptr<FullscreenNotificationObserver> waiter(
+      new FullscreenNotificationObserver());
+
+  browser()->fullscreen_controller()->ToggleFullscreenModeForTab(
+      browser_view->GetActiveWebContents(), true);
+  waiter->Wait();
+  ASSERT_TRUE(controller->IsEnabled());
+  EXPECT_EQ(ash::SHELF_HIDDEN, shelf->visibility_state());
+  EXPECT_TRUE(controller->ShouldHideTabIndicators());
+
+  // 2) Test that exiting tab fullscreen shows the tab indicators and autohides
+  // the shelf.
+  waiter.reset(new FullscreenNotificationObserver());
+  browser()->fullscreen_controller()->ToggleFullscreenModeForTab(
+      browser_view->GetActiveWebContents(), false);
+  waiter->Wait();
+  ASSERT_TRUE(controller->IsEnabled());
+  EXPECT_EQ(ash::SHELF_AUTO_HIDE, shelf->visibility_state());
+  EXPECT_FALSE(controller->ShouldHideTabIndicators());
+
+  // 3) Test that exiting tab fullscreen and immersive fullscreen
+  // simultaneously correctly updates the shelf visibility and whether the tab
+  // indicators should be hidden.
+  waiter.reset(new FullscreenNotificationObserver());
+  browser()->fullscreen_controller()->ToggleFullscreenModeForTab(
+      browser_view->GetActiveWebContents(), true);
+  waiter->Wait();
+  waiter.reset(new FullscreenNotificationObserver());
+  chrome::ToggleFullscreenMode(browser());
+  waiter->Wait();
+
+  ASSERT_FALSE(controller->IsEnabled());
+  EXPECT_EQ(ash::SHELF_VISIBLE, shelf->visibility_state());
+  EXPECT_TRUE(controller->ShouldHideTabIndicators());
 }
 
 #endif  // defined(OS_CHROMEOS)