Tracks changes to zoom icon and zoom percentage. Notifies browser window if the icon should be chan…

chrome/browser/ui/browser.h

Index: chrome/browser/ui/browser.h
diff --git a/chrome/browser/ui/browser.h b/chrome/browser/ui/browser.h
index 67f431fcdc6b4585d38875b167585513f0d5b647..78c4b71bad99dec78f95a4a0c6627bb15b9346cb 100644
--- a/chrome/browser/ui/browser.h
+++ b/chrome/browser/ui/browser.h
@@ -40,6 +40,7 @@
 #include "chrome/browser/ui/tabs/tab_strip_model_observer.h"
 #include "chrome/browser/ui/toolbar/toolbar_model.h"
 #include "chrome/browser/ui/webui/sync_promo/sync_promo_ui.h"
+#include "chrome/browser/ui/zoom/zoom_observer.h"
 #include "chrome/common/content_settings.h"
 #include "chrome/common/content_settings_types.h"
 #include "chrome/common/extensions/extension_constants.h"
@@ -94,6 +95,7 @@ class Browser : public TabStripModelDelegate,
                 public ConstrainedWindowTabHelperDelegate,
                 public BlockedContentTabHelperDelegate,
                 public BookmarkTabHelperDelegate,
+                public ZoomObserver,

[tfarina, 2012/06/01 00:47:53]

To be honest I think ZoomTabHelper and ZoomTabHelperDelegate is a more much
consistent naming scheme given the above. Ben, thoughts? I know you asked him to
rename, I wonder why ;)

[Kyle Horimoto, 2012/06/01 18:04:50]

I'd also be interested to hear the reasoning. Ben?

                 public ExtensionTabHelperDelegate,
                 public content::PageNavigator,
                 public CommandUpdater::CommandUpdaterDelegate,
@@ -1014,6 +1016,12 @@ class Browser : public TabStripModelDelegate,
   virtual void URLStarredChanged(TabContentsWrapper* source,
                                  bool starred) OVERRIDE;
 
+  // Overridden from ZoomObserver:
+  virtual void OnZoomIconChanged(TabContentsWrapper* source,
+                                 ZoomController::ZoomIconState state) OVERRIDE;
+  virtual void OnZoomChanged(TabContentsWrapper* source,
+                             int zoom_percent) OVERRIDE;
+
   // Overridden from ExtensionTabHelperDelegate:
   virtual void OnDidGetApplicationInfo(TabContentsWrapper* source,
                                        int32 page_id) OVERRIDE;