Implementation of chrome://cast page.

chrome/browser/ui/webui/cast/cast_ui.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/ui/webui/cast/cast_ui.h"
+
+#include "chrome/browser/media/router/media_router_factory.h"
+#include "chrome/browser/media/router/mojo/media_router_mojo_impl.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/common/url_constants.h"
+#include "content/public/browser/web_contents.h"
+#include "content/public/browser/web_ui.h"
+#include "content/public/browser/web_ui_data_source.h"
+#include "grit/browser_resources.h"
+#include "grit/generated_resources.h"
+
+namespace {
+const char kRequestExtensionId[] = "requestExtensionId";
+const char kSetExtensionId[] = "setExtensionId";
+}
+
+CastUI::CastUI(content::WebUI* web_ui)
+    : content::WebUIController(web_ui) {
+  // Retrieve the ID of the component extension.
+  auto wc = web_ui->GetWebContents();

[Bernhard Bauer, 2016/05/19 14:34:07]

Use the full type. Also, don't use abbreviations in variable names.

[sheretov, 2016/05/19 16:16:45]

Removed this temporary variable altogether.  Still using auto typing for
|router|, since it's assigned a result of a static cast where the type is
explicitly specified.

+  DCHECK(wc);

[Bernhard Bauer, 2016/05/19 14:34:07]

If this is null, you'll crash below in any case. (Also, the fact that you are
dereferencing |wc| serves as documentation that you don't expect it to be null,
so that aspect doesn't apply either).

[sheretov, 2016/05/19 16:16:45]

Done.

+  // TODO(crbug.com/597778): remove reference to MediaRouterMojoImpl.
+  auto router = static_cast<media_router::MediaRouterMojoImpl*>(
+      media_router::MediaRouterFactory::GetApiForBrowserContext(
+          wc->GetBrowserContext()));
+  DCHECK(router);

[Bernhard Bauer, 2016/05/19 14:34:07]

Same here.

[sheretov, 2016/05/19 16:16:45]

Done.

+  component_extension_id_ = router->media_route_provider_extension_id();

[Bernhard Bauer, 2016/05/19 14:34:07]

Is there a reason you're not just adding this as a string to the data source?

[sheretov, 2016/05/19 16:16:45]

Done.

+
+  web_ui->RegisterMessageCallback(
+      kRequestExtensionId,
+      base::Bind(&CastUI::OnRequestExtensionId, base::Unretained(this)));
+
+  // Set up the chrome://cast source.
+  content::WebUIDataSource* html_source =
+      content::WebUIDataSource::Create(chrome::kChromeUICastHost);
+
+  // Add required resources.
+  html_source->AddResourcePath("cast.css", IDR_CAST_CSS);
+  html_source->AddResourcePath("cast.js", IDR_CAST_JS);
+  html_source->AddResourcePath("cast_favicon.ico", IDR_CAST_FAVICON);
+  html_source->SetDefaultResource(IDR_CAST_HTML);
+  html_source->OverrideContentSecurityPolicyObjectSrc("object-src *;");
+
+  Profile* profile = Profile::FromWebUI(web_ui);
+  content::WebUIDataSource::Add(profile, html_source);
+}
+
+CastUI::~CastUI() {
+}
+
+void CastUI::OnRequestExtensionId(const base::ListValue* args) {
+  base::StringValue result(component_extension_id_);
+  web_ui()->CallJavascriptFunction(kSetExtensionId, result);
+}