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

Unified Diff: trunk/src/chrome/browser/ui/cocoa/bookmarks/bookmark_bar_controller.mm

Issue 13006019: Revert 190584 "Alternate NTP Mac: Show overlay at fixed position" (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 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: trunk/src/chrome/browser/ui/cocoa/bookmarks/bookmark_bar_controller.mm
===================================================================
--- trunk/src/chrome/browser/ui/cocoa/bookmarks/bookmark_bar_controller.mm (revision 190730)
+++ trunk/src/chrome/browser/ui/cocoa/bookmarks/bookmark_bar_controller.mm (working copy)
@@ -128,6 +128,10 @@
namespace {
+// Overlap (in pixels) between the toolbar and the bookmark bar (when showing in
+// normal mode).
+const CGFloat kBookmarkBarOverlap = 3.0;
+
// Duration of the bookmark bar animations.
const NSTimeInterval kBookmarkBarAnimationDuration = 0.12;
@@ -524,8 +528,7 @@
// If we ever need any other animation cases, code would go here.
}
- return [self isInState:BookmarkBar::SHOW] ? bookmarks::kBookmarkBarOverlap
- : 0;
+ return [self isInState:BookmarkBar::SHOW] ? kBookmarkBarOverlap : 0;
}
- (CGFloat)toolbarDividerOpacity {
@@ -892,7 +895,7 @@
// Height takes into account the extra height we have since the toolbar
// only compresses when we're done.
[view animateToNewHeight:(bookmarks::kBookmarkBarHeight -
- bookmarks::kBookmarkBarOverlap)
+ kBookmarkBarOverlap)
duration:kBookmarkBarAnimationDuration];
} else if ([self isAnimatingFromState:BookmarkBar::SHOW
toState:BookmarkBar::HIDDEN]) {
@@ -919,7 +922,7 @@
// Height takes into account the extra height we have since the toolbar
// only compresses when we're done.
[view animateToNewHeight:(bookmarks::kBookmarkBarHeight -
- bookmarks::kBookmarkBarOverlap)
+ kBookmarkBarOverlap)
duration:kBookmarkBarAnimationDuration];
} else {
// Oops! An animation we don't know how to handle.

Powered by Google App Engine
This is Rietveld 408576698