Chromium Code Reviews| 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 7b9a5d936d14055a368380a74ae84d31b0631363..35c005d1af69eefa392635567bbcf358f053b12c 100644 |
| --- a/chrome/browser/engagement/site_engagement_service.h |
| +++ b/chrome/browser/engagement/site_engagement_service.h |
| @@ -26,6 +26,11 @@ namespace base { |
| class Clock; |
| } |
| +namespace banners { |
| +FORWARD_DECLARE_TEST(AppBannerManagerBrowserTest, |
|
benwells
2017/03/30 08:22:37
Is this required becaues the test is in a namespac
dominickn
2017/03/30 23:42:52
Yes, unfortunately.
|
| + CheckOnLoadWithoutSufficientEngagement); |
| +} |
| + |
| namespace content { |
| class WebContents; |
| } |
| @@ -173,6 +178,8 @@ class SiteEngagementService : public KeyedService, |
| FRIEND_TEST_ALL_PREFIXES(SiteEngagementServiceTest, |
| IncognitoEngagementService); |
| FRIEND_TEST_ALL_PREFIXES(SiteEngagementServiceTest, GetScoreFromSettings); |
| + FRIEND_TEST_ALL_PREFIXES(banners::AppBannerManagerBrowserTest, |
| + CheckOnLoadWithoutSufficientEngagement); |
| FRIEND_TEST_ALL_PREFIXES(AppBannerSettingsHelperTest, SiteEngagementTrigger); |
| #if defined(OS_ANDROID) |