Add RecentTabsPageTest

chrome/browser/android/recently_closed_tabs_bridge.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/android/recently_closed_tabs_bridge.h"
 
 #include "base/android/jni_string.h"
 #include "chrome/browser/android/tab_android.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_android.h"
@@ skipping 35 matching lines @@
       auto& tab = static_cast<const sessions::TabRestoreService::Tab&>(*entry);
       AddTabToList(env, tab, jtabs_list);
       if (++added_count == max_tab_count)
         break;
     }
   }
 }
 
 }  // namespace
 
-RecentlyClosedTabsBridge::RecentlyClosedTabsBridge(Profile* profile)
-    : profile_(profile),
-      tab_restore_service_(NULL) {
-}
+RecentlyClosedTabsBridge::RecentlyClosedTabsBridge(
+    const JavaParamRef<jobject>& bridge,
+    Profile* profile)
+    : bridge_(AttachCurrentThread(), bridge),

[Bernhard Bauer, 2017/01/04 11:54:16]

Nit: I think what I would do is create the ScopedJavaGlobalRef at the call site
and then pass that in.

[Michael van Ouwerkerk, 2017/01/05 11:02:14]

Done.

+      profile_(profile),
+      tab_restore_service_(NULL) {}
 
 RecentlyClosedTabsBridge::~RecentlyClosedTabsBridge() {
   if (tab_restore_service_)
     tab_restore_service_->RemoveObserver(this);
 }
 
 void RecentlyClosedTabsBridge::Destroy(JNIEnv* env,
                                        const JavaParamRef<jobject>& obj) {
   delete this;
 }
 
-void RecentlyClosedTabsBridge::SetRecentlyClosedCallback(
-    JNIEnv* env,
-    const JavaParamRef<jobject>& obj,
-    const JavaParamRef<jobject>& jcallback) {
-  callback_.Reset(env, jcallback);
-}
-
 jboolean RecentlyClosedTabsBridge::GetRecentlyClosedTabs(
     JNIEnv* env,
     const JavaParamRef<jobject>& obj,
     const JavaParamRef<jobject>& jtabs_list,
     jint max_tab_count) {
   EnsureTabRestoreService();
   if (!tab_restore_service_)
     return false;
 
   AddTabsToList(env, tab_restore_service_->entries(), jtabs_list,
@@ skipping 57 matching lines @@
 void RecentlyClosedTabsBridge::ClearRecentlyClosedTabs(
     JNIEnv* env,
     const JavaParamRef<jobject>& obj) {
   EnsureTabRestoreService();
   if (tab_restore_service_)
     tab_restore_service_->ClearEntries();
 }
 
 void RecentlyClosedTabsBridge::TabRestoreServiceChanged(
     sessions::TabRestoreService* service) {
-  if (callback_.is_null())
-    return;
-  JNIEnv* env = AttachCurrentThread();
-  Java_RecentlyClosedCallback_onUpdated(env, callback_);
+  Java_RecentlyClosedBridge_onUpdated(AttachCurrentThread(), bridge_);
 }
 
 void RecentlyClosedTabsBridge::TabRestoreServiceDestroyed(
     sessions::TabRestoreService* service) {
   tab_restore_service_ = NULL;
 }
 
 void RecentlyClosedTabsBridge::EnsureTabRestoreService() {
   if (tab_restore_service_)
     return;
 
   tab_restore_service_ = TabRestoreServiceFactory::GetForProfile(profile_);
 
   // TabRestoreServiceFactory::GetForProfile() can return NULL (e.g. in
   // incognito mode).
   if (tab_restore_service_) {
     // This does nothing if the tabs have already been loaded or they
     // shouldn't be loaded.
     tab_restore_service_->LoadTabsFromLastSession();
     tab_restore_service_->AddObserver(this);
   }
 }
 
 static jlong Init(JNIEnv* env,
                   const JavaParamRef<jobject>& obj,
                   const JavaParamRef<jobject>& jprofile) {
   RecentlyClosedTabsBridge* bridge = new RecentlyClosedTabsBridge(
-      ProfileAndroid::FromProfileAndroid(jprofile));
+      obj, ProfileAndroid::FromProfileAndroid(jprofile));
   return reinterpret_cast<intptr_t>(bridge);
 }
 
 // static
 bool RecentlyClosedTabsBridge::Register(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }