From 670696347dd8d2a5c7e19ca6445569ad8d938c19 Mon Sep 17 00:00:00 2001 From: "glen@chromium.org" Date: Sat, 9 May 2009 02:19:19 +0000 Subject: 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 --- chrome/browser/extensions/extensions_service.cc | 20 ++------------------ 1 file changed, 2 insertions(+), 18 deletions(-) (limited to 'chrome/browser') 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]; -- cgit v1.1