Don't use process isolation for bookmark apps.

chrome/browser/extensions/extension_service.cc

Index: chrome/browser/extensions/extension_service.cc
diff --git a/chrome/browser/extensions/extension_service.cc b/chrome/browser/extensions/extension_service.cc
index 3f77711f17aa97c251d970882c1fcb24e9bfa3ab..8a725e5087a331ebc7ae1db53d14716ba1a65a18 100644
--- a/chrome/browser/extensions/extension_service.cc
+++ b/chrome/browser/extensions/extension_service.cc
@@ -2187,7 +2187,7 @@ const Extension* ExtensionService::GetDisabledExtensionByWebExtent(
 bool ExtensionService::ExtensionBindingsAllowed(const GURL& url) {
   // Allow bindings for all packaged extensions.
   // Note that GetExtensionByURL may return an Extension for hosted apps
-  // if the URL came from GetEffectiveURL.
+  // (excluding bookmark apps) if the URL came from GetEffectiveURL.

[Aaron Boodman, 2011/11/18 18:19:52]

I don't think the last part of this comment is necessary. ExtensionService
should have no knowledge of GetEffectiveURL.

[Charlie Reis, 2011/11/18 19:45:42]

I think it's useful to have that comment, since it explains why we need two
different types of checks for hosted apps below.

   const Extension* extension = GetExtensionByURL(url);
   if (extension && extension->GetType() != Extension::TYPE_HOSTED_APP)
     return true;