Cleanup BubbleFrameView and BubbleBorder construction.

ui/views/bubble/bubble_border.h

Index: ui/views/bubble/bubble_border.h
diff --git a/ui/views/bubble/bubble_border.h b/ui/views/bubble/bubble_border.h
index 6745278e76321b3ea6358239551979f54e0bcbab..78ba6cdcdb17e0dee8709acc44d027601737766a 100644
--- a/ui/views/bubble/bubble_border.h
+++ b/ui/views/bubble/bubble_border.h
@@ -65,7 +65,7 @@ class VIEWS_EXPORT BubbleBorder : public Border {
     ALIGN_EDGE_TO_ANCHOR_EDGE
   };
 
-  BubbleBorder(ArrowLocation arrow_location, Shadow shadow);
+  BubbleBorder(ArrowLocation arrow, Shadow shadow, SkColor color);

[sky, 2013/01/31 19:31:16]

Is there a compelling reason you're adding this? With the current approach it's
easy to get the right thing (white), but with the new approach folks have to
think about what color they should use and possibly get it wrong.

[msw, 2013/01/31 20:11:40]

My thought was that hardcoded white is wrong, and most/all bubbles should use
theme colors. Most/all bubble should use BubbleDelegateView, which makes it easy
to use theme colors (or override with a custom color). I hope to convert a few
outliers to BubbleDelegateView in followups; others should know their color.

 
   // Returns the radius of the corner of the border.
   // TODO(xiyuan): Get rid of this since it's part of BorderImages now?