CoreAnimation: Reduce VA usage by extra layers

chrome/browser/ui/cocoa/tab_contents/tab_contents_controller.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/tab_contents/tab_contents_controller.h"
 
 #include <utility>
 
+#include "base/command_line.h"
 #include "base/mac/scoped_nsobject.h"
 #include "chrome/browser/devtools/devtools_window.h"
 #import "chrome/browser/themes/theme_properties.h"
 #import "chrome/browser/themes/theme_service.h"
 #import "chrome/browser/ui/cocoa/themed_window.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/render_widget_host_view.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "content/public/browser/web_contents_view.h"
+#include "ui/base/cocoa/animation_utils.h"
+#include "ui/base/ui_base_switches.h"
 #include "ui/gfx/geometry/rect.h"
 
 using content::WebContents;
 using content::WebContentsObserver;
 
 // FullscreenObserver is used by TabContentsController to monitor for the
 // showing/destruction of fullscreen render widgets.  When notified,
 // TabContentsController will alter its child view hierarchy to either embed a
 // fullscreen render widget view or restore the normal WebContentsView render
 // view.  The embedded fullscreen render widget will fill the user's screen in
@@ skipping 37 matching lines @@
 - (NSRect)frameForContentsView;
 @end
 
 // An NSView with special-case handling for when the contents view does not
 // expand to fill the entire tab contents area. See 'AutoEmbedFullscreen mode'
 // in header file comments.
 @interface TabContentsContainerView : NSView {
  @private
   TabContentsController* delegate_;  // weak
 }
+
+- (NSColor*)computeBackgroundColor;
 @end
 
 @implementation TabContentsContainerView
 
 - (id)initWithDelegate:(TabContentsController*)delegate {
   if ((self = [super initWithFrame:NSZeroRect])) {
     delegate_ = delegate;
+    if (!CommandLine::ForCurrentProcess()->HasSwitch(
+            switches::kDisableCoreAnimation)) {
+      ScopedCAActionDisabler disabler;
+      base::scoped_nsobject<CALayer> layer([[CALayer alloc] init]);
+      [layer setBackgroundColor:CGColorGetConstantColor(kCGColorWhite)];
+      [self setLayer:layer];
+      [self setWantsLayer:YES];
+    }
   }
   return self;
 }
 
 // Called by the delegate during dealloc to invalidate the pointer held by this
 // view.
 - (void)delegateDestroyed {
   delegate_ = nil;
 }
 
+// Fill with a dark tint of the new tab page's background color.  This is
+// only seen when the subview is sized specially for fullscreen tab capture.
+- (NSColor*)computeBackgroundColor {
+  NSColor* bgColor = nil;
+  ThemeService* const theme =
+      static_cast<ThemeService*>([[self window] themeProvider]);
+  if (theme)
+    bgColor = theme->GetNSColor(ThemeProperties::COLOR_NTP_BACKGROUND);
+  if (!bgColor)
+    bgColor = [[self window] backgroundColor];
+  const float kDarknessFraction = 0.80f;
+  return [bgColor blendedColorWithFraction:kDarknessFraction
+                                   ofColor:[NSColor blackColor]];
+}
+
 // Override -drawRect to fill the view with a solid color outside of the
 // subview's frame.
 - (void)drawRect:(NSRect)dirtyRect {
   NSView* const contentsView =
       [[self subviews] count] > 0 ? [[self subviews] objectAtIndex:0] : nil;
   if (!contentsView || !NSContainsRect([contentsView frame], dirtyRect)) {
-    // Fill with a dark tint of the new tab page's background color.  This is
-    // only seen when the subview is sized specially for fullscreen tab capture.
-    NSColor* bgColor = nil;
-    ThemeService* const theme =
-        static_cast<ThemeService*>([[self window] themeProvider]);
-    if (theme)
-      bgColor = theme->GetNSColor(ThemeProperties::COLOR_NTP_BACKGROUND);
-    if (!bgColor)
-      bgColor = [[self window] backgroundColor];
-    const float kDarknessFraction = 0.80f;
-    [[bgColor blendedColorWithFraction:kDarknessFraction
-                               ofColor:[NSColor blackColor]] setFill];
+    [[self computeBackgroundColor] setFill];
     NSRectFill(dirtyRect);
   }
   [super drawRect:dirtyRect];
 }
 
 // Override auto-resizing logic to query the delegate for the exact frame to
 // use for the contents view.
 - (void)resizeSubviewsWithOldSize:(NSSize)oldBoundsSize {
   NSView* const contentsView =
       [[self subviews] count] > 0 ? [[self subviews] objectAtIndex:0] : nil;
   if (!contentsView || [contentsView autoresizingMask] == NSViewNotSizable ||
       !delegate_) {
     return;
   }
   [contentsView setFrame:[delegate_ frameForContentsView]];
 }
 
+// Update the background layer's color whenever the view needs to repaint.
+- (void)setNeedsDisplayInRect:(NSRect)rect {
+  [super setNeedsDisplayInRect:rect];
+

[ccameron, 2014/04/19 02:45:29]

[NSColor CGColor] is a 10.8+ function.

Not sure if a helper function is preferred or not (not for this change, cause it
needs to be merged, but I can add a TODO & do it after).

[Avi (use Gerrit), 2014/04/19 19:30:49]

That's fine. With 10.6, they changed things so that you could assign an NSImage
for a layer's image, so I'm a bit surprised that you can't assign an NSColor
here and have it work.

+  // Convert from an NSColor to a CGColorRef.
+  NSColor* nsBackgroundColor = [self computeBackgroundColor];
+  NSColorSpace* nsColorSpace = [nsBackgroundColor colorSpace];
+  CGColorSpaceRef cgColorSpace = [nsColorSpace CGColorSpace];
+  const NSInteger numberOfComponents = [nsBackgroundColor numberOfComponents];
+  CGFloat components[numberOfComponents];
+  [nsBackgroundColor getComponents:components];
+  CGColorRef cgBackgroundColor = CGColorCreate(cgColorSpace, components);

[Avi (use Gerrit), 2014/04/19 19:30:49]

ScopedCFObject if you must go this route.

[ccameron, 2014/04/19 22:21:39]

Done.

+
+  ScopedCAActionDisabler disabler;
+  [[self layer] setBackgroundColor:cgBackgroundColor];
+  CGColorRelease(cgBackgroundColor);
+}
+
 @end  // @implementation TabContentsContainerView
 
 @implementation TabContentsController
 @synthesize webContents = contents_;
 
 - (id)initWithContents:(WebContents*)contents
     andAutoEmbedFullscreen:(BOOL)enableEmbeddedFullscreen {
   if ((self = [super initWithNibName:nil bundle:nil])) {
     if (enableEmbeddedFullscreen)
       fullscreenObserver_.reset(new FullscreenObserver(self));
@@ skipping 168 matching lines @@
     } else {
       rect.ClampToCenteredSize(gfx::Size(
           static_cast<int>(x / captureSize.height()), rect.height()));
     }
   }
 
   return NSRectFromCGRect(rect.ToCGRect());
 }
 
 @end