Add icon to the panel titlebar on Mac.

chrome/browser/ui/panels/panel_titlebar_view_cocoa.mm

 // Copyright (c) 2011 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/panels/panel_titlebar_view_cocoa.h"
 
 #include <Carbon/Carbon.h>  // kVK_Escape
 #import <Cocoa/Cocoa.h>
 
 #include "base/logging.h"
 #include "base/mac/scoped_nsautorelease_pool.h"
 #include "chrome/browser/themes/theme_service.h"
 #import "chrome/browser/ui/cocoa/hover_image_button.h"
 #import "chrome/browser/ui/cocoa/nsview_additions.h"
 #import "chrome/browser/ui/cocoa/themed_window.h"
 #import "chrome/browser/ui/cocoa/tracking_area.h"
 #import "chrome/browser/ui/panels/panel_window_controller_cocoa.h"
 #include "grit/theme_resources_standard.h"
 #import "third_party/GTM/AppKit/GTMNSBezierPath+RoundRect.h"
+#include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/mac/nsimage_cache.h"
 #include "ui/gfx/scoped_ns_graphics_context_save_gstate_mac.h"
 
 const int kRoundedCornerSize = 3;
-const int kCloseButtonLeftPadding = 8;
+const int kButtonPadding = 8;
+const int kIconAndTextPadding = 5;
 
 // Used to implement TestingAPI
 static NSEvent* MakeMouseEvent(NSEventType type, NSPoint point) {
   return [NSEvent mouseEventWithType:type
                             location:point
                        modifierFlags:0
                            timestamp:0
                         windowNumber:0
                              context:nil
                          eventNumber:0
@@ skipping 140 matching lines @@
   [settingsButton_ setDefaultOpacity:0.6];
   [settingsButton_ setHoverImage:image];
   [settingsButton_ setHoverOpacity:0.9];
   [settingsButton_ setPressedImage:image];
   [settingsButton_ setPressedOpacity:1.0];
   [[settingsButton_ cell] setHighlightsBy:NSNoCellMask];
   [self checkMouseAndUpdateSettingsButtonVisibility];
 
   // Update layout of controls in the titlebar.
   [self updateCloseButtonLayout];
+  [self updateIconAndTitleLayout];
 
   // Set autoresizing behavior: glued to edges on left, top and right.
   [self setAutoresizingMask:(NSViewMinYMargin | NSViewWidthSizable)];
 
   [[NSNotificationCenter defaultCenter]
       addObserver:self
          selector:@selector(didChangeTheme:)
              name:kBrowserThemeDidChangeNotification
            object:nil];
   // Register for various window focus changes, so we can update our custom
   // titlebar appropriately.
   [[NSNotificationCenter defaultCenter]
       addObserver:self
          selector:@selector(didChangeMainWindow:)
              name:NSWindowDidBecomeMainNotification
            object:[self window]];
   [[NSNotificationCenter defaultCenter]
       addObserver:self
          selector:@selector(didChangeMainWindow:)
              name:NSWindowDidResignMainNotification
            object:[self window]];
 }
 
 - (void)setTitle:(NSString*)newTitle {
   [title_ setStringValue:newTitle];
+

[Avi (use Gerrit), 2011/09/21 21:04:25]

remove extra line

[jianli, 2011/09/21 23:15:25]

Done.

+}
+
+- (void)setIcon:(NSImage*)newIcon {
+  [icon_ setImage:newIcon];
 }
 
 - (void)updateCloseButtonLayout {
   NSRect buttonBounds = [closeButton_ bounds];
   NSRect bounds = [self bounds];
 
-  int x = kCloseButtonLeftPadding;
+  int x = kButtonPadding;
   int y = (NSHeight(bounds) - NSHeight(buttonBounds)) / 2;
   NSRect buttonFrame = NSMakeRect(x,
                                   y,
                                   NSWidth(buttonBounds),
                                   NSHeight(buttonBounds));
   [closeButton_ setFrame:buttonFrame];
 
   DCHECK(!closeButtonTrackingArea_.get());
   closeButtonTrackingArea_.reset(
       [[CrTrackingArea alloc] initWithRect:buttonFrame
                                    options:(NSTrackingMouseEnteredAndExited |
                                             NSTrackingActiveAlways)
                               proxiedOwner:self
                                   userInfo:nil]);
   NSWindow* panelWindow = [self window];
   [closeButtonTrackingArea_.get() clearOwnerWhenWindowWillClose:panelWindow];
   [self addTrackingArea:closeButtonTrackingArea_.get()];
 }
 
+- (void)updateIconAndTitleLayout {
+  NSRect closeButtonBounds = [closeButton_ bounds];

[Dmitry Titov, 2011/09/21 21:32:11]

We should call those 'frame' ratehr then 'bounds' - because from their parent's
NSView, they have frames (and we use setFrame later to set it. So it makes sense
to name these variables iconFrame, etc and use [icon_ frame] to pull the initial
values.

The only one in the same cordinate system is 'bounds', it should indeed be
'bounds'.

[jennb, 2011/09/21 22:10:38]

Is updateCloseButtonLayout also using bounds incorrectly?

[jianli, 2011/09/21 23:15:25]

Changed to call sizeToFit before querying its bounds, per suggestion from Dave.

[jianli, 2011/09/21 23:15:25]

Done.

[jianli, 2011/09/21 23:15:25]

Done.

+  NSRect iconBounds = [icon_ bounds];
+  NSRect titleBounds = [title_ bounds];
+  NSRect settingsButtonBounds = [settingsButton_ bounds];
+  NSRect bounds = [self bounds];
+
+  // Compute the size of the title text.
+  NSDictionary* attributes =
+      [NSDictionary dictionaryWithObjectsAndKeys:[NSFont userFontOfSize:12],

[Avi (use Gerrit), 2011/09/21 21:04:25]

Don't make assumptions. You should be able to use [title_ font] to get the font
used.

[jianli, 2011/09/21 23:15:25]

Removed since it is not needed.

+       NSFontAttributeName,
+       nil];
+  NSAttributedString* text =
+      [[NSAttributedString alloc] initWithString:[title_ stringValue]
+                                      attributes:attributes];

[Avi (use Gerrit), 2011/09/21 21:04:25]

Alternatively, does [title_ attributedStringValue] have the correct font
tagging? If so, use it.

[Dmitry Titov, 2011/09/21 21:32:11]

'test' should be autoreleased in the same statement. Unless lifetime is manages
explicitly, 'alloc', 'init' and 'autorelease' should always be in the same
statement.

[jianli, 2011/09/21 23:15:25]

Removed since it is not needed.

+  NSSize titleSize = [text size];
+  int titleWidth = titleSize.width;
+
+  // Place the icon and title at the center of the titlebar.
+  int iconWidthWithPadding = NSWidth(iconBounds) + kIconAndTextPadding;
+  int availableWidth = NSWidth(bounds) - kButtonPadding * 4 -
+      NSWidth(closeButtonBounds) - NSWidth(settingsButtonBounds);
+  if (iconWidthWithPadding + titleWidth > availableWidth)
+    titleWidth = availableWidth - iconWidthWithPadding;
+  int startX = kButtonPadding * 2 + NSWidth(closeButtonBounds) +
+      (availableWidth - iconWidthWithPadding - titleWidth) / 2;
+
+  iconBounds.origin.x = startX;
+  iconBounds.origin.y = (NSHeight(bounds) - NSHeight(iconBounds)) / 2;
+  [icon_ setFrame:iconBounds];
+
+  titleBounds.origin.x = startX + iconWidthWithPadding;
+  titleBounds.origin.y = (NSHeight(bounds) - NSHeight(titleBounds)) / 2;
+  titleBounds.size.width = titleWidth;
+  [title_ setFrame:titleBounds];
+}
+
 // PanelManager controls size/position of the window.
 - (BOOL)mouseDownCanMoveWindow {
   return NO;
 }
 
 - (BOOL)acceptsFirstMouse:(NSEvent*)event {
   return YES;
 }
 
 - (void)mouseEntered:(NSEvent*)event {
@@ skipping 133 matching lines @@
 }
 
 - (void)checkMouseAndUpdateSettingsButtonVisibility {
   BOOL mouseOverWindow = NSPointInRect([NSEvent mouseLocation],
                                        [[controller_ window] frame]);
   [self updateSettingsButtonVisibility:mouseOverWindow];
 }
 
 @end