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

Unified Diff: chrome/common/extensions/extension_set.h

Issue 8585016: Don't use process isolation for bookmark apps. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 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/common/extensions/extension_set.h
diff --git a/chrome/common/extensions/extension_set.h b/chrome/common/extensions/extension_set.h
index 36fd6dabaade61f418927512729a9251c4805c5e..b16ba587b0f9a9152705ee31d5b1ec6979de5cd2 100644
--- a/chrome/common/extensions/extension_set.h
+++ b/chrome/common/extensions/extension_set.h
@@ -52,6 +52,8 @@ class ExtensionSet {
// NOTE: This can return NULL if called before UpdateExtensions receives
// bulk extension data (e.g. if called from
// EventBindings::HandleContextCreated)
+ // We exclude bookmark apps. TODO(creis): Should they be excluded from the
+ // whole set in the first place?
const Extension* GetByURL(const GURL& url) const;
// Returns true if |new_url| is in the extent of the same extension as

Powered by Google App Engine
This is Rietveld 408576698