app_shell: Add ShellContentRendererClient

apps/shell/shell_content_renderer_client.cc

Index: apps/shell/shell_content_renderer_client.cc
diff --git a/apps/shell/shell_content_renderer_client.cc b/apps/shell/shell_content_renderer_client.cc
new file mode 100644
index 0000000000000000000000000000000000000000..0414700a9a03f44bad5b9d13c5c60a350edb964b
--- /dev/null
+++ b/apps/shell/shell_content_renderer_client.cc
@@ -0,0 +1,76 @@
+// 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 "apps/shell/shell_content_renderer_client.h"
+
+#include "apps/shell/shell_extensions_client.h"
+#include "chrome/renderer/extensions/dispatcher.h"
+#include "chrome/renderer/extensions/extension_helper.h"
+#include "content/public/renderer/render_thread.h"
+#include "extensions/common/extensions_client.h"
+
+using blink::WebFrame;
+using blink::WebString;
+using content::RenderThread;
+
+namespace apps {
+
+ShellContentRendererClient::ShellContentRendererClient() {}
+
+ShellContentRendererClient::~ShellContentRendererClient() {}
+
+void ShellContentRendererClient::RenderThreadStarted() {
+  RenderThread* thread = RenderThread::Get();
+
+  extension_dispatcher_.reset(new extensions::Dispatcher());
+  thread->AddObserver(extension_dispatcher_.get());
+
+  // TODO(jamescook): Init WebSecurityPolicy for chrome-extension: schemes.
+  // See ChromeContentRendererClient for details.
+
+  extensions_client_.reset(new ShellExtensionsClient);
+  extensions::ExtensionsClient::Set(extensions_client_.get());
+}
+
+void ShellContentRendererClient::RenderFrameCreated(
+    content::RenderFrame* render_frame) {
+  // TODO(jamescook): Create ContentSettingsObserver for allowScript?

[Yoyo Zhou, 2014/01/30 00:49:54]

This doesn't look extension-related but it might still be needed eventually. I
don't know.

[James Cook, 2014/01/30 01:10:13]

Removed TODO. We'll solve that problem if we need it.

+  // TODO(jamescook): Create ExtensionFrameHelper?

[Yoyo Zhou, 2014/01/30 00:49:54]

Looks like this is for Pepper web plugins?
https://codereview.chromium.org/105553005

[James Cook, 2014/01/30 01:10:13]

Hrm, so we might need it eventually if we want Pepper Flash?  I left it in.

+}
+
+void ShellContentRendererClient::RenderViewCreated(
+    content::RenderView* render_view) {
+  new extensions::ExtensionHelper(render_view, extension_dispatcher_.get());
+}
+
+bool ShellContentRendererClient::WillSendRequest(
+    blink::WebFrame* frame,
+    content::PageTransition transition_type,
+    const GURL& url,
+    const GURL& first_party_for_cookies,
+    GURL* new_url) {
+  // TODO(jamescook): Cause an error for bad extension scheme requests?
+  return false;
+}
+
+void ShellContentRendererClient::DidCreateScriptContext(
+    WebFrame* frame, v8::Handle<v8::Context> context, int extension_group,
+    int world_id) {
+  // TODO(jamescook): We need to make sure not to load PermissionFeatures
+  // that our PermissionsProvider doesn't know about. Once that is fixed we
+  // can call extension_dispatcher_->DidCreateScriptContext() to set up the
+  // extension's script context.
+}
+
+void ShellContentRendererClient::WillReleaseScriptContext(
+    WebFrame* frame, v8::Handle<v8::Context> context, int world_id) {
+  // TODO(jamescook): As above, but to release the extension script context.
+}
+
+bool ShellContentRendererClient::ShouldEnableSiteIsolationPolicy() const {
+  // Extension renderers don't need site isolation.
+  return false;
+}
+
+}  // namespace apps