Don't expose the go and star button views directly through the BrowserWindow ...

chrome/browser/browser_window.h

 // Copyright (c) 2006-2008 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.
 
 #ifndef CHROME_BROWSER_BROWSER_WINDOW_H_
 #define CHROME_BROWSER_BROWSER_WINDOW_H_
 
 class BookmarkBarView;
 class Browser;
 class BrowserList;
 class BrowserView;
 class BrowserWindowTesting;
-class GoButton;
+class GURL;
 class LocationBarView;
 class HtmlDialogContentsDelegate;
 class Profile;
 class StatusBubble;
 class TabContents;
-class TabStrip;
-class ToolbarStarToggle;
 
 namespace gfx {
 class Rect;
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // BrowserWindow interface
 //  An interface implemented by the "view" of the Browser window.
 //
 // NOTE: all getters, save GetTabStrip(), may return NULL.
@@ skipping 21 matching lines @@
   virtual void FlashFrame() = 0;
 
   // Return a platform dependent identifier for this frame. On Windows, this
   // returns an HWND. DO NOT USE IN CROSS PLATFORM CODE.
   virtual void* GetNativeHandle() = 0;
 
   // Returns a pointer to the testing interface to the Browser window, or NULL
   // if there is none.
   virtual BrowserWindowTesting* GetBrowserWindowTesting() = 0;
 
-  // TODO(beng): REMOVE (obtain via BrowserFrame).
-  // Return the TabStrip associated with the frame.
-  virtual TabStrip* GetTabStrip() const = 0;
-
   // Return the status bubble associated with the frame
   virtual StatusBubble* GetStatusBubble() = 0;
 
   // Inform the receiving frame that an animation has progressed in the
   // selected tab.
   // TODO(beng): Remove. Infobars/Boomarks bars should talk directly to
   //             BrowserView.
   virtual void SelectedTabToolbarSizeChanged(bool is_animating) = 0;
 
   // Inform the frame that the selected tab favicon or title has changed. Some
   // frames may need to refresh their title bar.
   // TODO(beng): make this pure virtual after XPFrame/VistaFrame retire.
   virtual void UpdateTitleBar() = 0;
 
   // Update any loading animations running in the window. |should_animate| is
   // true if there are tabs loading and the animations should continue, false
   // if there are no active loads and the animations should end.
   virtual void UpdateLoadingAnimations(bool should_animate) = 0;
 
+  // Sets the starred state for the current tab.
+  virtual void SetStarredState(bool is_starred) = 0;
+
   // TODO(beng): RENAME (GetRestoredBounds)
   // Returns the nonmaximized bounds of the frame (even if the frame is
   // currently maximized or minimized) in terms of the screen coordinates.
   virtual gfx::Rect GetNormalBounds() const = 0;
 
   // TODO(beng): REMOVE?
   // Returns true if the frame is maximized (aka zoomed).
   virtual bool IsMaximized() = 0;
 
-  // Returns the star button.
-  virtual ToolbarStarToggle* GetStarButton() const = 0;
-
   // Returns the location bar.
   virtual LocationBarView* GetLocationBarView() const = 0;
 
-  // Returns the go button.
-  virtual GoButton* GetGoButton() const = 0;
+  // Informs the view whether or not a load is in progress for the current tab.
+  // The view can use this notification to update the go/stop button.
+  virtual void UpdateStopGoState(bool is_loading) = 0;
 
   // Updates the toolbar with the state for the specified |contents|.
   virtual void UpdateToolbar(TabContents* contents,
                              bool should_restore_state) = 0;
 
   // Focuses the toolbar (for accessibility).
   virtual void FocusToolbar() = 0;
 
   // Returns whether the bookmark bar is visible or not.
   virtual bool IsBookmarkBarVisible() const = 0;
 
   // Shows or hides the bookmark bar depending on its current visibility.
   virtual void ToggleBookmarkBar() = 0;
 
   // Shows the About Chrome dialog box.
   virtual void ShowAboutChromeDialog() = 0;
 
   // Shows the Bookmark Manager window.
   virtual void ShowBookmarkManager() = 0;
 
+  // Returns true if the Bookmark bubble is visible.
+  virtual bool IsBookmarkBubbleVisible() const = 0;
+
+  // Shows the Bookmark bubble. |url| is the URL being bookmarked,
+  // |already_bookmarked| is true if the url is already bookmarked.
+  virtual void ShowBookmarkBubble(const GURL& url, bool already_bookmarked) = 0;
+
   // Shows the Report a Bug dialog box.
   virtual void ShowReportBugDialog() = 0;
 
   // Shows the Clear Browsing Data dialog box.
   virtual void ShowClearBrowsingDataDialog() = 0;
 
   // Shows the Import Bookmarks & Settings dialog box.
   virtual void ShowImportDialog() = 0;
   
   // Shows the Search Engines dialog box.
@@ skipping 21 matching lines @@
 // UI used only by UI test automation.
 class BrowserWindowTesting {
 public:
 #if defined(OS_WIN)
   // Returns the Bookmark Bar view.
   virtual BookmarkBarView* GetBookmarkBarView() = 0;
 #endif
 };
 
 #endif  // CHROME_BROWSER_BROWSER_WINDOW_H__