Index: chrome/browser/ui/cocoa/bookmarks/bookmark_bar_folder_controller.mm |
diff --git a/chrome/browser/ui/cocoa/bookmarks/bookmark_bar_folder_controller.mm b/chrome/browser/ui/cocoa/bookmarks/bookmark_bar_folder_controller.mm |
index a16ebb7a3e2219840d0ec122c64d0caf29fb5019..98e0acb74b961e19853be7680ff089ded27df5a5 100644 |
--- a/chrome/browser/ui/cocoa/bookmarks/bookmark_bar_folder_controller.mm |
+++ b/chrome/browser/ui/cocoa/bookmarks/bookmark_bar_folder_controller.mm |
@@ -124,26 +124,13 @@ |
NSRect GetFirstButtonFrameForHeight(CGFloat height) { |
CGFloat y = height - bookmarks::kBookmarkFolderButtonHeight - |
- bookmarks::BookmarkTopVerticalPadding(); |
+ bookmarks::kBookmarkTopVerticalPadding; |
return NSMakeRect(0, y, bookmarks::kDefaultBookmarkWidth, |
bookmarks::kBookmarkFolderButtonHeight); |
} |
} // namespace |
-namespace bookmarks { |
- |
-CGFloat BookmarkTopVerticalPadding() { |
- return bookmarks::BookmarkVerticalPadding(); |
-} |
- |
-CGFloat BookmarkBottomVerticalPadding() { |
- return ui::MaterialDesignController::IsModeMaterial() |
- ? 0 : bookmarks::BookmarkVerticalPadding(); |
-} |
- |
-} // bookmarks |
- |
// Required to set the right tracking bounds for our fake menus. |
@interface NSView(Private) |
@@ -331,18 +318,15 @@ - (void)dealloc { |
- (void)awakeFromNib { |
NSRect windowFrame = [[self window] frame]; |
NSRect scrollViewFrame = [scrollView_ frame]; |
- padding_ = NSWidth(windowFrame) - NSWidth(scrollViewFrame); |
- // In Material Design, each menu row spans the entire width of the menu. |
- if (ui::MaterialDesignController::IsModeMaterial()) { |
- scrollViewFrame.size.width += padding_; |
- scrollViewFrame.origin.x = 0; |
- // If we leave the scrollview's vertical position and height as is, it will |
- // cover the menu's rounded corners and we'll get a rectangular menu. |
- scrollViewFrame.origin.y += bookmarks::BookmarkVerticalPadding(); |
- scrollViewFrame.size.height -= 2 * bookmarks::BookmarkVerticalPadding(); |
- [scrollView_ setFrame:scrollViewFrame]; |
- padding_ = 0; |
- } |
+ // Each menu row spans the entire width of the menu. |
+ scrollViewFrame.size.width += NSWidth(windowFrame) - NSWidth(scrollViewFrame); |
+ scrollViewFrame.origin.x = 0; |
+ // If we leave the scrollview's vertical position and height as is, it will |
+ // cover the menu's rounded corners and we'll get a rectangular menu. |
+ scrollViewFrame.origin.y += bookmarks::kBookmarkVerticalPadding; |
+ scrollViewFrame.size.height -= 2 * bookmarks::kBookmarkVerticalPadding; |
+ [scrollView_ setFrame:scrollViewFrame]; |
+ padding_ = 0; |
verticalScrollArrowHeight_ = NSHeight([scrollUpArrowView_ frame]); |
ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance(); |
@@ -400,11 +384,7 @@ - (BookmarkButtonCell*)cellForBookmarkNode:(const BookmarkNode*)child { |
text:nil |
image:image |
menuController:menuController]; |
- if (ui::MaterialDesignController::IsModeMaterial()) { |
- [cell setTag:kMaterialMenuButtonTypeWithLimitedClickFeedback]; |
- } else { |
- [cell setTag:kStandardButtonTypeWithLimitedClickFeedback]; |
- } |
+ [cell setTag:kMaterialMenuButtonTypeWithLimitedClickFeedback]; |
return cell; |
} |
@@ -453,11 +433,9 @@ - (BookmarkButton*)makeButtonForNode:(const BookmarkNode*)node |
autorelease]; |
DCHECK(button); |
- // Folder menu buttons have a flat background color in Material Design. |
- if (ui::MaterialDesignController::IsModeMaterial()) { |
- [button setBackgroundColor: |
- [BookmarkBarFolderWindowContentView backgroundColor]]; |
- } |
+ // Folder menu buttons have a flat background color. |
+ [button setBackgroundColor: |
+ [BookmarkBarFolderWindowContentView backgroundColor]]; |
[button setCell:cell]; |
[button setDelegate:self]; |
if (node) { |
@@ -553,9 +531,6 @@ - (NSPoint)windowTopLeftForWidth:(int)windowWidth height:(int)windowHeight { |
newWindowTopLeft = NSMakePoint( |
buttonBottomLeftInScreen.x, |
bookmarkBarBottomLeftInScreen.y + bookmarks::kBookmarkBarMenuOffset); |
- if (!ui::MaterialDesignController::IsModeMaterial()) { |
- newWindowTopLeft.y += bookmarks::kBookmarkBarButtonOffset; |
- } |
// Make sure the window is on-screen; if not, push left or right. It is |
// intentional that top level folders "push left" or "push right" slightly |
// different than subfolders. |
@@ -586,7 +561,7 @@ - (NSPoint)windowTopLeftForWidth:(int)windowWidth height:(int)windowHeight { |
newWindowTopLeft.x = [self childFolderWindowLeftForWidth:windowWidth]; |
NSPoint topOfWindow = |
NSMakePoint(0, NSMaxY([parentButton_ frame]) + |
- bookmarks::BookmarkTopVerticalPadding()); |
+ bookmarks::kBookmarkTopVerticalPadding); |
topOfWindow = ui::ConvertPointFromWindowToScreen( |
[parentButton_ window], |
[[parentButton_ superview] convertPoint:topOfWindow toView:nil]); |
@@ -605,8 +580,8 @@ - (int)menuHeightForButtonCount:(int)buttonCount { |
// This does not take into account any padding which may be required at the |
// top and/or bottom of the window. |
return (buttonCount * bookmarks::kBookmarkFolderButtonHeight) + |
- bookmarks::BookmarkTopVerticalPadding() + |
- bookmarks::BookmarkBottomVerticalPadding(); |
+ bookmarks::kBookmarkTopVerticalPadding + |
+ bookmarks::kBookmarkBottomVerticalPadding; |
} |
- (void)adjustWindowLeft:(CGFloat)windowLeft |
@@ -889,9 +864,7 @@ - (void)layOutWindowWithHeight:(CGFloat)height { |
// 4pt of padding pushing the topmost item beyond the top of the |
// |scrollView_|. Scoot the |folderView_| down by this padding to avoid |
// clipping the topmost item. |
- if (ui::MaterialDesignController::IsModeMaterial()) { |
- folderFrame.origin.y -= bookmarks::BookmarkVerticalPadding(); |
- } |
+ folderFrame.origin.y -= bookmarks::kBookmarkVerticalPadding; |
[folderView_ setFrame:folderFrame]; |
// For some reason, when opening a "large" bookmark folder (containing 12 or |
@@ -1796,7 +1769,7 @@ - (CGFloat)indicatorPosForDragToPoint:(NSPoint)point { |
[buttons_ objectAtIndex:static_cast<NSUInteger>(destIndex)]; |
DCHECK(button); |
NSRect buttonFrame = [button frame]; |
- y = NSMaxY(buttonFrame) + 0.5 * bookmarks::BookmarkTopVerticalPadding(); |
+ y = NSMaxY(buttonFrame) + 0.5 * bookmarks::kBookmarkTopVerticalPadding; |
// If it's a drop at the end (past the last button, if there are any) ... |
} else if (destIndex == numButtons) { |
@@ -1808,7 +1781,7 @@ - (CGFloat)indicatorPosForDragToPoint:(NSPoint)point { |
DCHECK(button); |
NSRect buttonFrame = [button frame]; |
y = buttonFrame.origin.y - |
- 0.5 * bookmarks::BookmarkBottomVerticalPadding(); |
+ 0.5 * bookmarks::kBookmarkBottomVerticalPadding; |
} |
} else { |