[Media Router] Parse device description xml in utility process

chrome/browser/media/router/discovery/dial/safe_dial_device_description_parser.cc

Index: chrome/browser/media/router/discovery/dial/safe_dial_device_description_parser.cc
diff --git a/chrome/browser/media/router/discovery/dial/safe_dial_device_description_parser.cc b/chrome/browser/media/router/discovery/dial/safe_dial_device_description_parser.cc
new file mode 100644
index 0000000000000000000000000000000000000000..d910fd1e03dc4716ee09cbddadbc826ee095dc44
--- /dev/null
+++ b/chrome/browser/media/router/discovery/dial/safe_dial_device_description_parser.cc
@@ -0,0 +1,71 @@
+// Copyright 2017 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/media/router/discovery/dial/safe_dial_device_description_parser.h"
+
+#include <utility>
+
+#include "chrome/grit/generated_resources.h"
+#include "content/public/browser/browser_thread.h"
+#include "ui/base/l10n/l10n_util.h"
+
+namespace media_router {
+
+SafeDialDeviceDescriptionParser::SafeDialDeviceDescriptionParser() {}
+
+SafeDialDeviceDescriptionParser::~SafeDialDeviceDescriptionParser() {}
+
+void SafeDialDeviceDescriptionParser::Start(
+    const std::string& xml_text,
+    const DeviceDescriptionCallback& callback) {
+  DCHECK(thread_checker_.CalledOnValidThread());
+  DCHECK(!utility_process_mojo_client_);
+  DCHECK(callback);
+
+  device_description_callback_ = callback;
+
+  utility_process_mojo_client_ =
+      base::MakeUnique<content::UtilityProcessMojoClient<
+          chrome::mojom::DialDeviceDescriptionParser>>(
+          l10n_util::GetStringUTF16(
+              IDS_UTILITY_PROCESS_DIAL_DEVICE_DESCRIPTION_PARSER_NAME));
+
+  utility_process_mojo_client_->set_error_callback(base::Bind(
+      &SafeDialDeviceDescriptionParser::OnParseDeviceDescriptionFailed,
+      base::Unretained(this)));
+
+  // This starts utility process in the background.
+  utility_process_mojo_client_->Start();
+
+  // This call is queued up until the Mojo message pipe has been established to
+  // the service running in the utility process.
+  utility_process_mojo_client_->service()->ParseDialDeviceDescription(
+      xml_text,
+      base::Bind(
+          &SafeDialDeviceDescriptionParser::OnParseDeviceDescriptionComplete,
+          base::Unretained(this)));
+}
+
+void SafeDialDeviceDescriptionParser::OnParseDeviceDescriptionComplete(
+    bool result,
+    chrome::mojom::DialDeviceDescriptionPtr device_description) {
+  DCHECK(thread_checker_.CalledOnValidThread());
+
+  utility_process_mojo_client_.reset();  // Terminate the utility process.
+
+  DCHECK(device_description_callback_);
+  device_description_callback_.Run(result, std::move(device_description));
+}
+
+void SafeDialDeviceDescriptionParser::OnParseDeviceDescriptionFailed() {
+  DCHECK(thread_checker_.CalledOnValidThread());
+
+  utility_process_mojo_client_.reset();  // Terminate the utility process.
+
+  DCHECK(device_description_callback_);
+  device_description_callback_.Run(false,
+                                   chrome::mojom::DialDeviceDescriptionPtr());

[dcheng, 2017/03/25 00:59:38]

Note: it's legal to just pass nullptr here

[zhaobin, 2017/03/27 19:16:55]

Done.

+}
+
+}  // namespace media_router