Fix for flash fullscreen animations on OSX

chrome/browser/ui/cocoa/browser_window_controller_private.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/browser_window_controller_private.h"
 
 #include <cmath>
 
 #import "base/auto_reset.h"
 #include "base/command_line.h"
@@ skipping 1121 matching lines @@
 
 - (BOOL)shouldUseCustomAppKitFullscreenTransition:(BOOL)enterFullScreen {
   // Custom fullscreen transitions should only be available in OSX 10.9+.
   if (base::mac::IsOSMountainLionOrEarlier())
     return NO;
 
   // Disable the custom exit animation in OSX 10.9: http://crbug.com/526327#c3.
   if (base::mac::IsOSMavericks() && !enterFullScreen)
     return NO;
 
-  // TODO(spqchan): Fix Flash fullscreen animation in popups.
-  // See http://crbug.com/566588.
-  if ([self isFullscreenForTabContent] && browser_->is_type_popup())
-    return NO;
-
   NSView* root = [[self.window contentView] superview];
   if (!root.layer)
     return NO;
 
   // AppKit on OSX 10.9 has a bug for applications linked against OSX 10.8 SDK
   // and earlier. Under specific circumstances, it prevents the custom AppKit
   // transition from working well. See http://crbug.com/396980 for more
   // details.
   if ([[self class] systemSettingsRequireMavericksAppKitFullscreenHack] &&
       ![[[self window] screen] isEqual:[[NSScreen screens] firstObject]]) {
@@ skipping 86 matching lines @@
   return nil;
 }
 
 - (BOOL)isFullscreenForTabContent {
   return browser_->exclusive_access_manager()
                  ->fullscreen_controller()
                  ->IsWindowFullscreenForTabOrPending();
 }
 
 @end  // @implementation BrowserWindowController(Private)