Add GetMinimumSize() for Borders, and make LabelButton auto-size to at least as

ash/wm/caption_buttons/maximize_bubble_controller_bubble.cc

Index: ash/wm/caption_buttons/maximize_bubble_controller_bubble.cc
===================================================================
--- ash/wm/caption_buttons/maximize_bubble_controller_bubble.cc	(revision 236640)
+++ ash/wm/caption_buttons/maximize_bubble_controller_bubble.cc	(working copy)
@@ -147,6 +147,7 @@
   virtual gfx::Rect GetBounds(const gfx::Rect& position_relative_to,
                               const gfx::Size& contents_size) const OVERRIDE;
   virtual void Paint(const views::View& view, gfx::Canvas* canvas) OVERRIDE;
+  virtual gfx::Size GetMinimumSize() const OVERRIDE;
 
  private:
   // Note: Animations can continue after then main window frame was destroyed.
@@ -255,7 +256,12 @@
   canvas->DrawPath(path, paint);
 }
 
+gfx::Size MaximizeBubbleBorder::GetMinimumSize() const {

[msw, 2013/11/22 18:00:25]

Why is the minimum size needed at all in cases like this?

[Peter Kasting, 2013/11/22 22:07:55]

I don't understand the question.  Are you saying, this patch doesn't use a
MaximizeBubbleBorder's minimum size, so don't write code for it?  I don't think
we should be providing methods that can be called on any border but are only
accurate for some borders.  That's a land mine waiting to happen.

+  return gfx::Size(kLineWidth * 2 + kArrowWidth,
+                   std::max(kLineWidth, kArrowHeight) + kLineWidth);
+}
 
+
 // BubbleMouseWatcherHost -----------------------------------------------------
 
 // The mouse watcher host which makes sure that the bubble does not get closed