Stub for WebRTCDeviceProvider

chrome/browser/devtools/device/webrtc/webrtc_device_provider.cc

+// Copyright 2014 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/devtools/device/webrtc/webrtc_device_provider.h"
+
+#include "chrome/browser/ui/browser.h"
+#include "chrome/common/url_constants.h"
+#include "content/public/browser/web_contents.h"
+#include "content/public/browser/web_ui_data_source.h"
+#include "content/public/browser/web_ui_message_handler.h"
+#include "grit/webrtc_device_provider_resources.h"
+#include "grit/webrtc_device_provider_resources_map.h"
+#include "ui/base/page_transition_types.h"
+
+class WebRTCDeviceProvider::MessageHandler

[dgozman, 2014/11/27 13:45:27]

This class needs a pointer to device provider to delegate stuff.

[SeRya, 2014/11/27 14:15:08]

Sure. Will be added on first use.

+    : public content::WebUIMessageHandler {
+ public:
+  void RegisterMessages() override;
+
+ private:
+  void HandleLoaded(const base::ListValue* args);
+};
+
+// WebRTCDeviceProvider::WebUI
+WebRTCDeviceProvider::WebUI::WebUI(content::WebUI* web_ui)
+    : content::WebUIController(web_ui),
+      handler_(new MessageHandler) {
+  Profile* profile = Profile::FromWebUI(web_ui);
+
+  content::WebUIDataSource* source = content::WebUIDataSource::Create(
+      chrome::kChromeUIWebRTCDeviceProviderHost);
+
+  for (size_t i = 0; i < kWebrtcDeviceProviderResourcesSize; i++) {
+    source->AddResourcePath(kWebrtcDeviceProviderResources[i].name,
+                            kWebrtcDeviceProviderResources[i].value);
+  }
+  source->SetDefaultResource(IDR_BACKGROUND_WORKER_HTML);
+
+  web_ui->AddMessageHandler(handler_.get());
+
+  content::WebUIDataSource::Add(profile, source);
+}
+
+WebRTCDeviceProvider::WebUI::~WebUI() {}
+
+// WebRTCDeviceProvider::MessageHandler
+
+void WebRTCDeviceProvider::MessageHandler::RegisterMessages() {
+  web_ui()->RegisterMessageCallback(
+      "loaded",
+      base::Bind(&MessageHandler::HandleLoaded, base::Unretained(this)));
+}
+
+void WebRTCDeviceProvider::MessageHandler::HandleLoaded(
+    const base::ListValue* args) {
+}
+
+// WebRTCDeviceProvider
+
+WebRTCDeviceProvider::WebRTCDeviceProvider(content::BrowserContext* context) {
+  background_worker_.reset(content::WebContents::Create(
+      content::WebContents::CreateParams(context)));
+
+  // TODO(serya): Make sure background_worker_ destructed before profile.
+
+  background_worker_->GetController().LoadURL(
+      GURL(chrome::kChromeUIWebRTCDeviceProviderURL),

[dgozman, 2014/11/27 13:45:27]

I think the |SetDefaultResource()| call allows you to just use the host here and
remove the URL constant.

[SeRya, 2014/11/27 14:15:08]

Let's remove SetDefaultResource with webrtc_device_provider_resources.h instead.

[SeRya, 2014/11/27 14:15:08]

Lets remove SetDefaultResource. This URL is not intended to be typed, it removes
one generated file.

+      content::Referrer(),
+      ui::PAGE_TRANSITION_AUTO_TOPLEVEL,
+      std::string());
+
+  handler_ = static_cast<WebUI*>(

[dgozman, 2014/11/27 13:45:27]

Why don't you just AddMessageHandler from here instead of WebUIController?

[SeRya, 2014/11/27 14:15:08]

Problem is the following. If I manually type the URL (for testing purpose)
chrome fails because can't find a handler for 'loaded' message (because in this
way handler not set). It is not a problem (because page loaded manually useless)
but may cause confusion.

+      background_worker_->GetWebUI()->GetController())->handler_.get();
+}
+
+WebRTCDeviceProvider::~WebRTCDeviceProvider() {}
+
+void WebRTCDeviceProvider::QueryDevices(const SerialsCallback& callback) {
+  // TODO(serya): Implement
+}
+
+void WebRTCDeviceProvider::QueryDeviceInfo(const std::string& serial,
+                                          const DeviceInfoCallback& callback) {
+  // TODO(serya): Implement
+}
+
+void WebRTCDeviceProvider::OpenSocket(const std::string& serial,
+                                     const std::string& socket_name,
+                                     const SocketCallback& callback) {
+  // TODO(serya): Implement
+}