[Media Router] Parse device description xml in utility process

chrome/utility/extensions/extensions_handler.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/utility/extensions/extensions_handler.h"
 
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/path_service.h"
 #include "build/build_config.h"
 #include "chrome/common/chrome_utility_messages.h"
 #include "chrome/common/extensions/chrome_extensions_client.h"
 #include "chrome/common/extensions/chrome_utility_extensions_messages.h"
+#include "chrome/common/extensions/dial_device_description_parser.mojom.h"
 #include "chrome/common/extensions/media_parser.mojom.h"
 #include "chrome/common/extensions/removable_storage_writer.mojom.h"
 #include "chrome/common/media_galleries/metadata_types.h"
 #include "chrome/utility/chrome_content_utility_client.h"
 #include "chrome/utility/image_writer/image_writer_handler.h"
 #include "chrome/utility/media_galleries/ipc_data_source.h"
 #include "chrome/utility/media_galleries/media_metadata_parser.h"
+#include "chrome/utility/media_router/dial_device_description_parser.h"
 #include "content/public/common/content_paths.h"
 #include "content/public/utility/utility_thread.h"
 #include "extensions/common/extension.h"
 #include "extensions/common/extension_utility_messages.h"
 #include "extensions/utility/unpacker.h"
 #include "media/base/media.h"
 #include "mojo/public/cpp/bindings/strong_binding.h"
 #include "services/service_manager/public/cpp/interface_registry.h"
 #include "ui/base/ui_base_switches.h"
 
@@ skipping 128 matching lines @@
     if (!success)
       key_data.clear();
 
     callback.Run(success, key_data);
   }
 
   DISALLOW_COPY_AND_ASSIGN(WiFiCredentialsGetterImpl);
 };
 #endif  // defined(OS_WIN)
 
+#if defined(ENABLE_MEDIA_ROUTER)
+class DialDeviceDescriptionParserImpl
+    : public extensions::mojom::DialDeviceDescriptionParser {
+ public:
+  DialDeviceDescriptionParserImpl() = default;
+  ~DialDeviceDescriptionParserImpl() override = default;
+
+  static void Create(
+      extensions::mojom::DialDeviceDescriptionParserRequest request) {
+    mojo::MakeStrongBinding(base::MakeUnique<DialDeviceDescriptionParserImpl>(),
+                            std::move(request));
+  }
+
+ private:
+  // extensions::mojom::DialDeviceDescriptionParser:
+  void ParseDialDeviceDescription(
+      const std::string& device_description_xml_data,
+      const ParseDialDeviceDescriptionCallback& callback) override {
+    DCHECK(!callback.is_null());
+
+    media_router::DialDeviceDescriptionParser parser;
+    bool result = parser.Parse(device_description_xml_data);
+    std::string logging_xml =
+        parser.ScrubXmlForLogging(device_description_xml_data);
+
+    callback.Run(result, parser.device_description(), logging_xml);
+  }
+
+  DISALLOW_COPY_AND_ASSIGN(DialDeviceDescriptionParserImpl);
+};
+#endif  // defined(ENABLE_MEDIA_ROUTER)
 }  // namespace
 
 namespace extensions {
 
 ExtensionsHandler::ExtensionsHandler() {
   ExtensionsClient::Set(ChromeExtensionsClient::GetInstance());
 }
 
 ExtensionsHandler::~ExtensionsHandler() {
 }
@@ skipping 15 matching lines @@
     registry->AddInterface(base::Bind(&RemovableStorageWriterImpl::Create));
     registry->AddInterface(base::Bind(&WiFiCredentialsGetterImpl::Create));
 #endif
     return;
   }
 
   registry->AddInterface(base::Bind(&MediaParserImpl::Create));
 #if !defined(OS_WIN)
   registry->AddInterface(base::Bind(&RemovableStorageWriterImpl::Create));
 #endif
+#if defined(ENABLE_MEDIA_ROUTER)
+  registry->AddInterface(base::Bind(&DialDeviceDescriptionParserImpl::Create));

[mark a. foltz, 2017/03/17 19:05:47]

It isn't clear that this needs to be created from extension processes - from a
quick glance at the design doc, it looks like it will be created by
MediaRouterMojoImpl (or a DialSinkDiscovery class owned by it), so only from the
browser process.

[zhaobin, 2017/03/18 00:17:39]

Done.

+#endif  // defined(ENABLE_MEDIA_ROUTER)
 }
 
 bool ExtensionsHandler::OnMessageReceived(const IPC::Message& message) {
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP(ExtensionsHandler, message)
 #if defined(OS_WIN)
     IPC_MESSAGE_HANDLER(ChromeUtilityMsg_ParseITunesPrefXml,
                         OnParseITunesPrefXml)
 #endif  // defined(OS_WIN)
 
@@ skipping 66 matching lines @@
   picasa::PicasaAlbumsIndexer indexer(album_uids);
   indexer.ParseFolderINI(folders_inis);
   content::UtilityThread::Get()->Send(
       new ChromeUtilityHostMsg_IndexPicasaAlbumsContents_Finished(
           indexer.albums_images()));
   content::UtilityThread::Get()->ReleaseProcessIfNeeded();
 }
 #endif  // defined(OS_WIN) || defined(OS_MACOSX)
 
 }  // namespace extensions