summaryrefslogtreecommitdiffstats
path: root/chrome/browser
diff options
context:
space:
mode:
authorglen@chromium.org <glen@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-05-09 02:19:19 +0000
committerglen@chromium.org <glen@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-05-09 02:19:19 +0000
commit670696347dd8d2a5c7e19ca6445569ad8d938c19 (patch)
tree31a8ded45f932af3bed58f87622f5f46093806a7 /chrome/browser
parent75bb32b595b085a69572c96afc179847976bbe76 (diff)
downloadchromium_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.cc20
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];