Use the site engagement eviction policy for temporary storage eviction.

chrome/browser/engagement/site_engagement_service.cc

 // Copyright 2015 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/site_engagement_service.h"
 
 #include <algorithm>
 #include <cmath>
 
 #include "base/command_line.h"
 #include "base/time/clock.h"
 #include "base/values.h"
+#include "chrome/browser/engagement/site_engagement_eviction_policy.h"
 #include "chrome/browser/engagement/site_engagement_helper.h"
 #include "chrome/browser/engagement/site_engagement_service_factory.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/chrome_switches.h"
 #include "components/content_settings/core/browser/host_content_settings_map.h"
 #include "components/content_settings/core/common/content_settings_pattern.h"
 #include "url/gurl.h"
 
 namespace {
 
@@ skipping 127 matching lines @@
 }
 
 // static
 SiteEngagementService* SiteEngagementService::Get(Profile* profile) {
   return SiteEngagementServiceFactory::GetForProfile(profile);
 }
 
 // static
 bool SiteEngagementService::IsEnabled() {
   return base::CommandLine::ForCurrentProcess()->HasSwitch(
-      switches::kEnableSiteEngagementService);
+             switches::kEnableSiteEngagementService) ||
+         SiteEngagementEvictionPolicy::IsEnabled();
 }
 
 SiteEngagementService::SiteEngagementService(Profile* profile)
-    : profile_(profile) {
-}
+    : profile_(profile),
+      site_engagement_eviction_policy_(
+          new SiteEngagementEvictionPolicy(profile)) {}

[raymes, 2015/09/01 06:52:52]

Should SiteEngagementService depend on SiteEngagementEvictionPolicy (which
already depends on it)?

[calamity, 2015/09/04 05:44:21]

Forgot to put in the enable condition.

What do you mean by this? The eviction policy needs to live somewhere and this
seemed like a decent place. We could also add a new keyed service for it I
guess?

 
 SiteEngagementService::~SiteEngagementService() {
 }
 
 void SiteEngagementService::HandleNavigation(const GURL& url) {
   HostContentSettingsMap* settings_map = profile_->GetHostContentSettingsMap();
   scoped_ptr<base::DictionaryValue> score_dict =
       GetScoreDictForOrigin(settings_map, url);
   SiteEngagementScore score(&clock_, *score_dict);
 
@@ skipping 30 matching lines @@
     if (!origin.is_valid())
       continue;
 
     scoped_ptr<base::DictionaryValue> score_dict =
         GetScoreDictForOrigin(settings_map, origin);
     SiteEngagementScore score(&clock_, *score_dict);
     total_score += score.Score();
   }
   return total_score;
 }