Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(187)

Unified Diff: chrome/browser/renderer_host/chrome_render_view_host_observer.cc

Issue 8585016: Don't use process isolation for bookmark apps. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix bug with old_url. Created 9 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/renderer_host/chrome_render_view_host_observer.cc
diff --git a/chrome/browser/renderer_host/chrome_render_view_host_observer.cc b/chrome/browser/renderer_host/chrome_render_view_host_observer.cc
index cb924d3b6b035ffe2eaaf658f3879f7af9d0e5d0..082e05f6b3e2ee7668c16a87ec986f0602faeb2d 100644
--- a/chrome/browser/renderer_host/chrome_render_view_host_observer.cc
+++ b/chrome/browser/renderer_host/chrome_render_view_host_observer.cc
@@ -126,9 +126,9 @@ void ChromeRenderViewHostObserver::InitRenderViewForExtensions() {
}
const Extension* ChromeRenderViewHostObserver::GetExtension() {
- // Note that due to ChromeContentBrowserClient::GetEffectiveURL(), even hosted
- // apps will have a chrome-extension:// URL for their site, so we can ignore
- // that wrinkle here.
+ // Note that due to ChromeContentBrowserClient::GetEffectiveURL(), hosted apps
+ // (excluding bookmark apps) will have a chrome-extension:// URL for their
+ // site, so we can ignore that wrinkle here.
SiteInstance* site_instance = render_view_host()->site_instance();
const GURL& site = site_instance->site();

Powered by Google App Engine
This is Rietveld 408576698