Implement a bluetooth picker dialog for Android

chrome/browser/android/chrome_web_contents_delegate_android.h

Index: chrome/browser/android/chrome_web_contents_delegate_android.h
diff --git a/chrome/browser/android/chrome_web_contents_delegate_android.h b/chrome/browser/android/chrome_web_contents_delegate_android.h
index da47f2544030277ee0b0df84e4406f3ce70d3cdf..4dcbaaf49c9009021f939a8031f9c8e5fd0d0602 100644
--- a/chrome/browser/android/chrome_web_contents_delegate_android.h
+++ b/chrome/browser/android/chrome_web_contents_delegate_android.h
@@ -9,6 +9,7 @@
 
 #include "base/files/file_path.h"
 #include "components/web_contents_delegate_android/web_contents_delegate_android.h"
+#include "content/public/browser/bluetooth_chooser.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 
@@ -41,6 +42,10 @@ class ChromeWebContentsDelegateAndroid
                            bool to_different_document) override;
   void RunFileChooser(content::WebContents* web_contents,
                       const content::FileChooserParams& params) override;
+  scoped_ptr<content::BluetoothChooser> RunBluetoothChooser(

[newt (away), 2015/09/01 21:51:23]

does this method belong in this class?

[Finnur, 2015/09/02 16:01:43]

I struggled with this quite a bit, and believe it belongs here.

content/browser/bluetooth/bluetooth_dispatcher_host.cc calls
WebContentsDelegate::RunBluetoothChooser, which is what I'm overriding here (in
chrome/browser/chrome_web_contents_delegate_android.h). This layer
(chrome/browser) is needed because my override function needs to instantiate
BluetoothChooserAndroid, which depends on chrome/browser.

If there's a better way, I'm willing to re-consider, but I bounced this off a
couple of people already and this was what was settled on.

[newt (away), 2015/09/02 17:48:42]

Ok, if you're overriding a method in web_contents_delegate.h, then this makes
sense. Thanks for explaining.

+      content::WebContents* web_contents,
+      const content::BluetoothChooser::EventHandler& event_handler,
+      const GURL& origin) override;
   void CloseContents(content::WebContents* web_contents) override;
   blink::WebDisplayMode GetDisplayMode(
       const content::WebContents* web_contents) const override;