mac: Fix bug where exiting tab fullscreen would cause Chrome menubar to disappear.

chrome/browser/ui/cocoa/browser_window_controller_private.mm

Index: chrome/browser/ui/cocoa/browser_window_controller_private.mm
diff --git a/chrome/browser/ui/cocoa/browser_window_controller_private.mm b/chrome/browser/ui/cocoa/browser_window_controller_private.mm
index 3874f2005a146a1393b43bd605710d18c9cbd265..15d46274449b4c845eed3b867196129bcb3d1b49 100644
--- a/chrome/browser/ui/cocoa/browser_window_controller_private.mm
+++ b/chrome/browser/ui/cocoa/browser_window_controller_private.mm
@@ -490,13 +490,10 @@ willPositionSheet:(NSWindow*)sheet
 }
 
 - (void)adjustUIForSlidingFullscreenStyle:(fullscreen_mac::SlidingStyle)style {
-  if (!presentationModeController_) {
-    presentationModeController_.reset(
-        [self newPresentationModeControllerWithStyle:style]);
-    [self configurePresentationModeController];
-  } else {
-    presentationModeController_.get().slidingStyle = style;
-  }
+  [presentationModeController_ exitPresentationMode];

[Robert Sesek, 2014/10/15 20:28:14]

I don't follow this...

[erikchen, 2014/10/15 21:50:35]

presentationModeController_.get().slidingStyle = style; isn't sufficient to
reset all state of the presentationModeController. That being said, it doesn't
affect the current CL, or the current functionality, so I'm dropping it from
this CL

+  presentationModeController_.reset(
+      [self newPresentationModeControllerWithStyle:style]);
+  [self configurePresentationModeController];
 
   if (!floatingBarBackingView_.get() &&
       ([self hasTabStrip] || [self hasToolbar] || [self hasLocationBar])) {