Validate privacy blacklist format when validating an extension

containing a privacy blacklist.

TEST=Covered by unit_tests.
BUG=21541

Review URL: http://codereview.chromium.org/453016

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@33573 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 35 insertions, 0 deletions

diff --git a/chrome/browser/extensions/extension_file_util_unittest.cc b/chrome/browser/extensions/extension_file_util_unittest.cc
index c462032..6e00eb0 100644
--- a/chrome/browser/extensions/extension_file_util_unittest.cc
+++ b/chrome/browser/extensions/extension_file_util_unittest.cc
@@ -6,6 +6,7 @@
 
 #include "base/file_util.h"
 #include "base/scoped_temp_dir.h"
+#include "base/string_util.h"
 #include "base/path_service.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/extensions/extension.h"
@@ -220,6 +221,40 @@ TEST(ExtensionFileUtil, LoadExtensionGivesHelpfullErrorOnBadManifest) {
                "Line: 2, column: 16, Syntax error.", error.c_str());
 }
 
+TEST(ExtensionFileUtil, MissingPrivacyBlacklist) {
+  FilePath install_dir;
+  ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &install_dir));
+  install_dir = install_dir.AppendASCII("extensions")
+      .AppendASCII("privacy_blacklists")
+      .AppendASCII("missing_blacklist");
+
+  std::string error;
+  scoped_ptr<Extension> extension(
+      extension_file_util::LoadExtension(install_dir, false, &error));
+  ASSERT_TRUE(extension == NULL);
+  ASSERT_FALSE(error.empty());
+  EXPECT_TRUE(MatchPattern(error,
+      "Could not load '*privacy_blacklist.pbl' for privacy blacklist: "
+      "file does not exist.")) << error;
+}
+
+TEST(ExtensionFileUtil, InvalidPrivacyBlacklist) {
+  FilePath install_dir;
+  ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &install_dir));
+  install_dir = install_dir.AppendASCII("extensions")
+      .AppendASCII("privacy_blacklists")
+      .AppendASCII("invalid_blacklist");
+
+  std::string error;
+  scoped_ptr<Extension> extension(
+      extension_file_util::LoadExtension(install_dir, false, &error));
+  ASSERT_TRUE(extension == NULL);
+  ASSERT_FALSE(error.empty());
+  EXPECT_TRUE(MatchPattern(error,
+      "Could not load '*privacy_blacklist.pbl' for privacy blacklist: "
+      "Incorrect header.")) << error;
+}
+
 // TODO(aa): More tests as motivation allows. Maybe steal some from
 // ExtensionsService? Many of them could probably be tested here without the
 // MessageLoop shenanigans.