[SiteChip] Add the basic painting of the site chip button

chrome/browser/ui/views/toolbar/toolbar_view.h

Index: chrome/browser/ui/views/toolbar/toolbar_view.h
diff --git a/chrome/browser/ui/views/toolbar/toolbar_view.h b/chrome/browser/ui/views/toolbar/toolbar_view.h
index 6818931c37a59f596a2834882bd00db59d053f9e..f84a818815b190ddbec2a90e4322e176c8883de7 100644
--- a/chrome/browser/ui/views/toolbar/toolbar_view.h
+++ b/chrome/browser/ui/views/toolbar/toolbar_view.h
@@ -25,6 +25,7 @@ class BrowserActionsContainer;
 class Browser;
 class HomeButton;
 class ReloadButton;
+class SiteChipView;
 class ToolbarButton;
 class WrenchMenu;
 class WrenchMenuModel;
@@ -86,6 +87,7 @@ class ToolbarView : public views::AccessiblePaneView,
   BrowserActionsContainer* browser_actions() const { return browser_actions_; }
   ReloadButton* reload_button() const { return reload_; }
   LocationBarView* location_bar() const { return location_bar_; }
+  SiteChipView* site_chip() const { return site_chip_view_; }
   views::MenuButton* app_menu() const;
 
   // Overridden from AccessiblePaneView
@@ -154,6 +156,9 @@ class ToolbarView : public views::AccessiblePaneView,
   bool ShouldPaintBackground() const;
 
   enum {
+    // The apparent horizontal space around the site chip.

[Peter Kasting, 2013/12/03 23:04:52]

Based on my comments on the bug: we should nuke this and use kStandardSpacing
instead.  Using 2 instead of 3 here just looks strange.

[Greg Billock, 2013/12/04 00:33:34]

Done.

+    kSiteChipSpacing = 2,
+
     // The apparent horizontal space between most items, and the vertical
     // padding above and below them.
     kStandardSpacing = 3,
@@ -217,6 +222,7 @@ class ToolbarView : public views::AccessiblePaneView,
   ReloadButton* reload_;
   HomeButton* home_;
   LocationBarView* location_bar_;
+  SiteChipView* site_chip_view_;
   BrowserActionsContainer* browser_actions_;
   WrenchToolbarButton* app_menu_;
   Browser* browser_;