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

Unified Diff: chrome/common/extensions/features/manifest_feature.cc

Issue 12522004: Lazily load extension API schemas (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 7 years, 9 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: chrome/common/extensions/features/manifest_feature.cc
diff --git a/chrome/common/extensions/features/manifest_feature.cc b/chrome/common/extensions/features/manifest_feature.cc
index 28406d8fecacb2f006413a9d2315e6ea3c61ad7f..51d9fd51d5c39746fa52d00b74f5f0b0b3d6a1e5 100644
--- a/chrome/common/extensions/features/manifest_feature.cc
+++ b/chrome/common/extensions/features/manifest_feature.cc
@@ -17,16 +17,18 @@ ManifestFeature::~ManifestFeature() {
Feature::Availability ManifestFeature::IsAvailableToContext(
const Extension* extension,
Feature::Context context,
+ const GURL& url,
Feature::Platform platform) const {
Availability availability = SimpleFeature::IsAvailableToContext(extension,
context,
+ url,
platform);
if (!availability.is_available())
return availability;
// We know we can skip manifest()->GetKey() here because we just did the same
// validation it would do above.
- if (!extension->manifest()->value()->HasKey(name()))
+ if (extension && !extension->manifest()->value()->HasKey(name()))
return CreateAvailability(NOT_PRESENT, extension->GetType());
return CreateAvailability(IS_AVAILABLE);
« no previous file with comments | « chrome/common/extensions/features/manifest_feature.h ('k') | chrome/common/extensions/features/permission_feature.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698