📰 Move metrics and scheduling events out of SnippetsBridge

chrome/android/java/src/org/chromium/chrome/browser/suggestions/SuggestionsBottomSheetContent.java

Index: chrome/android/java/src/org/chromium/chrome/browser/suggestions/SuggestionsBottomSheetContent.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/suggestions/SuggestionsBottomSheetContent.java b/chrome/android/java/src/org/chromium/chrome/browser/suggestions/SuggestionsBottomSheetContent.java
index 28a430755c19f9a7cbe9600de05106d49e1f319c..9fd7a66cd553682937ae7d8fcb75cb6fea6ab811 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/suggestions/SuggestionsBottomSheetContent.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/suggestions/SuggestionsBottomSheetContent.java
@@ -41,13 +41,13 @@
  */
 public class SuggestionsBottomSheetContent implements BottomSheet.BottomSheetContent {
     private static SuggestionsSource sSuggestionsSourceForTesting;
-    private static SuggestionsMetricsReporter sMetricsReporterForTesting;
+    private static SuggestionsEventReporter sEventReporterForTesting;
 
     private final View mView;
     private final FadingShadowView mShadowView;
     private final SuggestionsRecyclerView mRecyclerView;
     private final ContextMenuManager mContextMenuManager;
-    private final SuggestionsUiDelegateImpl mSuggestionsManager;
+    private final SuggestionsUiDelegateImpl mSuggestionsUiDelegate;
     private final TileGroup.Delegate mTileGroupDelegate;
     private final BottomSheet mBottomSheet;
     private final BottomSheetObserver mBottomSheetObserver;
@@ -59,7 +59,7 @@ public SuggestionsBottomSheetContent(final ChromeActivity activity, final Bottom
                 new SuggestionsNavigationDelegateImpl(activity, profile, sheet, tabModelSelector);
         mTileGroupDelegate = new TileGroupDelegateImpl(
                 activity, profile, tabModelSelector, navigationDelegate, snackbarManager);
-        mSuggestionsManager = createSuggestionsDelegate(profile, navigationDelegate, sheet);
+        mSuggestionsUiDelegate = createSuggestionsDelegate(profile, navigationDelegate, sheet);
 
         mView = LayoutInflater.from(activity).inflate(
                 R.layout.suggestions_bottom_sheet_content, null);
@@ -74,7 +74,7 @@ public void setTouchEnabled(boolean enabled) {
         mContextMenuManager =
                 new ContextMenuManager(activity, navigationDelegate, touchEnabledDelegate);
         activity.getWindowAndroid().addContextMenuCloseListener(mContextMenuManager);
-        mSuggestionsManager.addDestructionObserver(new DestructionObserver() {
+        mSuggestionsUiDelegate.addDestructionObserver(new DestructionObserver() {
             @Override
             public void onDestroy() {
                 activity.getWindowAndroid().removeContextMenuCloseListener(mContextMenuManager);
@@ -83,12 +83,11 @@ public void onDestroy() {
 
         UiConfig uiConfig = new UiConfig(mRecyclerView);
 
-        final NewTabPageAdapter adapter = new NewTabPageAdapter(mSuggestionsManager,
+        final NewTabPageAdapter adapter = new NewTabPageAdapter(mSuggestionsUiDelegate,
                 /* aboveTheFoldView = */ null, uiConfig, OfflinePageBridge.getForProfile(profile),
                 mContextMenuManager, mTileGroupDelegate);
         mRecyclerView.init(uiConfig, mContextMenuManager, adapter);
 
-        final SuggestionsSource suggestionsSource = mSuggestionsManager.getSuggestionsSource();
         mBottomSheetObserver = new EmptyBottomSheetObserver() {
             @Override
             public void onSheetOpened() {
@@ -97,13 +96,13 @@ public void onSheetOpened() {
                 // TODO(https://crbug.com/689962) Ensure this call does not discard all suggestions
                 // every time the sheet is opened.
                 adapter.refreshSuggestions();
-                suggestionsSource.onNtpInitialized();
+                mSuggestionsUiDelegate.getEventReporter().onSurfaceOpened();
             }
         };
         mBottomSheet = activity.getBottomSheet();
         mBottomSheet.addObserver(mBottomSheetObserver);
         adapter.refreshSuggestions();
-        suggestionsSource.onNtpInitialized();
+        mSuggestionsUiDelegate.getEventReporter().onSurfaceOpened();
 
         mShadowView = (FadingShadowView) mView.findViewById(R.id.shadow);
         mShadowView.init(
@@ -156,7 +155,7 @@ public int getVerticalScrollOffset() {
     @Override
     public void destroy() {
         mBottomSheet.removeObserver(mBottomSheetObserver);
-        mSuggestionsManager.onDestroy();
+        mSuggestionsUiDelegate.onDestroy();
         mTileGroupDelegate.destroy();
     }
 
@@ -169,15 +168,15 @@ public static void setSuggestionsSourceForTesting(SuggestionsSource suggestionsS
         sSuggestionsSourceForTesting = suggestionsSource;
     }
 
-    public static void setMetricsReporterForTesting(SuggestionsMetricsReporter metricsReporter) {
-        sMetricsReporterForTesting = metricsReporter;
+    public static void setEventReporterForTesting(SuggestionsEventReporter eventReporter) {
+        sEventReporterForTesting = eventReporter;
     }
 
     private static SuggestionsUiDelegateImpl createSuggestionsDelegate(Profile profile,
             SuggestionsNavigationDelegate navigationDelegate, NativePageHost host) {
         SnippetsBridge snippetsBridge = null;
         SuggestionsSource suggestionsSource;
-        SuggestionsMetricsReporter metricsReporter;
+        SuggestionsEventReporter eventReporter;
 
         if (sSuggestionsSourceForTesting == null) {
             snippetsBridge = new SnippetsBridge(profile);
@@ -186,15 +185,14 @@ private static SuggestionsUiDelegateImpl createSuggestionsDelegate(Profile profi
             suggestionsSource = sSuggestionsSourceForTesting;
         }
 
-        if (sMetricsReporterForTesting == null) {
-            if (snippetsBridge == null) snippetsBridge = new SnippetsBridge(profile);
-            metricsReporter = snippetsBridge;
+        if (sEventReporterForTesting == null) {
+            eventReporter = new SuggestionsEventReporterBridge();
         } else {
-            metricsReporter = sMetricsReporterForTesting;
+            eventReporter = sEventReporterForTesting;
         }
 
         SuggestionsUiDelegateImpl delegate = new SuggestionsUiDelegateImpl(
-                suggestionsSource, metricsReporter, navigationDelegate, profile, host);
+                suggestionsSource, eventReporter, navigationDelegate, profile, host);
         if (snippetsBridge != null) delegate.addDestructionObserver(snippetsBridge);
 
         return delegate;