Refactor browser window zoom handling and enable zoom icon on all platforms.

chrome/browser/ui/views/location_bar/zoom_view.cc

Index: chrome/browser/ui/views/location_bar/zoom_view.cc
diff --git a/chrome/browser/ui/views/location_bar/zoom_view.cc b/chrome/browser/ui/views/location_bar/zoom_view.cc
index e29c75feb1f90ae9ea8271c53c14b1b3c101d9c5..a89dbf286d62e75e372b74f1f69fb50ab9f2684e 100644
--- a/chrome/browser/ui/views/location_bar/zoom_view.cc
+++ b/chrome/browser/ui/views/location_bar/zoom_view.cc
@@ -15,44 +15,25 @@
 #include "ui/gfx/size.h"
 
 ZoomView::ZoomView(ToolbarModel* toolbar_model)
-    : toolbar_model_(toolbar_model),
-      zoom_icon_state_(ZoomController::NONE),
-      zoom_percent_(100) {
+    : toolbar_model_(toolbar_model) {
   set_accessibility_focusable(true);
+  Update(NULL);
 }
 
 ZoomView::~ZoomView() {
 }
 
-void ZoomView::SetZoomIconState(ZoomController::ZoomIconState zoom_icon_state) {
-  if (zoom_icon_state == zoom_icon_state_)
-    return;
-
-  zoom_icon_state_ = zoom_icon_state;
-  Update();
-}
-
-void ZoomView::SetZoomIconTooltipPercent(int zoom_percent) {
-  if (zoom_percent == zoom_percent_)
-    return;
-
-  zoom_percent_ = zoom_percent;
-  Update();
-}
-
-void ZoomView::Update() {
-  if (toolbar_model_->input_in_progress() ||
-      zoom_icon_state_ == ZoomController::NONE) {
+void ZoomView::Update(ZoomController* zoom_controller) {
+  if (toolbar_model_->input_in_progress() || !zoom_controller) {
     SetVisible(false);
-    ZoomBubbleView::CloseBubble();
-  } else {
-    SetVisible(true);
-    SetTooltipText(
-        l10n_util::GetStringFUTF16Int(IDS_TOOLTIP_ZOOM, zoom_percent_));
-    SetImage(ui::ResourceBundle::GetSharedInstance().GetImageSkiaNamed(
-        zoom_icon_state_ == ZoomController::ZOOM_PLUS_ICON ?
-        IDR_ZOOM_PLUS : IDR_ZOOM_MINUS));
+    return;
   }
+
+  SetTooltipText(
+      l10n_util::GetStringFUTF16Int(IDS_TOOLTIP_ZOOM, zoom_percent_));
+  SetImage(ui::ResourceBundle::GetSharedInstance().GetImageSkiaNamed(
+      zoom_controller->zoom_percent() > 100 ? IDR_ZOOM_PLUS : IDR_ZOOM_MINUS));
+  SetVisible(true);
 }
 
 void ZoomView::GetAccessibleState(ui::AccessibleViewState* state) {
@@ -71,8 +52,8 @@ bool ZoomView::OnMousePressed(const views::MouseEvent& event) {
 }
 
 void ZoomView::OnMouseReleased(const views::MouseEvent& event) {
-  if (event.IsOnlyLeftMouseButton() && HitTest(event.location()))
-    ZoomBubbleView::ShowBubble(this, zoom_percent_, false);
+  //if (event.IsOnlyLeftMouseButton() && HitTest(event.location()))
+    //ZoomBubbleView::ShowBubble(this, zoom_percent_, false);
 }
 
 bool ZoomView::OnKeyPressed(const views::KeyEvent& event) {
@@ -80,6 +61,6 @@ bool ZoomView::OnKeyPressed(const views::KeyEvent& event) {
       event.key_code() != ui::VKEY_RETURN)
     return false;
 
-  ZoomBubbleView::ShowBubble(this, zoom_percent_, false);
+  //ZoomBubbleView::ShowBubble(this, zoom_percent_, false);
   return true;
 }