[Mac] Use NSThemeFrame instead of NSGrayFrame on 10.8.

chrome/browser/ui/cocoa/browser_frame_view.mm

Index: chrome/browser/ui/cocoa/browser_frame_view.mm
diff --git a/chrome/browser/ui/cocoa/browser_frame_view.mm b/chrome/browser/ui/cocoa/browser_frame_view.mm
index 41d78385354df3ae072b4cda6980950d584a6c19..f6e8c89b76e84113d508f56b5f5a452ebcf2c7a4 100644
--- a/chrome/browser/ui/cocoa/browser_frame_view.mm
+++ b/chrome/browser/ui/cocoa/browser_frame_view.mm
@@ -8,6 +8,7 @@
 #import <Carbon/Carbon.h>
 
 #include "base/logging.h"
+#include "base/mac/mac_util.h"
 #include "base/mac/scoped_nsautorelease_pool.h"
 #import "chrome/browser/themes/theme_service.h"
 #import "chrome/browser/ui/cocoa/framed_browser_window.h"
@@ -46,7 +47,11 @@ static BOOL gCanGetCornerRadius = NO;
   // roll overs for our close widgets, but things should still function
   // correctly.
   base::mac::ScopedNSAutoreleasePool pool;
-  Class grayFrameClass = NSClassFromString(@"NSGrayFrame");
+
+  // On 10.8+ textured windows are no longer drawn by NSGrayFrame, and

[Avi (use Gerrit), 2012/03/14 02:17:50]

I didn't know that we were using textured windows. "...the window background is
no longer drawn..."?

[Robert Sesek, 2012/03/14 15:00:29]

Done.

Textured windows are the easiest way to not get the titlebar drawn.

+  // NSThemeFrame is used instead <http://crbug.com/114745>.
+  Class grayFrameClass = NSClassFromString(

[Avi (use Gerrit), 2012/03/14 02:17:50]

variable rename?

[Robert Sesek, 2012/03/14 15:00:29]

Done.

+      base::mac::IsOSMountainLionOrLater() ? @"NSThemeFrame" : @"NSGrayFrame");
   DCHECK(grayFrameClass);
   if (!grayFrameClass) return;