Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3509)

Unified Diff: chrome/browser/extensions/extension_service_unittest.cc

Issue 397853002: Refactor safe-browsing build-config definitions. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase to pick up safe_browsing_resource_throttle_factory change. Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/extension_service_unittest.cc
diff --git a/chrome/browser/extensions/extension_service_unittest.cc b/chrome/browser/extensions/extension_service_unittest.cc
index 3b749066bfb741c5364d497814ee36b03abc4ea8..53bce710540ed05ec5deafcc04b0b83c9d5998fe 100644
--- a/chrome/browser/extensions/extension_service_unittest.cc
+++ b/chrome/browser/extensions/extension_service_unittest.cc
@@ -132,8 +132,8 @@
#include "chrome/browser/chromeos/settings/device_settings_service.h"
#endif
-// The blacklist tests rely on safe browsing.
-#if defined(FULL_SAFE_BROWSING) || defined(MOBILE_SAFE_BROWSING)
+// The blacklist tests rely on the safe-browsing database.
+#if defined(SAFE_BROWSING_DATABASE)
#define ENABLE_BLACKLIST_TESTS
#endif
@@ -3549,7 +3549,6 @@ TEST_F(ExtensionServiceTest, ReloadBlacklistedExtension) {
EXPECT_EQ(StringSet(good1, good2),
registry()->blacklisted_extensions().GetIDs());
}
-
#endif // defined(ENABLE_BLACKLIST_TESTS)
// Will not install extension blacklisted by policy.

Powered by Google App Engine
This is Rietveld 408576698