V1 of material design find in page bar

ui/views/controls/separator.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/separator.h"
 
 #include "ui/accessibility/ax_view_state.h"
 #include "ui/gfx/canvas.h"
 
 namespace views {
@@ skipping 26 matching lines @@
   if (size != size_) {
     size_ = size;
     PreferredSizeChanged();
   }
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // 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) {
   state->role = ui::AX_ROLE_SPLITTER;
 }
 
 void Separator::OnPaint(gfx::Canvas* canvas) {
   canvas->FillRect(GetContentsBounds(), color_);
 }
 
 const char* Separator::GetClassName() const {
   return kViewClassName;
 }
 
 }  // namespace views