ImportantSites: Rename function to be more precise.

I will be adding a function to ImportantSitesUtil that populates
an info map with entries for each origin that have an installed
app, so I am renaming this function to represent the fact that
it only populates the map with installed origins that have had
user engagement within the specified time period.

Bug: 1095732
Change-Id: I370e89b100b67d922d562ab540e4a4e1405f67eb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2245674
Commit-Queue: Jarryd Goodman <jarrydg@chromium.org>
Reviewed-by: Dominick Ng <dominickn@chromium.org>
Cr-Commit-Position: refs/heads/master@{#778989}

chrome/browser/engagement/important_sites_util.cc

@@ -366,7 +366,7 @@ void PopulateInfoMapWithBookmarks(
 // used to warn about clearing data for installed apps can be excluded from the
 // Android build.
 #if !defined(OS_ANDROID)
-void PopulateInfoMapWithInstalled(
+void PopulateInfoMapWithInstalledEngagedInTimePeriod(
     browsing_data::TimePeriod time_period,
     Profile* profile,
     std::map<std::string, ImportantDomainInfo>* output) {
@@ -490,7 +490,8 @@ ImportantSitesUtil::GetInstalledRegisterableDomains(
     size_t max_results) {
   std::vector<ImportantDomainInfo> installed_domains;
   std::map<std::string, ImportantDomainInfo> installed_app_info;
-  PopulateInfoMapWithInstalled(time_period, profile, &installed_app_info);
+  PopulateInfoMapWithInstalledEngagedInTimePeriod(time_period, profile,
+                                                  &installed_app_info);
 
   std::unordered_set<std::string> excluded_domains =
       GetBlacklistedImportantDomains(profile);