Makes Separator more configurable.

ui/views/controls/separator.h

Index: ui/views/controls/separator.h
diff --git a/ui/views/controls/separator.h b/ui/views/controls/separator.h
index 39004baf378e41b7c839d7199cb4da9fb8d37430..e0021a1b99f216d560d5806541fb3a8678ed725c 100644
--- a/ui/views/controls/separator.h
+++ b/ui/views/controls/separator.h
@@ -25,6 +25,9 @@ class VIEWS_EXPORT Separator : public View {
   static const char kViewClassName[];
 
   explicit Separator(Orientation orientation);
+  Separator(Orientation orientation, SkColor color);

[sky, 2014/11/18 18:04:06]

We generally don't have constructors like this. At one point the style guide
frowned upon this, not sure that it does anymore.

Don't add any new constructors, instead add setters for the color and height.
Also, it shouldn't be called height but rather size.

[melandory, 2014/11/18 19:12:47]

Done.

+  Separator(Orientation orientation, SkColor color, int height);
+
   ~Separator() override;
 
   // Overridden from View:
@@ -35,6 +38,8 @@ class VIEWS_EXPORT Separator : public View {
 
  private:
   const Orientation orientation_;
+  const SkColor color_;
+  const int height_;
 
   DISALLOW_COPY_AND_ASSIGN(Separator);
 };