diff options
author | glen@chromium.org <glen@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-09 02:19:19 +0000 |
---|---|---|
committer | glen@chromium.org <glen@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-09 02:19:19 +0000 |
commit | 670696347dd8d2a5c7e19ca6445569ad8d938c19 (patch) | |
tree | 31a8ded45f932af3bed58f87622f5f46093806a7 /chrome/browser | |
parent | 75bb32b595b085a69572c96afc179847976bbe76 (diff) | |
download | chromium_src-670696347dd8d2a5c7e19ca6445569ad8d938c19.zip chromium_src-670696347dd8d2a5c7e19ca6445569ad8d938c19.tar.gz chromium_src-670696347dd8d2a5c7e19ca6445569ad8d938c19.tar.bz2 |
Add browser_theme_provider to GYP so that Mac and Linux can build.
Temporarily remove theme resource validation to get unittests passing.
TBR=beng
Review URL: http://codereview.chromium.org/113178
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@15709 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser')
-rw-r--r-- | chrome/browser/extensions/extensions_service.cc | 20 |
1 files changed, 2 insertions, 18 deletions
diff --git a/chrome/browser/extensions/extensions_service.cc b/chrome/browser/extensions/extensions_service.cc index 2a152e1..2d1807e 100644 --- a/chrome/browser/extensions/extensions_service.cc +++ b/chrome/browser/extensions/extensions_service.cc @@ -348,24 +348,8 @@ Extension* ExtensionsServiceBackend::LoadExtension( return NULL; } - // Validate that claimed image resources actually exist. - DictionaryValue* images_value = extension->GetThemeImages(); - if (images_value) { - DictionaryValue::key_iterator iter = images_value->begin_keys(); - while (iter != images_value->end_keys()) { - std::string val; - images_value->GetString(*iter, &val); - const FilePath& path = extension->path().AppendASCII(val); - if (!file_util::PathExists(path)) { - ReportExtensionLoadError(extension_path, - StringPrintf("Could not load '%s' for theme.", - WideToUTF8(path.ToWStringHack()).c_str())); - return NULL; - } - ++iter; - } - } - // Validate that claimed script resources actually exist. + // TODO(glen): Add theme resource validation here. http://crbug.com/11678 + // Validate that claimed script resources actually exist. for (size_t i = 0; i < extension->content_scripts().size(); ++i) { const UserScript& script = extension->content_scripts()[i]; |