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

Unified Diff: chrome/browser/extensions/extension_prefs_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/extension_prefs_unittest.cc
diff --git a/chrome/browser/extensions/extension_prefs_unittest.cc b/chrome/browser/extensions/extension_prefs_unittest.cc
index 7df1f84d0bce0a9790a46643e8609978ecee2cc8..3e8d0a2123550c26031bd08094861a97827ea4f2 100644
--- a/chrome/browser/extensions/extension_prefs_unittest.cc
+++ b/chrome/browser/extensions/extension_prefs_unittest.cc
@@ -851,7 +851,8 @@ class ExtensionPrefsBlacklistState : public ExtensionPrefsTest {
ExtensionIdSet empty_ids;
EXPECT_EQ(empty_ids, prefs()->GetBlacklistedExtensions());
- prefs()->SetExtensionBlacklisted(extension_a_->id(), true);
+ prefs()->SetExtensionBlacklistState(extension_a_->id(),
+ extensions::BLACKLISTED_MALWARE);
Devlin 2016/07/19 16:02:44 here, too
atuchin 2016/07/20 06:02:45 Done.
EXPECT_EQ(BLACKLISTED_MALWARE,
prefs()->GetExtensionBlacklistState(extension_a_->id()));
@@ -862,7 +863,8 @@ class ExtensionPrefsBlacklistState : public ExtensionPrefsTest {
EXPECT_FALSE(prefs()->IsExtensionBlacklisted(extension_a_->id()));
EXPECT_EQ(empty_ids, prefs()->GetBlacklistedExtensions());
- prefs()->SetExtensionBlacklisted(extension_a_->id(), true);
+ prefs()->SetExtensionBlacklistState(extension_a_->id(),
+ extensions::BLACKLISTED_MALWARE);
EXPECT_TRUE(prefs()->IsExtensionBlacklisted(extension_a_->id()));
EXPECT_EQ(BLACKLISTED_MALWARE,
prefs()->GetExtensionBlacklistState(extension_a_->id()));

Powered by Google App Engine
This is Rietveld 408576698