Chromium Code Reviews| Index: chrome/common/extensions/features/permission_feature.cc |
| diff --git a/chrome/common/extensions/features/permission_feature.cc b/chrome/common/extensions/features/permission_feature.cc |
| index 5fa2ba2ec8373f34b606155718979413f476148b..801ed95955b7d3eca22c2aa3307b71dbe5404ea7 100644 |
| --- a/chrome/common/extensions/features/permission_feature.cc |
| +++ b/chrome/common/extensions/features/permission_feature.cc |
| @@ -27,10 +27,8 @@ Feature::Availability PermissionFeature::IsAvailableToContext( |
| if (!availability.is_available()) |
| return availability; |
| - // Optional permissions need to be checked so an API will not be set to |
| - // undefined forever, when it could just need optional permissions. |
| if (extension && !extension->HasAPIPermission(name()) && |
| - !PermissionsData::GetOptionalPermissions(extension)-> |
| + !PermissionsData::GetActivePermissions(extension)-> |
|
not at google - send to devlin
2013/06/05 19:03:44
you can just remove this, HasAPIPermission checks
cduvall
2013/06/05 23:27:59
Done.
|
| HasAnyAccessToAPI(name())) { |
| return CreateAvailability(NOT_PRESENT, extension->GetType()); |
| } |