Whitelisting exts and plugins from cross-site document blocking

chrome/renderer/chrome_content_renderer_client.cc

Index: chrome/renderer/chrome_content_renderer_client.cc
diff --git a/chrome/renderer/chrome_content_renderer_client.cc b/chrome/renderer/chrome_content_renderer_client.cc
index ef9c3fc04d9a306ee5b4af59a4784042814afd79..86c9f7b76feeffb51eba42d66bb2c78f1ff2bb80 100644
--- a/chrome/renderer/chrome_content_renderer_client.cc
+++ b/chrome/renderer/chrome_content_renderer_client.cc
@@ -1305,4 +1305,15 @@ bool ChromeContentRendererClient::ShouldReportDetailedMessageForSource(
   return extensions::IsSourceFromAnExtension(source);
 }
 
+bool ChromeContentRendererClient::ShouldEnableSiteIsolationPolicy() const {
+  // SiteIsolationPolicy is off by default. We would like to activate the

[Charlie Reis, 2013/09/03 22:59:17]

nit: drop "the"

[dsjang, 2013/09/04 00:37:37]

Done.

+  // cross-site document blocking (for UMA data collection) for normal renderer
+  // processes running an normal web page from the Internet. We only turn on

[Charlie Reis, 2013/09/03 22:59:17]

nit: a normal

[dsjang, 2013/09/04 00:37:37]

Done.

+  // SiteIsolationPolicy for a renderer process that does not have the extension
+  // flag on. Currently, this cannot be turned off for resource requests from

[Charlie Reis, 2013/09/03 22:59:17]

No need to mention plugins here, since that's explained elsewhere.  Feel free to
drop the last sentence.

[dsjang, 2013/09/04 00:37:37]

Done.

+  // plugins since they request resources via their embedding renderer process.
+  CommandLine* command_line = CommandLine::ForCurrentProcess();
+  return !command_line->HasSwitch(switches::kExtensionProcess);
+}
+
 }  // namespace chrome