Implemented fullscreen exit animation with AppKit

chrome/browser/ui/cocoa/framed_browser_window.mm

 // Copyright (c) 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.
 
 #import "chrome/browser/ui/cocoa/framed_browser_window.h"
 
 #include "base/logging.h"
 #include "base/mac/sdk_forward_declarations.h"
 #include "chrome/browser/global_keyboard_shortcuts_mac.h"
 #include "chrome/browser/profiles/profile_avatar_icon_util.h"
@@ skipping 84 matching lines @@
                  object:miniaturizeButton_];
     [center addObserver:self
                selector:@selector(adjustZoomButton:)
                    name:NSViewFrameDidChangeNotification
                  object:zoomButton_];
   }
 
   return self;
 }
 
+// Overriding this method to prevent the OSX from automatically setting

[erikchen, 2015/08/10 18:06:26]

grammar.

This method is overridden to prevent AppKit from changing the style mask when
frameAndStyleLock_ is YES.

Declare your overrides in the header, and move the comments there.

[spqchan1, 2015/08/11 21:43:48]

Done.

+// the style mask
+- (void)setStyleMask:(NSUInteger)styleMask {

[erikchen, 2015/08/10 18:06:26]

reformat this method to meet Chrome style guidelines.

[spqchan1, 2015/08/11 21:43:48]

ditto to below.

+    if (frameAndStyleLock_) {
+        return;

[erikchen, 2015/08/10 18:06:25]

This is, in general, a very dangerous operation. I'd feel a lot better if we
could add some safety checks.

e.g., if we expect to override one AppKit invocation, then upload UMA metrics if
it happens more than once.

ditto for below.

[spqchan1, 2015/08/11 21:43:48]

Done.

+    }
+    [super setStyleMask:styleMask];
+}
+
+// Overriding this method to prevent the OSX from automatically setting

[erikchen, 2015/08/10 18:06:26]

ditto

[spqchan1, 2015/08/11 21:43:48]

Done.

+// the frame
+- (void)setFrame:(NSRect)frameRect
+         display:(BOOL)flag
+         animate:(BOOL)animateFlag {

[erikchen, 2015/08/10 18:06:26]

reformat this method to meet Chrome style guidelines.

[spqchan1, 2015/08/11 21:43:48]

I moved the overriden methods to the top. I'm confused about this though, are
the parameters not supposed to match up?

[erikchen, 2015/08/11 22:42:54]

The rest of the method is formatted incorrectly. The first 3 lines are fine.

[spqchan1, 2015/08/12 19:34:41]

Done. Removed the curly brace

+    if (frameAndStyleLock_) {
+        return;
+    }
+    [super setFrame:frameRect display:flag animate:animateFlag];
+}
+
 - (void)dealloc {
   [[NSNotificationCenter defaultCenter] removeObserver:self];
   [super dealloc];
 }
 
 - (void)adjustCloseButton:(NSNotification*)notification {
   [self adjustButton:[notification object]
               ofKind:NSWindowCloseButton];
 }
 
@@ skipping 55 matching lines @@
   if (!value) {
     value = [super accessibilityHitTest:point];
   }
   return value;
 }
 
 - (void)setShouldHideTitle:(BOOL)flag {
   shouldHideTitle_ = flag;
 }
 
+// This method sets a lock which prevents the the frame and style
+// of the window to be changed
+- (void) setFrameAndStyleLock:(BOOL)lock {

[erikchen, 2015/08/10 18:06:26]

run "git cl format" to clang-format your entire CL. clang-format has as couple
of bugs, but it will catch all of these small nits.

[spqchan1, 2015/08/11 21:43:48]

Done. This is magical

+  frameAndStyleLock_ = lock;
+}
+
 - (BOOL)_isTitleHidden {
   return shouldHideTitle_;
 }
 
 - (CGFloat)windowButtonsInterButtonSpacing {
   return windowButtonsInterButtonSpacing_;
 }
 
 // This method is called whenever a window is moved in order to ensure it fits
 // on the screen.  We cannot always handle resizes without breaking, so we
@@ skipping 148 matching lines @@
   ThemedWindowStyle windowStyle = [self themedWindowStyle];
   BOOL incognito = windowStyle & THEMED_INCOGNITO;
 
   if (incognito)
     return [NSColor whiteColor];
   else
     return [NSColor windowFrameTextColor];
 }
 
 @end