[Mac][Material Design] Fix bookmark chevron button focus ring location.

chrome/browser/ui/cocoa/bookmarks/bookmark_button_cell.mm

Index: chrome/browser/ui/cocoa/bookmarks/bookmark_button_cell.mm
diff --git a/chrome/browser/ui/cocoa/bookmarks/bookmark_button_cell.mm b/chrome/browser/ui/cocoa/bookmarks/bookmark_button_cell.mm
index 67f2aa5433f113d3b391ed3f95b5982fa315a21b..06aeb23d1e74dcd3d54e44a078efe3c577840ea6 100644
--- a/chrome/browser/ui/cocoa/bookmarks/bookmark_button_cell.mm
+++ b/chrome/browser/ui/cocoa/bookmarks/bookmark_button_cell.mm
@@ -7,6 +7,8 @@
 #include "base/logging.h"
 #include "base/strings/sys_string_conversions.h"
 #include "chrome/browser/ui/cocoa/bookmarks/bookmark_bar_constants.h"
+#import "chrome/browser/ui/cocoa/bookmarks/bookmark_bar_controller.h"
+#import "chrome/browser/ui/cocoa/bookmarks/bookmark_bar_view_cocoa.h"
 #import "chrome/browser/ui/cocoa/bookmarks/bookmark_button.h"
 #import "chrome/browser/ui/cocoa/bookmarks/bookmark_context_menu_cocoa_controller.h"
 #include "chrome/grit/generated_resources.h"
@@ -48,6 +50,9 @@ const int kDefaultFontSize = 12;
 - (NSString*)visibleTitle;
 // Returns the dictionary of attributes to associate with the button title.
 - (NSDictionary*)titleTextAttributes;
+// Returns the BookmarkBarController for the BookmarkButtonCell's controlView.
+- (BookmarkBarController*)
+    bookmarkBarViewControllerFromControlView:(NSView*)controlView;
 @end
 
 
@@ -371,13 +376,34 @@ const int kDefaultFontSize = 12;
   return kIconLeftPadding + [[self image] size].width + kIconTextSpacer;
 }
 
+- (BookmarkBarController*)
+    bookmarkBarViewControllerFromControlView:(NSView*)controlView {

[tapted, 2016/06/16 05:53:19]

having the button cell know much about its parents, and using rtti to find it,
probably breaks some kind of rule. I think the nice way to do this is to add a
property like `isOffTheSideButton()`, and have it return YES -- either by
setting a data member in bookmark_button_cell.mm's

// Used by the off-the-side menu, the only case where a
// BookmarkButtonCell is loaded from a nib.
- (void)awakeFromNib {
  [self configureBookmarkButtonCell];
}

(somewhat hacky)

OR having -[BookmarkBarController
configureOffTheSideButtonContentsAndVisibility] set it

OR creating a new subclass, just in bookmark_button_cell.mm like:

@interface OffTheSideButtonCell : BookmarkButtonCell
@end
@implementation
- (BOOL)isOffTheSideButton {
  return YES;
}
@end

then open BookmarkBar.xib in a text editor and change
customClass="BookmarkButtonCell" on line 25 to OffTheSideButtonCell

That's essentially the approach currently used by isFolderButtonCell - the only
downside is editing the .xib by hand, but this should be safe.

[shrike, 2016/06/16 15:19:09]

Great! My original preference was to change the cell class but I didn't think of
a way to do that outside of IB.

I am also unsure why we need a 2pt shift. My understanding of how the focus ring
drawing works is that the Appkit builds some kind of path based on the graphics
commands you execute during a certain window of time. That means the focus ring
should follow the bitmap.

+  NSView* nextView = [controlView superview];
+  while (nextView) {
+    if ([nextView isKindOfClass:[BookmarkBarView class]]) {
+      BookmarkBarView* bookmarkBarView = (BookmarkBarView*)nextView;
+      return [bookmarkBarView controller];
+    }
+    nextView = [nextView superview];
+  }
+  return nil;
+}
+
 - (void)drawFocusRingMaskWithFrame:(NSRect)cellFrame
                             inView:(NSView*)controlView {
-  // In Material Design we have to move the focus ring over by 2 pts to get it
-  // to line up with the image.
-  if (ui::MaterialDesignController::IsModeMaterial() &&
-      [self visibleTitle].length > 0) {
-    cellFrame.origin.x += 2;
+  if (ui::MaterialDesignController::IsModeMaterial()) {
+    // In Material Design we have to move the focus ring over by 2 pts to get it
+    // to line up with the image.
+    if ([self visibleTitle].length > 0) {
+      cellFrame.origin.x += 2;
+    }
+
+    // We also have to nudge the chevron button's focus ring up 2pts.
+    BookmarkBarController* bookmarkBarController =
+        [self bookmarkBarViewControllerFromControlView:controlView];
+    if ([bookmarkBarController offTheSideButton] == controlView) {
+      cellFrame.origin.y -= 2;
+    }
   }
   [super drawFocusRingMaskWithFrame:cellFrame inView:controlView];
 }