bluetooth: Add URLs for Web Bluetooth help links.

chrome/browser/ui/android/bluetooth_chooser_android.h

 // Copyright 2015 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_ANDROID_BLUETOOTH_CHOOSER_ANDROID_H_
 #define CHROME_BROWSER_UI_ANDROID_BLUETOOTH_CHOOSER_ANDROID_H_
 
 #include "base/android/scoped_java_ref.h"
 #include "content/public/browser/bluetooth_chooser.h"
 #include "content/public/browser/web_contents.h"
 
 namespace url {
 class Origin;
 }
 
 // Represents a way to ask the user to select a Bluetooth device from a list of
 // options.
 class BluetoothChooserAndroid : public content::BluetoothChooser {
  public:
   BluetoothChooserAndroid(content::RenderFrameHost* frame,

[Jeffrey Yasskin, 2016/02/18 02:17:24]

Could you add a comment here that the event_handler and frame must outlive the
BluetoothChooserAndroid? It's guaranteed by the requirements on
WebContentsDelegate::RunBluetoothChooser, and it's nice to have a tight chain of
lifetime requirements like this.

[ortuno, 2016/02/18 18:26:33]

Done.

                           const EventHandler& event_handler);
   ~BluetoothChooserAndroid() override;
 
   // content::BluetoothChooser:
   bool CanAskForScanningPermission() override;
   void SetAdapterPresence(AdapterPresence presence) override;
   void ShowDiscoveryState(DiscoveryState state) override;
   void AddDevice(const std::string& device_id,
                  const base::string16& device_name) override;
   void RemoveDevice(const std::string& device_id) override;
 
   // Report the dialog's result.
   void OnDialogFinished(JNIEnv* env,
                         const base::android::JavaParamRef<jobject>& obj,
                         jint event_type,
                         const base::android::JavaParamRef<jstring>& device_id);
 
   // Notify bluetooth stack that the search needs to be re-issued.
   void RestartSearch(JNIEnv* env,
                      const base::android::JavaParamRef<jobject>& obj);
 
   void ShowBluetoothOverviewLink(
       JNIEnv* env,
       const base::android::JavaParamRef<jobject>& obj);
-  void ShowBluetoothPairingLink(
-      JNIEnv* env,
-      const base::android::JavaParamRef<jobject>& obj);
   void ShowBluetoothAdapterOffLink(
       JNIEnv* env,
       const base::android::JavaParamRef<jobject>& obj);
   void ShowNeedLocationPermissionLink(
       JNIEnv* env,
       const base::android::JavaParamRef<jobject>& obj);
 
   static bool Register(JNIEnv* env);
 
  private:
+  void OpenURL(const char url[]);

[Jeffrey Yasskin, 2016/02/18 02:17:24]

Use "const char* url". The behavior of [] in argument lists is equivalent but
confuses everyone.

[ortuno, 2016/02/18 18:26:33]

Done.

   base::android::ScopedJavaGlobalRef<jobject> java_dialog_;
 
+  content::WebContents* web_contents_;
   BluetoothChooser::EventHandler event_handler_;
 };
 
 #endif  // CHROME_BROWSER_UI_ANDROID_BLUETOOTH_CHOOSER_ANDROID_H_