Use the site engagement eviction policy for temporary storage eviction.

chrome/browser/engagement/site_engagement_service.cc

Index: chrome/browser/engagement/site_engagement_service.cc
diff --git a/chrome/browser/engagement/site_engagement_service.cc b/chrome/browser/engagement/site_engagement_service.cc
index 5f36f19ba525753cb54e04c913119451d858a8fa..79c9ed2e847418aaf0e1bd4f4e58f8db45fffbd6 100644
--- a/chrome/browser/engagement/site_engagement_service.cc
+++ b/chrome/browser/engagement/site_engagement_service.cc
@@ -10,6 +10,7 @@
 #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"
@@ -157,12 +158,14 @@ SiteEngagementService* SiteEngagementService::Get(Profile* 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() {
 }