Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1234)

Unified Diff: chrome/browser/engagement/site_engagement_service.cc

Issue 1465363002: [Storage] Android - ManageSpace UI, Important Origins, and CBD Dialog (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Android Java Hookup Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 e97cade234a0d939cc5cef30e2885094feef7bd9..752ea4c448ee839e3e353340da22dc3ad4459ff3 100644
--- a/chrome/browser/engagement/site_engagement_service.cc
+++ b/chrome/browser/engagement/site_engagement_service.cc
@@ -376,22 +376,7 @@ SiteEngagementService* SiteEngagementService::Get(Profile* profile) {
// static
bool SiteEngagementService::IsEnabled() {
- // If the engagement service or any of its dependencies are force-enabled,
- // return true immediately.
- if (base::CommandLine::ForCurrentProcess()->HasSwitch(
- switches::kEnableSiteEngagementService) ||
- SiteEngagementEvictionPolicy::IsEnabled() ||
- AppBannerSettingsHelper::ShouldUseSiteEngagementScore()) {
- return true;
- }
-
- if (base::CommandLine::ForCurrentProcess()->HasSwitch(
- switches::kDisableSiteEngagementService)) {
- return false;
- }
- const std::string group_name =
- base::FieldTrialList::FindFullName(kEngagementParams);
- return base::StartsWith(group_name, "Enabled", base::CompareCase::SENSITIVE);
+ return true;
}
SiteEngagementService::SiteEngagementService(Profile* profile)

Powered by Google App Engine
This is Rietveld 408576698