Enable immersive fullscreen for hosted apps part 2

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

Index: chrome/browser/ui/views/frame/browser_frame_ash.cc
diff --git a/chrome/browser/ui/views/frame/browser_frame_ash.cc b/chrome/browser/ui/views/frame/browser_frame_ash.cc
index 531011e7b34f7fd1fd2fe034aa3d2f4977cfc1d7..6bc4b3664cf2967a62026542c9b7cc31ae661e2f 100644
--- a/chrome/browser/ui/views/frame/browser_frame_ash.cc
+++ b/chrome/browser/ui/views/frame/browser_frame_ash.cc
@@ -63,7 +63,17 @@ BrowserFrameAsh::BrowserFrameAsh(BrowserFrame* browser_frame,
   window_state->SetDelegate(
       scoped_ptr<ash::wm::WindowStateDelegate>(
           new BrowserWindowStateDelegate(browser)).Pass());
-  window_state->set_animate_to_fullscreen(!browser->is_type_tabbed());
+
+#if defined(OS_CHROMEOS)

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

nit: Can this move into the OS_CHROMEOS block below?

+  if (browser->is_type_tabbed()) {
+    // Animating to immersive fullscreen does not look good. Immersive
+    // fullscreen is the default fullscreen type on ChromeOS for tabbed browser
+    // windows. The WindowState constructor disables animating to fullscreen
+    // completely when the kAshEnableImmersiveFullscreenForAllWindows command
+    // line flag is set.
+    window_state->set_animate_to_fullscreen(false);
+  }
+#endif
 
   // Turn on auto window management if we don't need an explicit bounds.
   // This way the requested bounds are honored.