WebUsb Android chooser UI

chrome/browser/android/usb/web_usb_chooser_service_android.cc

+// Copyright 2016 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/android/usb/web_usb_chooser_service_android.h"
+
+#include <utility>
+
+#include "chrome/browser/ui/android/usb_chooser_android.h"
+#include "content/public/browser/browser_thread.h"
+
+WebUsbChooserServiceAndroid::WebUsbChooserServiceAndroid(
+    content::RenderFrameHost* render_frame_host)
+    : render_frame_host_(render_frame_host) {
+  DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
+  DCHECK(render_frame_host_);
+}
+
+WebUsbChooserServiceAndroid::~WebUsbChooserServiceAndroid() {}
+
+void WebUsbChooserServiceAndroid::GetPermission(
+    mojo::Array<device::usb::DeviceFilterPtr> device_filters,
+    const GetPermissionCallback& callback) {
+  usb_chooser_android_.reset(new UsbChooserAndroid(

[Reilly Grant (use Gerrit), 2016/03/04 20:53:39]

Never mind. This is already explicitly owned here so being a WebContentsObserver
is unnecessary. However, is only supporting a single call to GetPermission at
once what we want?

[juncai, 2016/03/05 00:18:24]

Now WebUsbChooserServiceAndroid has a std::vector<scoped_ptr<UsbChooserAndroid>>
instead of scoped_ptr<UsbChooserAndroid> to allow multiple calls to
GetPermission.

Done.

+      std::move(device_filters), render_frame_host_, callback));
+}
+
+void WebUsbChooserServiceAndroid::Bind(
+    mojo::InterfaceRequest<device::usb::ChooserService> request) {
+  DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
+  bindings_.AddBinding(this, std::move(request));
+}