Make web intents picker work as constrained dialog instead of InfoBubble

chrome/browser/ui/cocoa/web_intent_picker_cocoa.h

Index: chrome/browser/ui/cocoa/web_intent_picker_cocoa.h
diff --git a/chrome/browser/ui/cocoa/web_intent_picker_cocoa.h b/chrome/browser/ui/cocoa/web_intent_picker_cocoa.h
index 1264ad31b318aade1a66731bb3e8ffde6f86abb7..b5a667783b13295953d4cd2ecd011564463b8e73 100644
--- a/chrome/browser/ui/cocoa/web_intent_picker_cocoa.h
+++ b/chrome/browser/ui/cocoa/web_intent_picker_cocoa.h
@@ -14,6 +14,7 @@
 #include "chrome/browser/ui/intents/web_intent_picker_model.h"
 #include "chrome/browser/ui/intents/web_intent_picker_model_observer.h"
 
+class ConstrainedWindow;
 class TabContentsWrapper;
 @class WebIntentBubbleController;
 class WebIntentInlineDispositionDelegate;
@@ -22,20 +23,19 @@ class WebIntentInlineDispositionDelegate;
 class WebIntentPickerCocoa : public WebIntentPicker,
                              public WebIntentPickerModelObserver {
  public:
-  // |browser| and |delegate| cannot be NULL.
-  // |wrapper| is unused.
+  // |browser|, |wrapper|, and |delegate| cannot be NULL.
   WebIntentPickerCocoa(Browser* browser,
                        TabContentsWrapper* wrapper,
                        WebIntentPickerDelegate* delegate,
                        WebIntentPickerModel* model);
   virtual ~WebIntentPickerCocoa();
 
+  void OnSheetDidEnd(NSWindow* sheet);
+
   // WebIntentPickerDelegate forwarding API.
   void OnCancelled();
   void OnServiceChosen(size_t index);
 
-  void set_controller(WebIntentBubbleController* controller);
-
   // WebIntentPicker:
   virtual void Close() OVERRIDE;
 
@@ -48,6 +48,9 @@ class WebIntentPickerCocoa : public WebIntentPicker,
   virtual void OnInlineDisposition(WebIntentPickerModel* model) OVERRIDE;
 
  private:
+
+  ConstrainedWindow* window_;  // Window for constrained sheet. Weak reference.
+
   // Weak pointer to the |delegate_| to notify about user choice/cancellation.
   WebIntentPickerDelegate* delegate_;
 
@@ -56,10 +59,7 @@ class WebIntentPickerCocoa : public WebIntentPicker,
 
   Browser* browser_;  // The browser we're in. Weak Reference.
 
-  WebIntentBubbleController* controller_;  // Weak reference.
-
-  // Factory for weak ptrs, used for delayed callbacks.
-  base::WeakPtrFactory<WebIntentPickerCocoa> weak_ptr_factory_;
+  WebIntentBubbleController* sheet_controller_;  // Weak reference.
 
   // Tab contents wrapper to hold intent page if inline disposition is used.
   scoped_ptr<TabContentsWrapper> inline_disposition_tab_contents_;
@@ -70,9 +70,6 @@ class WebIntentPickerCocoa : public WebIntentPicker,
   // Indicate that we invoked a service, instead of just closing/cancelling.
   bool service_invoked;
 
-  // Post a delayed task to do layout, if it isn't already pending.
-  void PerformDelayedLayout();
-
   // Re-layout the intent picker.
   void PerformLayout();