Clean-up positioning and spacing in chrome/browser/ui/views/location_bar/location_bar_view.cc

chrome/browser/ui/views/location_bar/ev_bubble_view.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/views/location_bar/ev_bubble_view.h"
 
+namespace {
+
+// Set a minimum size for elided EV bubbles 150 px.

[Peter Kasting, 2012/12/21 19:17:01]

Nit: Grammar

[beaudoin, 2013/01/03 03:30:45]

Done.

+static const int kMinBubbleWidth = 150;

[Peter Kasting, 2012/12/21 19:17:01]

Nit: Just declare this in the function where you use it.

[beaudoin, 2013/01/03 03:30:45]

Done.

+
+}  // namespace
+
 EVBubbleView::EVBubbleView(const int background_images[],
                            int contained_image,
                            SkColor color,
                            LocationBarView* location_bar)
     : IconLabelBubbleView(background_images, contained_image, color),
       ALLOW_THIS_IN_INITIALIZER_LIST(page_info_helper_(this, location_bar)) {
   SetElideInMiddle(true);
 }
 
 EVBubbleView::~EVBubbleView() {
 }
 
+gfx::Size EVBubbleView::GetMinimumSize() {
+  gfx::Size minimum(GetPreferredSize());
+  minimum.set_width(std::min(kMinBubbleWidth, minimum.width()));
+  return minimum;
+}
+
 bool EVBubbleView::OnMousePressed(const ui::MouseEvent& event) {
   // We want to show the dialog on mouse release; that is the standard behavior
   // for buttons.
   return true;
 }
 
 void EVBubbleView::OnMouseReleased(const ui::MouseEvent& event) {
   page_info_helper_.ProcessEvent(event);
 }
 
 void EVBubbleView::OnGestureEvent(ui::GestureEvent* event) {
   if (event->type() == ui::ET_GESTURE_TAP) {
     page_info_helper_.ProcessEvent(*event);
     event->SetHandled();
   }
 }