diff options
| author | aa@chromium.org <aa@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-09-18 01:00:06 +0000 |
|---|---|---|
| committer | aa@chromium.org <aa@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-09-18 01:00:06 +0000 |
| commit | 7e730b5dea428a825bdc0a3b687e9213be995a0d (patch) | |
| tree | 747cf1379f4978419ee462f96981fd5701050431 | |
| parent | aea80604eee7d26eeb7947bfc577acaed05ac7b1 (diff) | |
| download | chromium_src-7e730b5dea428a825bdc0a3b687e9213be995a0d.zip chromium_src-7e730b5dea428a825bdc0a3b687e9213be995a0d.tar.gz chromium_src-7e730b5dea428a825bdc0a3b687e9213be995a0d.tar.bz2 | |
Fix bug where we fail to install an extension that has an empty
permissions array in its manifest.
This was caused an old attempt to display a warning in this
situation, back when we were ambitious.
BUG=22157
TEST=Install extension associated with bug.
Review URL: http://codereview.chromium.org/213009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26534 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | chrome/common/extensions/extension.cc | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/chrome/common/extensions/extension.cc b/chrome/common/extensions/extension.cc index 2566d41..cd2a8c7 100644 --- a/chrome/common/extensions/extension.cc +++ b/chrome/common/extensions/extension.cc @@ -947,11 +947,6 @@ bool Extension::InitFromValue(const DictionaryValue& source, bool require_id, return false; } - if (permissions->GetSize() == 0) { - ExtensionErrorReporter::GetInstance()->ReportError( - errors::kInvalidPermissionCountWarning, false); - } - for (size_t i = 0; i < permissions->GetSize(); ++i) { std::string permission_str; if (!permissions->GetString(i, &permission_str)) { |