Don't use process isolation for bookmark apps.

chrome/browser/extensions/extension_service.cc

 // Copyright (c) 2011 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/extensions/extension_service.h"
 
 #include <algorithm>
 #include <set>
 
 #include "base/basictypes.h"
@@ skipping 2169 matching lines @@
   for (size_t i = 0; i < disabled_extensions_.size(); ++i) {
     if (disabled_extensions_[i]->web_extent().MatchesURL(url))
       return disabled_extensions_[i];
   }
   return NULL;
 }
 
 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;
 
   // Allow bindings for all component, hosted apps.
   if (!extension)
     extension = GetExtensionByWebExtent(url);
   return (extension && extension->location() == Extension::COMPONENT);
 }
 
@@ skipping 301 matching lines @@
 
 ExtensionService::NaClModuleInfoList::iterator
     ExtensionService::FindNaClModule(const GURL& url) {
   for (NaClModuleInfoList::iterator iter = nacl_module_list_.begin();
        iter != nacl_module_list_.end(); ++iter) {
     if (iter->url == url)
       return iter;
   }
   return nacl_module_list_.end();
 }