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

Unified Diff: extensions/common/features/complex_feature.cc

Issue 2241203003: Pass user session type to extension feature checks (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: split out some stuff Created 4 years, 4 months 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: extensions/common/features/complex_feature.cc
diff --git a/extensions/common/features/complex_feature.cc b/extensions/common/features/complex_feature.cc
index ed129da1e6413884c70e853bcb3b27c4aa4ed90b..b5a76b926a3b37744c6f19e1cd8ba911911c0e67 100644
--- a/extensions/common/features/complex_feature.cc
+++ b/extensions/common/features/complex_feature.cc
@@ -59,17 +59,18 @@ Feature::Availability ComplexFeature::IsAvailableToManifest(
Feature::Availability ComplexFeature::IsAvailableToContext(
const Extension* extension,
Context context,
+ SessionType session_type,
const GURL& url,
Platform platform) const {
- Feature::Availability first_availability =
- features_[0]->IsAvailableToContext(extension, context, url, platform);
+ Feature::Availability first_availability = features_[0]->IsAvailableToContext(
+ extension, context, session_type, url, platform);
if (first_availability.is_available())
return first_availability;
for (FeatureList::const_iterator it = features_.begin() + 1;
it != features_.end(); ++it) {
- Availability availability =
- (*it)->IsAvailableToContext(extension, context, url, platform);
+ Availability availability = (*it)->IsAvailableToContext(
+ extension, context, session_type, url, platform);
if (availability.is_available())
return availability;
}

Powered by Google App Engine
This is Rietveld 408576698