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

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

Issue 1757163002: [ImportantSites] JNI bindings for CBD filtering and Important Sites. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added ImportantSitesUtil, cleaned up Created 4 years, 8 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.h
diff --git a/chrome/browser/engagement/site_engagement_service.h b/chrome/browser/engagement/site_engagement_service.h
index 4cfdc78a6f7d6891be8c25f2a1993f5be06fe73c..f520fc78fb7395136164c03523e1d046e2bebbee 100644
--- a/chrome/browser/engagement/site_engagement_service.h
+++ b/chrome/browser/engagement/site_engagement_service.h
@@ -127,6 +127,7 @@ class SiteEngagementScore {
friend class SiteEngagementHelperTest;
friend class SiteEngagementScoreTest;
friend class SiteEngagementServiceTest;
+ friend class ImportantSitesUtilTest;
// Array holding the values corresponding to each item in Variation array.
static double param_values[];
@@ -276,6 +277,7 @@ class SiteEngagementService : public KeyedService,
FRIEND_TEST_ALL_PREFIXES(SiteEngagementServiceTest, EngagementLevel);
FRIEND_TEST_ALL_PREFIXES(SiteEngagementServiceTest, ScoreDecayHistograms);
FRIEND_TEST_ALL_PREFIXES(AppBannerSettingsHelperTest, SiteEngagementTrigger);
+ FRIEND_TEST_ALL_PREFIXES(ImportantSitesUtilTest, NotificationsThenEngagement);
// Only used in tests.
SiteEngagementService(Profile* profile, scoped_ptr<base::Clock> clock);

Powered by Google App Engine
This is Rietveld 408576698