Add native file picker impl using SelectFileDialog.

chrome/browser/ui/intents/web_intent_picker_controller.cc

 // 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.
 
 #include "chrome/browser/ui/intents/web_intent_picker_controller.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 117 matching lines @@
   virtual void WebContentsDestroyed(content::WebContents* web_contents) {
     if (controller_)
       controller_->SourceWebContentsDestroyed(web_contents);
     delete this;
   }
 
  private:
   base::WeakPtr<WebIntentPickerController> controller_;
 };
 
+// A WebIntentsDispatcher wrapper that deletes a itself,
+// and a service once a response is received.

[Greg Billock, 2012/10/10 17:12:51]

I think we can just do this with a closure using RegisterReplyNotification on
the original dispatcher, right?

dispatcher.RegisterReplyNotification(&DeleteService, service);

void DeleteService(IntentServiceHost* service) {
  delete service;
}

[Steve McKay, 2012/10/10 23:12:57]

Done.

+class DeletingDispatcherWrapper : public content::WebIntentsDispatcher {
+ public:
+  explicit DeletingDispatcherWrapper(
+      web_intents::IntentServiceHost* service,
+      content::WebIntentsDispatcher* delegate)
+      : service_(service),
+        delegate_(delegate) {}
+
+  virtual const webkit_glue::WebIntentData& GetIntent() OVERRIDE {
+    return delegate_->GetIntent();
+  }
+
+  virtual void DispatchIntent(content::WebContents* web_contents) OVERRIDE {
+    NOTREACHED();
+  }
+  virtual void ResetDispatch() OVERRIDE {
+    NOTREACHED();
+  }
+
+  virtual void SendReplyMessage(
+      webkit_glue::WebIntentReplyType reply_type,
+      const string16& data) OVERRIDE {
+    delegate_->SendReplyMessage(reply_type, data);
+  }
+
+  virtual void RegisterReplyNotification(
+      const base::Callback<void(webkit_glue::WebIntentReplyType)>&) OVERRIDE {
+    NOTREACHED();
+  }
+
+ private:
+  ~DeletingDispatcherWrapper() {
+    delete service_;
+    delete this;
+  }
+
+  web_intents::IntentServiceHost* service_;
+  content::WebIntentsDispatcher* delegate_;
+};
+
 }  // namespace
 
 // UMAReporter handles reporting Web Intents events to UMA.
 class WebIntentPickerController::UMAReporter {
  public:
 
   // Resets the service active duration timer to "now".
   void ResetServiceActiveTimer();
 
   // Records the duration of time spent using the service. Uses |reply_type|
@@ skipping 207 matching lines @@
 
   // TODO(smckay): This entire method shold basically be pulled out
   // into a separate class dedicated to the execution of intents.
   // The tricky part is with the "INLINE" disposition where we
   // want to (re)use the picker to handle the intent. A bit of
   // artful composition + lazy instantiation should make that possible.
   switch (disposition) {
     case webkit_glue::WebIntentServiceData::DISPOSITION_NATIVE: {
       web_intents::IntentServiceHost* service =
           native_services_->CreateServiceInstance(
-              url, intents_dispatcher_->GetIntent());
+              url, intents_dispatcher_->GetIntent(), web_contents_);
       DCHECK(service);
-      service->HandleIntent(intents_dispatcher_);
+      service->HandleIntent(
+          new DeletingDispatcherWrapper(service, intents_dispatcher_));
       break;
     }
 
     case webkit_glue::WebIntentServiceData::DISPOSITION_INLINE: {
       // Set the model to inline disposition. It will notify the picker which
       // will respond (via OnInlineDispositionWebContentsCreated) with the
       // WebContents to dispatch the intent to.
       picker_model_->SetInlineDisposition(url);
       break;
     }
@@ skipping 742 matching lines @@
   content::DownloadManager* download_manager =
       content::BrowserContext::GetDownloadManager(profile);
   if (!download_manager)
     return;
   content::DownloadItem* item =
       download_manager->GetDownload(download_id_.local());
   if (item)
     item->Cancel(true);
   download_id_ = content::DownloadId();
 }