[MacViews] Fix behavior of non-resizable windows in fullscreen.

ui/views/cocoa/bridged_native_widget.mm

Index: ui/views/cocoa/bridged_native_widget.mm
diff --git a/ui/views/cocoa/bridged_native_widget.mm b/ui/views/cocoa/bridged_native_widget.mm
index 24275787c04e43ac1ab87b032e13ff5f4665461c..fd51109f66063ef33c934b3eb37120675f8895ae 100644
--- a/ui/views/cocoa/bridged_native_widget.mm
+++ b/ui/views/cocoa/bridged_native_widget.mm
@@ -430,13 +430,11 @@ void BridgedNativeWidget::ToggleDesiredFullscreenState() {
     return;  // TODO(tapted): Implement this for Snow Leopard.
   }
 
-  // Since fullscreen requests are ignored if the collection behavior does not
-  // allow it, save the collection behavior and restore it after.
-  NSWindowCollectionBehavior behavior = [window_ collectionBehavior];
-  [window_ setCollectionBehavior:behavior |
-                                 NSWindowCollectionBehaviorFullScreenPrimary];
+  // Enable fullscreen collection behavior because:
+  // 1: -[NSWindow toggleFullscreen:] would otherwise be ignored,
+  // 2: the fullscreen button must be enabled so the user can leave fullscreen.
+  gfx::SetNSWindowCanFullscreen(window_, true);

[tapted, 2015/05/06 06:20:41]

Why always `true`? (this function is a toggle).

i.e. when coming out of fullscreen, what hides the button?

[jackhou1, 2015/05/13 04:27:53]

Changed to only set the collection behavior if not already fullscreen. It also
removes size constraints now.

   [window_ toggleFullScreen:nil];
-  [window_ setCollectionBehavior:behavior];
 }
 
 void BridgedNativeWidget::OnSizeChanged() {
@@ -528,8 +526,10 @@ void BridgedNativeWidget::OnSizeConstraintsChanged() {
   bool is_resizable = widget->widget_delegate()->CanResize();
   bool shows_resize_controls =
       is_resizable && (min_size.IsEmpty() || min_size != max_size);
-  bool shows_fullscreen_controls =
-      is_resizable && widget->widget_delegate()->CanMaximize();
+  // Fullscreen controls should be enabled when in fullscreen so the user can
+  // leave.
+  bool shows_fullscreen_controls = target_fullscreen_state_ ||

[tapted, 2015/05/06 06:20:41]

What will call OnSizeConstraintsChanged() when target_fullscreen_state_ changes?

Maybe call OnSizeConstraintsChanged() unconditionally from
OnFullscreenTransitionComplete(), and update the comment in
ToggleDesiredFullscreenState to mention that the flag will be cleared once the
animation completes

[jackhou1, 2015/05/13 04:27:53]

There's a RootView::Layout after leaving fullscreen:

2   Chromium Framework                  0x0000000113502ab0
views::BridgedNativeWidget::OnSizeConstraintsChanged() + 208
3   Chromium Framework                  0x000000011360a9a2
views::NativeWidgetMac::OnSizeConstraintsChanged() + 50
4   Chromium Framework                  0x000000011360a902
views::NativeWidgetMac::OnRootViewLayout() + 50
5   Chromium Framework                  0x00000001136194d6
views::Widget::OnRootViewLayout() + 54
6   Chromium Framework                  0x000000011360ee7a
views::internal::RootView::Layout() + 58
7   Chromium Framework                  0x00000001135eb718
views::View::BoundsChanged(gfx::Rect const&) + 776
8   Chromium Framework                  0x00000001135eb325
views::View::SetBoundsRect(gfx::Rect const&) + 261
9   Chromium Framework                  0x00000001135eb1f0
views::View::SetBounds(int, int, int, int) + 144
10  Chromium Framework                  0x00000001135eb897
views::View::SetSize(gfx::Size const&) + 103
11  Chromium Framework                  0x00000001134f9d85 -[BridgedContentView
setFrameSize:] + 501
12  AppKit                              0x00007fff876ec090 -[NSWindow
_oldPlaceWindow:] + 1627
13  AppKit                              0x00007fff876eb3aa -[NSWindow
_setFrameCommon:display:stashSize:] + 2527
14  AppKit                              0x00007fff87e7df1c
-[_NSWindowFullScreenTransition setTransitionedWindowFrame:] + 84
15  AppKit                              0x00007fff878a2152
-[_NSWindowFullScreenTransition exitFullScreenTransitionForWindow:animated:] +
799
16  AppKit                              0x00007fff878a1d53 -[NSWindow
_resignFullScreenWindowAndExitFullScreen:] + 103
Done.

+      (is_resizable && widget->widget_delegate()->CanMaximize());
 
   gfx::ApplyNSWindowSizeConstraints(window, min_size, max_size,
                                     shows_resize_controls,