Remove BubbleDelegateView ctor color arg, add simple setter/getter.

ui/views/bubble/bubble_delegate.h

Index: ui/views/bubble/bubble_delegate.h
diff --git a/ui/views/bubble/bubble_delegate.h b/ui/views/bubble/bubble_delegate.h
index 0e7da9ba627f7395f0118a24f5fd69c03539ac32..c171b6c822d3c989fb6cf855629d4dff56a158e3 100644
--- a/ui/views/bubble/bubble_delegate.h
+++ b/ui/views/bubble/bubble_delegate.h
@@ -28,10 +28,12 @@ class VIEWS_EXPORT BubbleDelegateView : public WidgetDelegateView,
                                         public ui::AnimationDelegate,
                                         public Widget::Observer {
  public:
+  // The default bubble background color.
+  static const SkColor kBackgroundColor;
+
   BubbleDelegateView();
   BubbleDelegateView(View* anchor_view,
-                     BubbleBorder::ArrowLocation arrow_location,
-                     const SkColor& color);
+                     BubbleBorder::ArrowLocation arrow_location);
   virtual ~BubbleDelegateView();
 
   // Create and initialize the bubble Widget(s) with proper bounds.
@@ -62,6 +64,9 @@ class VIEWS_EXPORT BubbleDelegateView : public WidgetDelegateView,
   View* anchor_view() const { return anchor_view_; }
   void set_anchor_view(View* anchor_view) { anchor_view_ = anchor_view; }
 
+  SkColor color() const { return color_; }
+  void set_color(const SkColor& color) { color_ = color; }

[sky, 2011/12/07 22:12:47]

nit: make these two consistent in the type. I would be inclined to make
set_color not take a ref.

[tfarina, 2011/12/07 22:15:19]

yeah, probably it should be passed just by value as SkColor is typedefed to
uint32_t.

[msw, 2011/12/07 22:26:24]

Done.

+
   bool use_focusless() const { return use_focusless_; }
   void set_use_focusless(bool use_focusless) {
     use_focusless_ = use_focusless;
@@ -73,9 +78,6 @@ class VIEWS_EXPORT BubbleDelegateView : public WidgetDelegateView,
   // Get the arrow's location on the bubble.
   virtual BubbleBorder::ArrowLocation GetArrowLocation() const;
 
-  // Get the color used for the background and border.
-  virtual SkColor GetColor() const;
-
   // Show the bubble's widget (and |border_widget_| on Windows).
   void Show();
 
@@ -87,9 +89,6 @@ class VIEWS_EXPORT BubbleDelegateView : public WidgetDelegateView,
   // bubble to the setting before StartFade() was called.
   void ResetFade();
 
-  // The default bubble background color.
-  static const SkColor kBackgroundColor;
-
  protected:
   // View overrides:
   virtual bool AcceleratorPressed(const ui::Accelerator& accelerator) OVERRIDE;