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

Unified Diff: chrome/renderer/searchbox/searchbox.cc

Issue 12047107: Change the SearchBox API from using the start/end margins of the location bar to using the start ma… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Sync Created 7 years, 10 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/renderer/searchbox/searchbox.cc
diff --git a/chrome/renderer/searchbox/searchbox.cc b/chrome/renderer/searchbox/searchbox.cc
index 41c29e47181e3e1f9704c26129206dc45bda458e..6df4c9be8f4fcd803b654c2e97921b05fb7fe453 100644
--- a/chrome/renderer/searchbox/searchbox.cc
+++ b/chrome/renderer/searchbox/searchbox.cc
@@ -17,7 +17,6 @@ SearchBox::SearchBox(content::RenderView* render_view)
selection_end_(0),
results_base_(0),
start_margin_(0),
- end_margin_(0),
last_results_base_(0),
is_key_capture_enabled_(false),
display_instant_results_(false),
@@ -69,10 +68,6 @@ int SearchBox::GetStartMargin() const {
return static_cast<int>(start_margin_ / GetZoom());
}
-int SearchBox::GetEndMargin() const {
- return static_cast<int>(end_margin_ / GetZoom());
-}
-
gfx::Rect SearchBox::GetPopupBounds() const {
double zoom = GetZoom();
return gfx::Rect(static_cast<int>(popup_bounds_.x() / zoom),
@@ -184,9 +179,12 @@ void SearchBox::OnPopupResize(const gfx::Rect& bounds) {
}
}
-void SearchBox::OnMarginChange(int start, int end) {
- start_margin_ = start;
- end_margin_ = end;
+void SearchBox::OnMarginChange(int margin, int width) {
+ start_margin_ = margin;
+
+ // Override only the width parameter of the popup bounds.
+ popup_bounds_.set_width(width);
+
if (render_view()->GetWebView() && render_view()->GetWebView()->mainFrame()) {
extensions_v8::SearchBoxExtension::DispatchMarginChange(
render_view()->GetWebView()->mainFrame());
@@ -280,7 +278,6 @@ void SearchBox::Reset() {
results_base_ = 0;
popup_bounds_ = gfx::Rect();
start_margin_ = 0;
- end_margin_ = 0;
autocomplete_results_.clear();
is_key_capture_enabled_ = false;
theme_info_ = ThemeBackgroundInfo();

Powered by Google App Engine
This is Rietveld 408576698