V1 of material design find in page bar

ui/views/controls/separator.cc

Index: ui/views/controls/separator.cc
diff --git a/ui/views/controls/separator.cc b/ui/views/controls/separator.cc
index 9a9bbbdc881c4892648a69d5c28b29e349a2b153..e16346127ba4010be0e23670e856280dff926eda 100644
--- a/ui/views/controls/separator.cc
+++ b/ui/views/controls/separator.cc
@@ -44,9 +44,11 @@ void Separator::SetPreferredSize(int size) {
 // Separator, View overrides:
 
 gfx::Size Separator::GetPreferredSize() const {
-  if (orientation_ == HORIZONTAL)
-    return gfx::Size(width(), size_);
-  return gfx::Size(size_, height());
+  gfx::Size size =
+      orientation_ == HORIZONTAL ? gfx::Size(1, size_) : gfx::Size(size_, 1);
+  gfx::Insets insets = GetInsets();
+  size.Enlarge(insets.width(), insets.height());

[sky, 2015/08/05 00:06:01]

Your change here looks right, but how about landing this separately? I suggest
this mostly because I wouldn't be surprised if there are places that rely on the
old logic, and it might be best to resolve that separately.
As I'm ok with this change feel free to TBR it.

[Evan Stade, 2015/08/05 19:04:25]

Done.

+  return size;
 }
 
 void Separator::GetAccessibleState(ui::AXViewState* state) {