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

ui/views/controls/focusable_border.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 "ui/views/controls/focusable_border.h"
 
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/insets.h"
 #include "ui/gfx/skia_util.h"
 #include "ui/native_theme/native_theme.h"
@@ skipping 42 matching lines @@
   paint.setColor(color);
   paint.setStrokeWidth(SkIntToScalar(2));
 
   canvas->DrawPath(path, paint);
 }
 
 gfx::Insets FocusableBorder::GetInsets() const {
   return insets_;
 }
 
+gfx::Size FocusableBorder::GetMinimumSize() const {
+  return gfx::Size();

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

Shouldn't this minimum size match the insets or the 2px stroke width?

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

In the case where the local bounds are an empty rect, Paint() draws no border at
all (as opposed to trying to calculate rects with negative widths or something),
which means the border can draw "correctly" even at zero size.

+}
+
 void FocusableBorder::SetInsets(int top, int left, int bottom, int right) {
   insets_.Set(top, left, bottom, right);
 }
 
 }  // namespace views