[Cocoa] Clean-up: Update comments regarding drawRect override in TabContentsContainerView.

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_cftyperef.h"
@@ skipping 70 matching lines @@
 - (NSColor*)computeBackgroundColor;
 @end
 
 @implementation TabContentsContainerView
 
 - (id)initWithDelegate:(TabContentsController*)delegate {
   if ((self = [super initWithFrame:NSZeroRect])) {
     delegate_ = delegate;
     if (!CommandLine::ForCurrentProcess()->HasSwitch(
             switches::kDisableCoreAnimation)) {
+      // TODO(ccameron): Remove the -drawRect: method once the
+      // kDisableCoreAnimation switch is removed.
       ScopedCAActionDisabler disabler;
       base::scoped_nsobject<CALayer> layer([[CALayer alloc] init]);
       [layer setBackgroundColor:CGColorGetConstantColor(kCGColorWhite)];
       [self setLayer:layer];
       [self setWantsLayer:YES];
     }
   }
   return self;
 }
 
@@ skipping 10 matching lines @@
     return [NSColor whiteColor];
 
   // 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];
+    bgColor = [NSColor whiteColor];
   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.
+//
+// Note: This method is never called when CoreAnimation is enabled.
 - (void)drawRect:(NSRect)dirtyRect {
   NSView* const contentsView =
       [[self subviews] count] > 0 ? [[self subviews] objectAtIndex:0] : nil;
   if (!contentsView || !NSContainsRect([contentsView frame], dirtyRect)) {
     [[self computeBackgroundColor] setFill];
     NSRectFill(dirtyRect);
   }
   [super drawRect:dirtyRect];
 }
 
@@ skipping 87 matching lines @@
   [contentsNativeView setFrame:[self frameForContentsView]];
   if ([subviews count] == 0) {
     [contentsContainer addSubview:contentsNativeView];
   } else if ([subviews objectAtIndex:0] != contentsNativeView) {
     [contentsContainer replaceSubview:[subviews objectAtIndex:0]
                                  with:contentsNativeView];
   }
   [contentsNativeView setAutoresizingMask:NSViewWidthSizable|
                                           NSViewHeightSizable];
 
-  // TODO(miu): The following can be removed once we use a CALayer in
-  // TabContentsContainerView.  http://crbug.com/354598
   [contentsContainer setNeedsDisplay:YES];
 
   // The rendering path with overlapping views disabled causes bugs when
   // transitioning between composited and non-composited mode.
   // http://crbug.com/279472
   if (!fullscreenView)
     contents_->SetAllowOverlappingViews(true);
 }
 
 - (void)changeWebContents:(WebContents*)newContents {
@@ skipping 108 matching lines @@
     } else {
       rect.ClampToCenteredSize(gfx::Size(
           static_cast<int>(x / captureSize.height()), rect.height()));
     }
   }
 
   return NSRectFromCGRect(rect.ToCGRect());
 }
 
 @end