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

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

Issue 2147923002: [Extensions] Fixed removing from blacklist. (Closed) Base URL: ssh://bitbucket.browser.yandex-team.ru/chromium/src.git@master
Patch Set: Review issues are fixed. Created 4 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/blacklist_unittest.cc
diff --git a/chrome/browser/extensions/blacklist_unittest.cc b/chrome/browser/extensions/blacklist_unittest.cc
index 644fb4d9741def8faffcd415b0f483fbc14de4a9..c1c1277720fa195c2fe97bbaa36e6a3ae24e3d3c 100644
--- a/chrome/browser/extensions/blacklist_unittest.cc
+++ b/chrome/browser/extensions/blacklist_unittest.cc
@@ -124,14 +124,14 @@ TEST_F(BlacklistTest, ClearsPreferencesBlacklist) {
std::string b = AddExtension("b");
// Blacklist an installed extension.
- prefs()->SetExtensionBlacklisted(a, true);
+ prefs()->SetExtensionBlacklistState(a, extensions::BLACKLISTED_MALWARE);
Devlin 2016/07/19 16:02:44 This file is in the extensions namespace, so the e
atuchin 2016/07/20 06:02:45 Done.
// Blacklist some non-installed extensions. This is what the old preferences
// blacklist looked like.
std::string c = "cccccccccccccccccccccccccccccccc";
std::string d = "dddddddddddddddddddddddddddddddd";
- prefs()->SetExtensionBlacklisted(c, true);
- prefs()->SetExtensionBlacklisted(d, true);
+ prefs()->SetExtensionBlacklistState(c, extensions::BLACKLISTED_MALWARE);
+ prefs()->SetExtensionBlacklistState(d, extensions::BLACKLISTED_MALWARE);
EXPECT_EQ(Set(a, c, d), prefs()->GetBlacklistedExtensions());

Powered by Google App Engine
This is Rietveld 408576698