| OLD | NEW |
| 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 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 "chrome/common/extensions/manifest.h" | 5 #include "chrome/common/extensions/manifest.h" |
| 6 | 6 |
| 7 #include "base/basictypes.h" | 7 #include "base/basictypes.h" |
| 8 #include "base/lazy_instance.h" | 8 #include "base/lazy_instance.h" |
| 9 #include "base/logging.h" | 9 #include "base/logging.h" |
| 10 #include "base/stringprintf.h" | 10 #include "base/stringprintf.h" |
| (...skipping 121 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 132 *error = errors::kPlatformAppNeedsManifestVersion2; | 132 *error = errors::kPlatformAppNeedsManifestVersion2; |
| 133 return false; | 133 return false; |
| 134 } | 134 } |
| 135 | 135 |
| 136 // Check every feature to see if its in the manifest. Note that this means | 136 // Check every feature to see if its in the manifest. Note that this means |
| 137 // we will ignore keys that are not features; we do this for forward | 137 // we will ignore keys that are not features; we do this for forward |
| 138 // compatibility. | 138 // compatibility. |
| 139 // TODO(aa): Consider having an error here in the case of strict error | 139 // TODO(aa): Consider having an error here in the case of strict error |
| 140 // checking to let developers know when they screw up. | 140 // checking to let developers know when they screw up. |
| 141 | 141 |
| 142 std::set<std::string> feature_names = | 142 FeatureProvider* provider = BaseFeatureProvider::GetByName("manifest"); |
| 143 BaseFeatureProvider::GetManifestFeatures()->GetAllFeatureNames(); | 143 std::set<std::string> feature_names = provider->GetAllFeatureNames(); |
| 144 for (std::set<std::string>::iterator feature_name = feature_names.begin(); | 144 for (std::set<std::string>::iterator feature_name = feature_names.begin(); |
| 145 feature_name != feature_names.end(); ++feature_name) { | 145 feature_name != feature_names.end(); ++feature_name) { |
| 146 // Use Get instead of HasKey because the former uses path expansion. | 146 // Use Get instead of HasKey because the former uses path expansion. |
| 147 if (!value_->Get(*feature_name, NULL)) | 147 if (!value_->Get(*feature_name, NULL)) |
| 148 continue; | 148 continue; |
| 149 | 149 |
| 150 Feature* feature = | 150 Feature* feature = provider->GetFeature(*feature_name); |
| 151 BaseFeatureProvider::GetManifestFeatures()->GetFeature(*feature_name); | |
| 152 Feature::Availability result = feature->IsAvailableToManifest( | 151 Feature::Availability result = feature->IsAvailableToManifest( |
| 153 extension_id_, type_, Feature::ConvertLocation(location_), | 152 extension_id_, type_, Feature::ConvertLocation(location_), |
| 154 GetManifestVersion()); | 153 GetManifestVersion()); |
| 155 if (!result.is_available()) | 154 if (!result.is_available()) |
| 156 warnings->push_back(InstallWarning( | 155 warnings->push_back(InstallWarning( |
| 157 InstallWarning::FORMAT_TEXT, result.message())); | 156 InstallWarning::FORMAT_TEXT, result.message())); |
| 158 } | 157 } |
| 159 | 158 |
| 160 // Also generate warnings for keys that are not features. | 159 // Also generate warnings for keys that are not features. |
| 161 for (base::DictionaryValue::Iterator it(*value_); !it.IsAtEnd(); | 160 for (base::DictionaryValue::Iterator it(*value_); !it.IsAtEnd(); |
| 162 it.Advance()) { | 161 it.Advance()) { |
| 163 if (!BaseFeatureProvider::GetManifestFeatures()->GetFeature(it.key())) { | 162 if (!provider->GetFeature(it.key())) { |
| 164 warnings->push_back(InstallWarning( | 163 warnings->push_back(InstallWarning( |
| 165 InstallWarning::FORMAT_TEXT, | 164 InstallWarning::FORMAT_TEXT, |
| 166 base::StringPrintf("Unrecognized manifest key '%s'.", | 165 base::StringPrintf("Unrecognized manifest key '%s'.", |
| 167 it.key().c_str()))); | 166 it.key().c_str()))); |
| 168 } | 167 } |
| 169 } | 168 } |
| 170 return true; | 169 return true; |
| 171 } | 170 } |
| 172 | 171 |
| 173 bool Manifest::HasKey(const std::string& key) const { | 172 bool Manifest::HasKey(const std::string& key) const { |
| (...skipping 67 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 241 key += components[i]; | 240 key += components[i]; |
| 242 if (!CanAccessKey(key)) | 241 if (!CanAccessKey(key)) |
| 243 return false; | 242 return false; |
| 244 key += '.'; | 243 key += '.'; |
| 245 } | 244 } |
| 246 return true; | 245 return true; |
| 247 } | 246 } |
| 248 | 247 |
| 249 bool Manifest::CanAccessKey(const std::string& key) const { | 248 bool Manifest::CanAccessKey(const std::string& key) const { |
| 250 Feature* feature = | 249 Feature* feature = |
| 251 BaseFeatureProvider::GetManifestFeatures()->GetFeature(key); | 250 BaseFeatureProvider::GetByName("manifest")->GetFeature(key); |
| 252 if (!feature) | 251 if (!feature) |
| 253 return true; | 252 return true; |
| 254 | 253 |
| 255 return feature->IsAvailableToManifest( | 254 return feature->IsAvailableToManifest( |
| 256 extension_id_, type_, Feature::ConvertLocation(location_), | 255 extension_id_, type_, Feature::ConvertLocation(location_), |
| 257 GetManifestVersion()).is_available(); | 256 GetManifestVersion()).is_available(); |
| 258 } | 257 } |
| 259 | 258 |
| 260 } // namespace extensions | 259 } // namespace extensions |
| OLD | NEW |