Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2694)

Unified Diff: chrome/browser/ui/cocoa/bookmarks/bookmark_bar_view_cocoa.mm

Issue 2751573002: [Mac] Refactor bookmark bar controller (Closed)
Patch Set: Restore unused button pool Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/cocoa/bookmarks/bookmark_bar_view_cocoa.mm
diff --git a/chrome/browser/ui/cocoa/bookmarks/bookmark_bar_view_cocoa.mm b/chrome/browser/ui/cocoa/bookmarks/bookmark_bar_view_cocoa.mm
index 1a44e33a0f07ce7fcce9f498ca81bd20b04cd94c..d5805df4bc99b72b7134a6f8ccf1602e3046513f 100644
--- a/chrome/browser/ui/cocoa/bookmarks/bookmark_bar_view_cocoa.mm
+++ b/chrome/browser/ui/cocoa/bookmarks/bookmark_bar_view_cocoa.mm
@@ -28,14 +28,10 @@ using base::UserMetricsAction;
using bookmarks::BookmarkModel;
using bookmarks::BookmarkNode;
-static const CGFloat kInitialContainerWidth = 596;
-static const CGFloat kInitialContainerHeight = 41;
static const CGFloat kInitialElementYOrigin = 20;
static const CGFloat kInitialElementHeight = 14;
static const CGFloat kInitialTextFieldXOrigin = 5;
-// static const CGFloat kInitialTextFieldWidth = 167;
static const CGFloat kTextFieldTrailingPadding = 5;
-// static const CGFloat kInitialButtonWidth = 199;
@interface BookmarkBarView (Private)
- (void)themeDidChangeNotification:(NSNotification*)aNotification;
@@ -80,48 +76,40 @@ static const CGFloat kTextFieldTrailingPadding = 5;
NSFont* smallSystemFont =
[NSFont systemFontOfSize:[NSFont smallSystemFontSize]];
- noItemContainer_.reset(
- [[NSView alloc] initWithFrame:NSMakeRect(0, 0, kInitialContainerWidth,
- kInitialContainerHeight)]);
- [noItemContainer_ setAutoresizingMask:NSViewMaxXMargin];
- noItemTextfield_.reset([[NSTextField alloc]
+ noItemTextField_.reset([[NSTextField alloc]
initWithFrame:NSMakeRect(kInitialTextFieldXOrigin,
kInitialElementYOrigin, CGFLOAT_MAX,
kInitialElementHeight)]);
- [noItemTextfield_ setAutoresizingMask:NSViewWidthSizable];
- [noItemTextfield_ setFont:smallSystemFont];
- [noItemTextfield_
+ [noItemTextField_ setFont:smallSystemFont];
+ [noItemTextField_
setStringValue:l10n_util::GetNSString(IDS_BOOKMARKS_NO_ITEMS)];
- [noItemTextfield_ setEditable:NO];
+ [noItemTextField_ setEditable:NO];
- [noItemTextfield_ setBordered:NO];
- [[noItemTextfield_ cell] setLineBreakMode:NSLineBreakByTruncatingTail];
+ [noItemTextField_ setBordered:NO];
+ [[noItemTextField_ cell] setLineBreakMode:NSLineBreakByTruncatingTail];
- [noItemTextfield_ setTextColor:[NSColor controlTextColor]];
- [noItemTextfield_ setBackgroundColor:[NSColor controlColor]];
+ [noItemTextField_ setTextColor:[NSColor controlTextColor]];
+ [noItemTextField_ setBackgroundColor:[NSColor controlColor]];
- [noItemTextfield_ setDrawsBackground:NO];
- [noItemTextfield_ setTextColor:[NSColor controlTextColor]];
- [noItemTextfield_ setBackgroundColor:[NSColor controlColor]];
- [noItemTextfield_ sizeToFit];
+ [noItemTextField_ setDrawsBackground:NO];
+ [noItemTextField_ setTextColor:[NSColor controlTextColor]];
+ [noItemTextField_ setBackgroundColor:[NSColor controlColor]];
+ [noItemTextField_ sizeToFit];
NSButton* importButton = [HyperlinkButtonCell
buttonWithString:l10n_util::GetNSString(IDS_BOOKMARK_BAR_IMPORT_LINK)];
importBookmarksButton_.reset([importButton retain]);
[importBookmarksButton_
- setFrame:NSMakeRect(NSMaxX([noItemTextfield_ frame]) +
+ setFrame:NSMakeRect(NSMaxX([noItemTextField_ frame]) +
kTextFieldTrailingPadding,
kInitialElementYOrigin, CGFLOAT_MAX,
kInitialElementHeight)];
- [importBookmarksButton_ setAutoresizingMask:NSViewMaxXMargin];
[importBookmarksButton_ setFont:smallSystemFont];
[importBookmarksButton_ sizeToFit];
- [noItemContainer_ addSubview:importBookmarksButton_];
- [noItemContainer_ addSubview:noItemTextfield_];
-
- [self addSubview:noItemContainer_];
+ [self addSubview:noItemTextField_];
+ [self addSubview:importBookmarksButton_];
[self registerForNotificationsAndDraggedTypes];
}
return self;
@@ -169,7 +157,7 @@ static const CGFloat kTextFieldTrailingPadding = 5;
NSColor* color =
themeProvider->GetNSColor(ThemeProperties::COLOR_BOOKMARK_TEXT);
- [noItemTextfield_ setTextColor:color];
+ [noItemTextField_ setTextColor:color];
}
// Mouse down events on the bookmark bar should not allow dragging the parent
@@ -178,12 +166,8 @@ static const CGFloat kTextFieldTrailingPadding = 5;
return NO;
}
-- (NSTextField*)noItemTextfield {
- return noItemTextfield_;
-}
-
-- (NSView*)noItemContainer {
- return noItemContainer_;
+- (NSTextField*)noItemTextField {
+ return noItemTextField_;
}
- (NSButton*)importBookmarksButton {
@@ -252,7 +236,6 @@ static const CGFloat kTextFieldTrailingPadding = 5;
- (void)draggingEnded:(id<NSDraggingInfo>)info {
[controller_ draggingEnded:info];
- [[BookmarkButton draggedButton] setHidden:NO];
if (dropIndicatorShown_) {
dropIndicatorShown_ = NO;
[self dropIndicatorChanged];

Powered by Google App Engine
This is Rietveld 408576698