Enable SiteEngagementEvictionPolicy behind --enable-site-eviction-policy flag.

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/content_settings/host_content_settings_map_factory.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/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 {
 
 // Delta within which to consider scores equal.
@@ skipping 126 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) ||

[raymes, 2015/09/24 01:55:30]

Hmm - should we just get rid of this switch? Do you see a good use for it? Is it
just for the chrome:// page?

[calamity, 2015/10/26 03:01:57]

We definitely want this switch around for experiments. I was thinking running
with control, engagement service only, and engagement service with eviction
groups. That would give us a better idea of the base cost of the engagement
service.

This is already done in the rebase actually.

+         SiteEngagementEvictionPolicy::IsEnabled();
 }
 
 SiteEngagementService::SiteEngagementService(Profile* profile)
     : profile_(profile) {
 }
 
 SiteEngagementService::~SiteEngagementService() {
 }
 
 void SiteEngagementService::HandleNavigation(const GURL& url) {
@@ skipping 38 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;
 }