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

Unified Diff: chrome/browser/ui/views/frame/browser_view_layout.cc

Issue 2555623002: Last round of cleaning up MD layout constants. (Closed)
Patch Set: pkasting review, remove debug code, rebase Created 4 years 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/views/frame/browser_view_layout.cc
diff --git a/chrome/browser/ui/views/frame/browser_view_layout.cc b/chrome/browser/ui/views/frame/browser_view_layout.cc
index 18c70c1a859b17c7e808fae8d708dae13517a9b3..11214a86ecf0cc95c204eea760a98d20fcf6504d 100644
--- a/chrome/browser/ui/views/frame/browser_view_layout.cc
+++ b/chrome/browser/ui/views/frame/browser_view_layout.cc
@@ -12,7 +12,6 @@
#include "chrome/browser/ui/browser_window.h"
#include "chrome/browser/ui/find_bar/find_bar.h"
#include "chrome/browser/ui/find_bar/find_bar_controller.h"
-#include "chrome/browser/ui/layout_constants.h"
#include "chrome/browser/ui/search/search_model.h"
#include "chrome/browser/ui/views/bookmarks/bookmark_bar_view.h"
#include "chrome/browser/ui/views/download/download_shelf_view.h"
@@ -222,10 +221,8 @@ gfx::Rect BrowserViewLayout::GetFindBarBoundingBox() const {
find_bar_y = top_container_bounds.bottom();
} else {
// Overlap the find bar atop |top_container_|.
- // The find bar should look connected to the top container when material
- // design is not enabled.
- find_bar_y = top_container_bounds.bottom() -
- GetLayoutConstant(FIND_BAR_TOOLBAR_OVERLAP);
+ const int kTopOverlap = 6;
+ find_bar_y = top_container_bounds.bottom() - kTopOverlap;
}
// Grow the height of |bounding_box| by the height of any elements between

Powered by Google App Engine
This is Rietveld 408576698