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

Unified Diff: chrome/browser/ui/search/search_ui.cc

Issue 10832216: Clicks broken in NTP section of Search overlay (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressing comments. Created 8 years, 4 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
« no previous file with comments | « chrome/browser/ui/search/search_tab_helper.cc ('k') | chrome/browser/ui/views/frame/browser_view.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/search/search_ui.cc
diff --git a/chrome/browser/ui/search/search_ui.cc b/chrome/browser/ui/search/search_ui.cc
index 94d8edc9d9ef9fcd02e9e5873e157b0c7f00fead..bcab76113e03deb122357fec8fe142a9017fde45 100644
--- a/chrome/browser/ui/search/search_ui.cc
+++ b/chrome/browser/ui/search/search_ui.cc
@@ -23,7 +23,10 @@ const int kNTPOmniboxFontSize = 18;
const int kNTPOmniboxHeight = 40;
// See the comments in browser_defaults on kAutocompleteEditFontPixelSize.
const int kOmniboxFontSize = 16;
+// Relative to top of ContentsContainer.
const int kOmniboxYPosition = 310;
+// Relative to top of WebView in search overlay.
+const int kOmniboxYOffset = -50;
const int kSearchResultsHeight = 122;
gfx::Rect GetNTPOmniboxBounds(const gfx::Size& web_contents_size) {
@@ -33,7 +36,7 @@ gfx::Rect GetNTPOmniboxBounds(const gfx::Size& web_contents_size) {
int width = static_cast<int>(kNTPPageWidthRatio *
static_cast<double>(web_contents_size.width()));
int x = (web_contents_size.width() - width) / 2;
- return gfx::Rect(x, kOmniboxYPosition, width, 0);
+ return gfx::Rect(x, kOmniboxYOffset, width, 0);
}
gfx::Font GetNTPOmniboxFont(const gfx::Font& font) {
« no previous file with comments | « chrome/browser/ui/search/search_tab_helper.cc ('k') | chrome/browser/ui/views/frame/browser_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698