OLD | NEW |
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 #include "extensions/common/permissions/permissions_data.h" | 5 #include "extensions/common/permissions/permissions_data.h" |
6 | 6 |
7 #include "base/command_line.h" | 7 #include "base/command_line.h" |
8 #include "base/memory/scoped_ptr.h" | 8 #include "base/memory/scoped_ptr.h" |
9 #include "base/strings/string16.h" | 9 #include "base/strings/string16.h" |
10 #include "base/strings/string_number_conversions.h" | 10 #include "base/strings/string_number_conversions.h" |
(...skipping 116 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
127 | 127 |
128 // The feature should exist since we just got an APIPermission for it. The | 128 // The feature should exist since we just got an APIPermission for it. The |
129 // two systems should be updated together whenever a permission is added. | 129 // two systems should be updated together whenever a permission is added. |
130 DCHECK(feature) << "Could not find feature for " << iter->name(); | 130 DCHECK(feature) << "Could not find feature for " << iter->name(); |
131 // http://crbug.com/176381 | 131 // http://crbug.com/176381 |
132 if (!feature) { | 132 if (!feature) { |
133 to_remove.push_back(iter->id()); | 133 to_remove.push_back(iter->id()); |
134 continue; | 134 continue; |
135 } | 135 } |
136 | 136 |
137 Feature::Availability availability = feature->IsAvailableToManifest( | 137 Feature::Availability availability = |
138 extension->id(), | 138 feature->IsAvailableToExtension(extension); |
139 extension->GetType(), | |
140 Feature::ConvertLocation(extension->location()), | |
141 extension->manifest_version()); | |
142 | 139 |
143 if (!availability.is_available()) { | 140 if (!availability.is_available()) { |
144 // Don't fail, but warn the developer that the manifest contains | 141 // Don't fail, but warn the developer that the manifest contains |
145 // unrecognized permissions. This may happen legitimately if the | 142 // unrecognized permissions. This may happen legitimately if the |
146 // extensions requests platform- or channel-specific permissions. | 143 // extensions requests platform- or channel-specific permissions. |
147 extension->AddInstallWarning(InstallWarning(availability.message(), | 144 extension->AddInstallWarning(InstallWarning(availability.message(), |
148 feature->name())); | 145 feature->name())); |
149 to_remove.push_back(iter->id()); | 146 to_remove.push_back(iter->id()); |
150 continue; | 147 continue; |
151 } | 148 } |
(...skipping 456 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
608 initial_optional_permissions_->api_permissions, | 605 initial_optional_permissions_->api_permissions, |
609 initial_optional_permissions_->manifest_permissions, | 606 initial_optional_permissions_->manifest_permissions, |
610 initial_optional_permissions_->host_permissions, | 607 initial_optional_permissions_->host_permissions, |
611 URLPatternSet()); | 608 URLPatternSet()); |
612 | 609 |
613 initial_required_permissions_.reset(); | 610 initial_required_permissions_.reset(); |
614 initial_optional_permissions_.reset(); | 611 initial_optional_permissions_.reset(); |
615 } | 612 } |
616 | 613 |
617 } // namespace extensions | 614 } // namespace extensions |
OLD | NEW |