[Extensions] Fixed removing from blacklist.

chrome/browser/extensions/blacklist_unittest.cc

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/bind.h"
 #include "base/run_loop.h"
 #include "base/stl_util.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "chrome/browser/extensions/blacklist.h"
 #include "chrome/browser/extensions/blacklist_state_fetcher.h"
@@ skipping 106 matching lines @@
   // Safe browsing blacklist empty, now enabled.
   EXPECT_EQ(NOT_BLACKLISTED, tester.GetBlacklistState(a));
 }
 
 // Tests that Blacklist clears the old prefs blacklist on startup.
 TEST_F(BlacklistTest, ClearsPreferencesBlacklist) {
   std::string a = AddExtension("a");
   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 extensions:: prefix here is
unnecessary.

[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());
 
   Blacklist blacklist(prefs());
   TestBlacklist tester(&blacklist);
 
   // Blacklist has been cleared. Only the installed extension "a" left.
   EXPECT_EQ(Set(a), prefs()->GetBlacklistedExtensions());
   EXPECT_TRUE(prefs()->GetInstalledExtensionInfo(a).get());
   EXPECT_TRUE(prefs()->GetInstalledExtensionInfo(b).get());
@@ skipping 108 matching lines @@
   base::RunLoop().RunUntilIdle();
 
   // No new fetchers.
   EXPECT_FALSE(fetcher_tester.HandleFetcher(2));
   EXPECT_EQ(BLACKLISTED_CWS_POLICY_VIOLATION, cached_states[a]);
   EXPECT_EQ(BLACKLISTED_POTENTIALLY_UNWANTED, cached_states[b]);
   EXPECT_EQ(0U, cached_states.count(c));
 }
 
 }  // namespace extensions