Rename SiteEngagementService::ResetScoreForURL.

chrome/browser/engagement/important_sites_util_unittest.cc

 // Copyright 2016 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 "chrome/browser/engagement/important_sites_util.h"
 
 #include <memory>
 #include <utility>
 
 #include "base/files/scoped_temp_dir.h"
@@ skipping 136 matching lines @@
   ASSERT_TRUE(service);
 
   GURL url1("http://www.google.com/");
   GURL url2("https://www.google.com/");
   GURL url3("https://drive.google.com/");
   GURL url4("https://www.chrome.com/");
   GURL url5("https://www.example.com/");
   GURL url6("https://youtube.com/");
   GURL url7("https://foo.bar/");
 
-  service->ResetScoreForURL(url1, 5);
-  service->ResetScoreForURL(url2, 2);  // Below medium engagement (5).
-  service->ResetScoreForURL(url3, 7);
-  service->ResetScoreForURL(url4, 8);
-  service->ResetScoreForURL(url5, 9);
-  service->ResetScoreForURL(url6, 1);  // Below the medium engagement (5).
-  service->ResetScoreForURL(url7, 11);
+  service->ResetBaseScoreForURL(url1, 5);
+  service->ResetBaseScoreForURL(url2, 2);  // Below medium engagement (5).
+  service->ResetBaseScoreForURL(url3, 7);
+  service->ResetBaseScoreForURL(url4, 8);
+  service->ResetBaseScoreForURL(url5, 9);
+  service->ResetBaseScoreForURL(url6, 1);  // Below the medium engagement (5).
+  service->ResetBaseScoreForURL(url7, 11);
 
   // Here we should have:
   // 1: removed domains below minimum engagement,
   // 2: combined the google.com entries, and
   // 3: sorted by the score.
   std::vector<ImportantDomainInfo> important_sites =
       ImportantSitesUtil::GetImportantRegisterableDomains(profile(),
                                                           kNumImportantSites);
   std::vector<std::string> expected_sorted_domains = {
       "foo.bar", "example.com", "chrome.com", "google.com"};
@@ skipping 58 matching lines @@
   // Add the rest, which should put us over the limit.
   AddBookmark(url6);
   AddBookmark(url7);
   // Too many bookmarks! Nothing shows up now.
   important_sites = ImportantSitesUtil::GetImportantRegisterableDomains(
       profile(), kNumImportantSites);
   EXPECT_EQ(0u, important_sites.size());
 
   // If we add some site engagement, they should show up (even though the site
   // engagement score is too low for a signal by itself).
-  service->ResetScoreForURL(url1, 2);
-  service->ResetScoreForURL(url4, 3);
-  service->ResetScoreForURL(url7, 0);
+  service->ResetBaseScoreForURL(url1, 2);
+  service->ResetBaseScoreForURL(url4, 3);
+  service->ResetBaseScoreForURL(url7, 0);
 
   important_sites = ImportantSitesUtil::GetImportantRegisterableDomains(
       profile(), kNumImportantSites);
   ASSERT_EQ(2u, important_sites.size());
   std::vector<std::string> expected_sorted_domains = {"google.com",
                                                       "chrome.com"};
   std::vector<GURL> expected_sorted_origins = {url1, url4};
   ExpectImportantResultsEqualUnordered(
       expected_sorted_domains, expected_sorted_origins, important_sites);
 }
 
 TEST_F(ImportantSitesUtilTest, Blacklisting) {
   SiteEngagementService* service = SiteEngagementService::Get(profile());
   ASSERT_TRUE(service);
 
   GURL url1("http://www.google.com/");
   GURL url2("http://www.gmail.com/");
 
   // Set a bunch of positive signals.
-  service->ResetScoreForURL(url1, 5);
+  service->ResetBaseScoreForURL(url1, 5);
   AddBookmark(url2);
   AddContentSetting(CONTENT_SETTINGS_TYPE_NOTIFICATIONS, CONTENT_SETTING_ALLOW,
                     url1);
 
   // Important fetch 1.
   std::vector<ImportantDomainInfo> important_sites =
       ImportantSitesUtil::GetImportantRegisterableDomains(profile(),
                                                           kNumImportantSites);
   std::vector<std::string> expected_sorted_domains = {"google.com",
                                                       "gmail.com"};
@@ skipping 34 matching lines @@
 }
 
 TEST_F(ImportantSitesUtilTest, BlacklistingReset) {
   SiteEngagementService* service = SiteEngagementService::Get(profile());
   ASSERT_TRUE(service);
 
   GURL url1("http://www.google.com/");
   GURL url2("http://www.gmail.com/");
 
   // Set a bunch of positive signals.
-  service->ResetScoreForURL(url1, 5);
+  service->ResetBaseScoreForURL(url1, 5);
   AddBookmark(url2);
   AddContentSetting(CONTENT_SETTINGS_TYPE_NOTIFICATIONS, CONTENT_SETTING_ALLOW,
                     url1);
 
   // Important fetch 1.
   std::vector<ImportantDomainInfo> important_sites =
       ImportantSitesUtil::GetImportantRegisterableDomains(profile(),
                                                           kNumImportantSites);
   ASSERT_EQ(2u, important_sites.size());
   // Record ignore twice.
@@ skipping 48 matching lines @@
   ExpectImportantResultsEq(expected_sorted_domains, expected_sorted_origins,
                            important_sites);
 }
 
 TEST_F(ImportantSitesUtilTest, Metrics) {
   SiteEngagementService* service = SiteEngagementService::Get(profile());
   ASSERT_TRUE(service);
   base::HistogramTester histogram_tester;
 
   GURL url1("http://www.google.com/");
-  service->ResetScoreForURL(url1, 5);
+  service->ResetBaseScoreForURL(url1, 5);
   AddContentSetting(CONTENT_SETTINGS_TYPE_NOTIFICATIONS, CONTENT_SETTING_ALLOW,
                     url1);
 
   GURL url2("http://www.youtube.com/");
   AddBookmark(url2);
 
   GURL url3("http://www.bad.com/");
   AddBookmark(url3);
 
   std::vector<ImportantDomainInfo> important_sites =
@@ skipping 24 matching lines @@
 }
 
 TEST_F(ImportantSitesUtilTest, DialogBlacklisting) {
   SiteEngagementService* service = SiteEngagementService::Get(profile());
   ASSERT_TRUE(service);
 
   GURL url1("http://www.google.com/");
   GURL url2("http://www.yahoo.com/");
 
   // Set a bunch of positive signals.
-  service->ResetScoreForURL(url2, 5);
+  service->ResetBaseScoreForURL(url2, 5);
   AddBookmark(url1);
   AddContentSetting(CONTENT_SETTINGS_TYPE_NOTIFICATIONS, CONTENT_SETTING_ALLOW,
                     url1);
 
   // Start off not disabled.
   EXPECT_FALSE(ImportantSitesUtil::IsDialogDisabled(profile()));
 
   // Important fetch 1.
   std::vector<ImportantDomainInfo> important_sites =
       ImportantSitesUtil::GetImportantRegisterableDomains(profile(),
@@ skipping 25 matching lines @@
 
   // Items should still be present.
   important_sites = ImportantSitesUtil::GetImportantRegisterableDomains(
       profile(), kNumImportantSites);
   ExpectImportantResultsEq(expected_sorted_domains, expected_sorted_origins,
                            important_sites);
 
   // Dialog should be blacklisted.
   EXPECT_TRUE(ImportantSitesUtil::IsDialogDisabled(profile()));
 }