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

Unified Diff: chrome/browser/ui/views/location_bar/ev_bubble_view.cc

Issue 271013002: Compute minimum widths for the toolbar components. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 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/views/location_bar/ev_bubble_view.cc
diff --git a/chrome/browser/ui/views/location_bar/ev_bubble_view.cc b/chrome/browser/ui/views/location_bar/ev_bubble_view.cc
index ad5c27afcdf7db1b339b908ed4d871b0d01db7fd..4698cce4a0f2a7c9642da99ee43e49221322bdb7 100644
--- a/chrome/browser/ui/views/location_bar/ev_bubble_view.cc
+++ b/chrome/browser/ui/views/location_bar/ev_bubble_view.cc
@@ -28,7 +28,7 @@ gfx::Size EVBubbleView::GetMinimumSize() {
// Height will be ignored by the LocationBarView.
gfx::Size minimum(GetPreferredSize());
static const int kMinBubbleWidth = 150;
- minimum.SetToMax(gfx::Size(kMinBubbleWidth, 0));
+ minimum.SetToMin(gfx::Size(kMinBubbleWidth, minimum.height()));
return minimum;
}
@@ -48,3 +48,11 @@ void EVBubbleView::OnGestureEvent(ui::GestureEvent* event) {
event->SetHandled();
}
}
+
+gfx::Size EVBubbleView::GetMinimumSizeForLabelText(
+ const base::string16& text) const {
+ gfx::Size minimum(GetSizeForLabelText(text));
+ static const int kMinBubbleWidth = 150;
+ minimum.SetToMin(gfx::Size(kMinBubbleWidth, minimum.height()));
+ return minimum;
+}

Powered by Google App Engine
This is Rietveld 408576698