Whitelisting exts and plugins from cross-site document blocking

content/public/renderer/content_renderer_client.h

 // Copyright (c) 2012 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.
 
 #ifndef CONTENT_PUBLIC_RENDERER_CONTENT_RENDERER_CLIENT_H_
 #define CONTENT_PUBLIC_RENDERER_CONTENT_RENDERER_CLIENT_H_
 
 #include <string>
 
 #include "base/memory/weak_ptr.h"
@@ skipping 232 matching lines @@
 
   // Returns true if the page at |url| can use Pepper MediaStream APIs.
   virtual bool AllowPepperMediaStreamAPI(const GURL& url);
 
   // Returns true if we should report a detailed message (including a stack
   // trace) for console [logs|errors|exceptions]. |source| is the WebKit-
   // reported source for the error; this can point to a page or a script,
   // and can be external or internal.
   virtual bool ShouldReportDetailedMessageForSource(
       const base::string16& source) const;
+
+  // Returns true if we apply the cross-site document blocking policy to this
+  // renderer process. Currently, we apply the policy only to a render process

[Charlie Reis, 2013/09/03 20:22:10]

render -> renderer
(here and below)

[dsjang, 2013/09/03 22:31:30]

Done.

+  // running on a normal page from the internet, excluding render processes for

[Charlie Reis, 2013/09/03 20:22:10]

internet -> web

[dsjang, 2013/09/03 22:31:30]

Done.

+  // extensions. Unfortunately, there's no easy way to exclude a renderer

[Charlie Reis, 2013/09/03 20:22:10]

nit: Drop "excluding render processes for extensions," since the concept of
extensions doesn't belong in content.

[dsjang, 2013/09/03 22:31:30]

Done.

+  // process for plugins here, but our ultimate goal is to exclude plugin

[Charlie Reis, 2013/09/03 20:22:10]

This last sentence is a bit confusing to me.  We don't create a renderer process
for plugins-- it's sort of the other way around.  Can you rephrase?

[dsjang, 2013/09/03 22:31:30]

Done.

+  // processes too.
+  virtual bool ShouldEnableSiteIsolationPolicy() const;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_RENDERER_CONTENT_RENDERER_CLIENT_H_