🏠 Add instrumentation test for Suggestions BottomSheet

chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetsBridge.java

Index: chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetsBridge.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetsBridge.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetsBridge.java
index 716439a01c45e81f683c79af193d308c727600a5..eb97251ce714bfc7ed6bb2536de048062da272b7 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetsBridge.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetsBridge.java
@@ -8,6 +8,7 @@
 
 import org.chromium.base.Callback;
 import org.chromium.base.annotations.CalledByNative;
+import org.chromium.chrome.browser.ntp.NewTabPage.DestructionObserver;
 import org.chromium.chrome.browser.ntp.NewTabPageUma;
 import org.chromium.chrome.browser.ntp.cards.ActionItem;
 import org.chromium.chrome.browser.ntp.cards.SuggestionsCategoryInfo;
@@ -22,7 +23,8 @@
 /**
  * Provides access to the snippets to display on the NTP using the C++ ContentSuggestionsService.
  */
-public class SnippetsBridge implements SuggestionsSource, SuggestionsMetricsReporter {
+public class SnippetsBridge
+        implements SuggestionsSource, SuggestionsMetricsReporter, DestructionObserver {

[Michael van Ouwerkerk, 2017/02/07 11:12:41]

Maybe SuggestionsSource could be a DestructionObserver? Also fine for a later
CL.

     private static final String TAG = "SnippetsBridge";
 
     private long mNativeSnippetsBridge;
@@ -63,7 +65,8 @@ public SnippetsBridge(Profile profile) {
      * Destroys the native bridge. This object can no longer be used to send native commands, and
      * any observer is nulled out and will stop receiving updates. This object should be discarded.
      */
-    public void destroy() {
+    @Override
+    public void onDestroy() {
         assert mNativeSnippetsBridge != 0;
         nativeDestroy(mNativeSnippetsBridge);
         mNativeSnippetsBridge = 0;