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_map.h"
+#include "ui/base/page_transition_types.h"
+
+const char kBackgroundWorkerURL[] =
+    "chrome://webrtc-device-provider/background_worker.html";
+
+class WebRTCDeviceProvider::MessageHandler
+    : public content::WebUIMessageHandler {
+ public:
+

[dgozman, 2014/11/28 12:40:34]

nit: extra empty line.

[SeRya, 2014/11/28 12:53:04]

Done.

+  void RegisterMessages() override;
+
+ protected:
+  virtual void OnLoaded() {}
+
+ private:
+  void HandleLoaded(const base::ListValue* args);

[dgozman, 2014/11/28 12:40:34]

If you just add a check |if (!owner_) return;| to all handlers, you will not
need 2 methods per each message and 2 classes.

[SeRya, 2014/11/28 12:53:04]

Done.

+};
+
+class WebRTCDeviceProvider::MessageHandlerImpl : public MessageHandler {
+ public:
+  explicit MessageHandlerImpl(WebRTCDeviceProvider* owner);
+  ~MessageHandlerImpl() override;
+
+ protected:
+  void OnLoaded() override {}
+
+ private:
+  WebRTCDeviceProvider* const owner_;
+};
+
+// WebRTCDeviceProvider::WebUI
+WebRTCDeviceProvider::WebUI::WebUI(content::WebUI* web_ui)
+    : content::WebUIController(web_ui) {
+  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);
+  }
+
+  // Sets a stub message handler. If web contents load from WebRTCDeviceProvider

[dgozman, 2014/11/28 12:40:34]

nit: load from -> was created by

[SeRya, 2014/11/28 12:53:04]

Done.

+  // message callbacks will be overridden by a real implementation.
+  web_ui->AddMessageHandler(new MessageHandler);
+
+  content::WebUIDataSource::Add(profile, source);
+}
+
+WebRTCDeviceProvider::WebUI::~WebUI() {}

[dgozman, 2014/11/28 12:40:34]

nit: } on a separate line

[SeRya, 2014/11/28 12:53:04]

Done.

+
+// 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) {
+  OnLoaded();
+}
+
+// WebRTCDeviceProvider::MessageHandlerImpl
+WebRTCDeviceProvider::MessageHandlerImpl::MessageHandlerImpl(
+    WebRTCDeviceProvider* owner) : owner_(owner) {
+  DCHECK(owner_->handler_ == NULL);
+  owner_->handler_ = this;

[dgozman, 2014/11/28 12:40:34]

This is unnecessary.

[SeRya, 2014/11/28 12:53:04]

Done.

+}
+
+WebRTCDeviceProvider::MessageHandlerImpl::~MessageHandlerImpl() {
+  DCHECK(owner_->handler_ == this);
+  owner_->handler_ = NULL;
+}
+
+// WebRTCDeviceProvider
+
+WebRTCDeviceProvider::WebRTCDeviceProvider(content::BrowserContext* context)
+    : handler_(NULL) {
+  background_worker_.reset(content::WebContents::Create(
+      content::WebContents::CreateParams(context)));
+
+  // TODO(serya): Make sure background_worker_ destructed before profile.
+  GURL url(kBackgroundWorkerURL);
+
+  DCHECK_EQ(chrome::kChromeUIWebRTCDeviceProviderHost, url.host());

[dgozman, 2014/11/28 12:40:34]

Nice one!

+
+  background_worker_->GetController().LoadURL(
+      url,
+      content::Referrer(),
+      ui::PAGE_TRANSITION_AUTO_TOPLEVEL,
+      std::string());
+
+  background_worker_->GetWebUI()->AddMessageHandler(
+      new MessageHandlerImpl(this));
+}
+
+WebRTCDeviceProvider::~WebRTCDeviceProvider() {
+  background_worker_.reset();
+  DCHECK(!handler_);
+}
+
+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
+}