Enable immersive fullscreen for hosted apps part 2

chrome/browser/ui/views/frame/immersive_mode_controller_ash.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/shell.h"
 #include "ash/wm/window_state.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/ui/fullscreen/fullscreen_controller.h"
@@ skipping 56 matching lines @@
   native_window_ = browser_view_->GetNativeWindow();
   controller_->Init(this, browser_view_->frame(),
       browser_view_->top_container());
 }
 
 void ImmersiveModeControllerAsh::SetEnabled(bool enabled) {
   if (controller_->IsEnabled() == enabled)
     return;
 
   EnableWindowObservers(enabled);
-
-  // Use a short "light bar" version of the tab strip when the top-of-window
-  // views are closed. If the user additionally enters into tab fullscreen,
-  // the tab indicators will be hidden.
-  use_tab_indicators_ = enabled;
-
+  UpdateTabIndicators();
   controller_->SetEnabled(enabled);
 }
 
 bool ImmersiveModeControllerAsh::IsEnabled() const {
   return controller_->IsEnabled();
 }
 
 bool ImmersiveModeControllerAsh::ShouldHideTabIndicators() const {
   return !use_tab_indicators_;
 }
@@ skipping 69 matching lines @@
     // |window| is NULL if the renderer crashed.
     if (window) {
       gfx::Insets inset(top_inset, 0, 0, 0);
       window->SetHitTestBoundsOverrideOuter(
           window->hit_test_bounds_override_outer_mouse(),
           inset);
     }
   }
 }
 
-void ImmersiveModeControllerAsh::SetTabIndicatorsVisible(bool visible) {
-  DCHECK(!visible || use_tab_indicators_);
-  if (browser_view_->tabstrip())
-    browser_view_->tabstrip()->SetImmersiveStyle(visible);
+bool ImmersiveModeControllerAsh::UpdateTabIndicators() {

[James Cook, 2013/11/25 19:09:20]

Nice, I like consolidating all this logic here.

+  bool has_tabstrip = browser_view_->IsBrowserTypeNormal();
+  if (!IsEnabled() || !has_tabstrip) {
+    use_tab_indicators_ = false;
+  } else {
+    // Use a "light bar" version of the tab strip when the top-of-window views
+    // are closed and not in tab fullscreen.
+    bool in_tab_fullscreen = browser_view_->browser()->fullscreen_controller()->
+        IsFullscreenForTabOrPending();
+    use_tab_indicators_ = !in_tab_fullscreen;
+  }
+
+  bool show_tab_indicators = use_tab_indicators_ && !IsRevealed();
+  if (show_tab_indicators != browser_view_->tabstrip()->IsImmersiveStyle()) {
+    browser_view_->tabstrip()->SetImmersiveStyle(show_tab_indicators);
+    return true;
+  }
+  return false;
 }
 
 void ImmersiveModeControllerAsh::OnImmersiveRevealStarted() {
   visible_fraction_ = 0;
   browser_view_->top_container()->SetPaintToLayer(true);
-  SetTabIndicatorsVisible(false);
+  UpdateTabIndicators();
   SetRenderWindowTopInsetsForTouch(0);
   LayoutBrowserRootView();
   FOR_EACH_OBSERVER(Observer, observers_, OnImmersiveRevealStarted());
 }
 
 void ImmersiveModeControllerAsh::OnImmersiveRevealEnded() {
   visible_fraction_ = 0;
   browser_view_->top_container()->SetPaintToLayer(false);
-  SetTabIndicatorsVisible(use_tab_indicators_);
+  UpdateTabIndicators();
   SetRenderWindowTopInsetsForTouch(
       kStealTouchEventsFromWebContentsRegionHeightPx);
   LayoutBrowserRootView();
 }
 
 void ImmersiveModeControllerAsh::OnImmersiveFullscreenExited() {
   browser_view_->top_container()->SetPaintToLayer(false);
-  SetTabIndicatorsVisible(false);
+  UpdateTabIndicators();
   SetRenderWindowTopInsetsForTouch(0);
   LayoutBrowserRootView();
 }
 
 void ImmersiveModeControllerAsh::SetVisibleFraction(double visible_fraction) {
   if (visible_fraction_ != visible_fraction) {
     visible_fraction_ = visible_fraction;
     browser_view_->Layout();
   }
 }
@@ skipping 31 matching lines @@
 }
 
 void ImmersiveModeControllerAsh::Observe(
     int type,
     const content::NotificationSource& source,
     const content::NotificationDetails& details) {
   DCHECK_EQ(chrome::NOTIFICATION_FULLSCREEN_CHANGED, type);
   if (!controller_->IsEnabled())
     return;
 
-  bool in_tab_fullscreen = content::Source<FullscreenController>(source)->
-      IsFullscreenForTabOrPending();
-
-  bool used_tab_indicators = use_tab_indicators_;
-  use_tab_indicators_ = !in_tab_fullscreen;
-  SetTabIndicatorsVisible(use_tab_indicators_ && !controller_->IsRevealed());
+  bool tab_indicator_visibility_changed = UpdateTabIndicators();
 
   // Auto hide the shelf in immersive browser fullscreen. When auto hidden, the
   // shelf displays a 3px 'light bar' when it is closed. When in immersive
   // browser fullscreen and tab fullscreen, hide the shelf completely and
   // prevent it from being revealed.
+  bool in_tab_fullscreen = content::Source<FullscreenController>(source)->
+      IsFullscreenForTabOrPending();
   ash::wm::GetWindowState(native_window_)->set_hide_shelf_when_fullscreen(
       in_tab_fullscreen);
   ash::Shell::GetInstance()->UpdateShelfVisibility();
 
-  if (use_tab_indicators_ != used_tab_indicators)
+  if (tab_indicator_visibility_changed)
     LayoutBrowserRootView();
 }