[Mac] Implement the basic zoom bubble.

chrome/browser/ui/cocoa/location_bar/zoom_decoration.h

 // Copyright (c) 2012 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_UI_COCOA_LOCATION_BAR_ZOOM_DECORATION_H_
 #define CHROME_BROWSER_UI_COCOA_LOCATION_BAR_ZOOM_DECORATION_H_
 
 #import <Cocoa/Cocoa.h>
 
 #include "base/basictypes.h"
 #include "chrome/browser/ui/cocoa/location_bar/image_decoration.h"
 
-class ToolbarModel;
+class LocationBarViewMac;
+@class ZoomBubbleController;
 class ZoomController;
 
 // Zoom icon at the end of the omnibox (close to page actions) when at a
 // non-standard zoom level.
-// TODO(dbeam): add mouse event handlers to show a zoom bubble on click.
 class ZoomDecoration : public ImageDecoration {
  public:
-  explicit ZoomDecoration(ToolbarModel* toolbar_model);
+  explicit ZoomDecoration(LocationBarViewMac* owner);
   virtual ~ZoomDecoration();
 
   // Called when this decoration should show or hide itself in its most current
   // state.
   void Update(ZoomController* zoom_controller);
 
  private:
+  NSPoint GetBubblePointInFrame(NSRect frame);
+
   // LocationBarDecoration implementation.
   virtual bool AcceptsMousePress() OVERRIDE;
+  virtual bool OnMousePressed(NSRect frame) OVERRIDE;
   virtual NSString* GetToolTip() OVERRIDE;
 
-  // A reference to the toolbar model to query whether input is currently
-  // happening in the location bar. Not owned by this class.
-  ToolbarModel* toolbar_model_;
+  // The control that owns this. Weak.
+  LocationBarViewMac* owner_;
+
+  // The bubble that this decoration shows. Weak, owns self.

[sail, 2013/02/15 21:05:12]

In a different review thakis recommended using a scoped_nsobject for this and
having the bubble controller retain itself.

[Robert Sesek, 2013/02/15 21:39:20]

I wrote it like that earlier but the closeObserver actually makes it nice to
keep the same pattern as the other bubbles as well as making it testable.

[sail, 2013/02/15 21:58:20]

It should still be easy to test. In closeObserver you just do bubble_reset()
instead of bubble = nil;

+  ZoomBubbleController* bubble_;
 
   // The string to show for a tooltip.
   scoped_nsobject<NSString> tooltip_;
 
   DISALLOW_COPY_AND_ASSIGN(ZoomDecoration);
 };
 
 #endif  // CHROME_BROWSER_UI_COCOA_LOCATION_BAR_ZOOM_DECORATION_H_