Makes Separator more configurable.

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 {
 
 // static
 const char Separator::kViewClassName[] = "Separator";
 
 // The separator height in pixels.
 const int kSeparatorHeight = 1;
 
 // Default color of the separator.
 const SkColor kDefaultColor = SkColorSetARGB(255, 233, 233, 233);
 
-Separator::Separator(Orientation orientation) : orientation_(orientation) {
+Separator::Separator(Orientation orientation)
+    : Separator(orientation, kDefaultColor, kSeparatorHeight) {
+}
+
+Separator::Separator(Orientation orientation, SkColor color)
+  : Separator(orientation, color, kSeparatorHeight) {
+}
+
+Separator::Separator(Orientation orientation, SkColor color, int height)
+    : orientation_(orientation),
+      color_(color),
+      height_(height) {
   SetFocusable(false);
 }
 
 Separator::~Separator() {
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // Separator, View overrides:
 
 gfx::Size Separator::GetPreferredSize() const {
   if (orientation_ == HORIZONTAL)
-    return gfx::Size(width(), kSeparatorHeight);
-  return gfx::Size(kSeparatorHeight, height());
+    return gfx::Size(width(), height_);
+  return gfx::Size(height_, height());
 }
 
 void Separator::GetAccessibleState(ui::AXViewState* state) {
   state->role = ui::AX_ROLE_SPLITTER;
 }
 
 void Separator::Paint(gfx::Canvas* canvas, const views::CullSet& cull_set) {
-  canvas->FillRect(bounds(), kDefaultColor);
+  canvas->FillRect(bounds(), color_);
 }
 
 const char* Separator::GetClassName() const {
   return kViewClassName;
 }
 
 }  // namespace views