For mozilla-mobile#15265 - Save cached top sites for metrics ping

ekager · Oct 12, 2020 · 9817a9b · 9817a9b
1 parent ee5f678
commit 9817a9b
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 15 deletions.
diff --git a/app/src/main/java/org/mozilla/fenix/HomeActivity.kt b/app/src/main/java/org/mozilla/fenix/HomeActivity.kt
@@ -326,6 +326,8 @@ open class HomeActivity : LocaleAwareAppCompatActivity(), NavHostActivity {
             window.addFlags(WindowManager.LayoutParams.FLAG_SECURE)
         }
 
+        settings().topSitesSize = components.core.topSitesStorage.cachedTopSites.size
+
         super.onPause()
 
         // Diagnostic breadcrumb for "Display already aquired" crash:

diff --git a/app/src/main/java/org/mozilla/fenix/browser/UriOpenedObserver.kt b/app/src/main/java/org/mozilla/fenix/browser/UriOpenedObserver.kt
@@ -46,7 +46,7 @@ class UriOpenedObserver(
     }
 
     private fun saveOpenTabsCount() {
-        settings.setOpenTabsCount(sessionManager.sessionsOfType(private = false).count())
+        settings.openTabsCount = sessionManager.sessionsOfType(private = false).count()
     }
 
     override fun onAllSessionsRemoved() {

diff --git a/app/src/main/java/org/mozilla/fenix/utils/Settings.kt b/app/src/main/java/org/mozilla/fenix/utils/Settings.kt
@@ -914,7 +914,7 @@ class Settings(private val appContext: Context) : PreferencesHolder {
         return overrideAmoUser.isNotEmpty() || overrideAmoCollection.isNotEmpty()
     }
 
-    val topSitesSize by intPreference(
+    var topSitesSize by intPreference(
         appContext.getPreferenceKey(R.string.pref_key_top_sites_size),
         default = 0
     )
@@ -924,18 +924,10 @@ class Settings(private val appContext: Context) : PreferencesHolder {
         default = topSitesMaxCount
     )
 
-    fun setOpenTabsCount(count: Int) {
-        preferences.edit().putInt(
-            appContext.getPreferenceKey(R.string.pref_key_open_tabs_count),
-            count
-        ).apply()
-    }
-
-    val openTabsCount: Int
-        get() = preferences.getInt(
-            appContext.getPreferenceKey(R.string.pref_key_open_tabs_count),
-            0
-        )
+    var openTabsCount by intPreference(
+        appContext.getPreferenceKey(R.string.pref_key_open_tabs_count),
+        0
+    )
 
     private var savedLoginsSortingStrategyString by stringPreference(
         appContext.getPreferenceKey(R.string.pref_key_saved_logins_sorting_strategy),

diff --git a/app/src/test/java/org/mozilla/fenix/browser/UriOpenedObserverTest.kt b/app/src/test/java/org/mozilla/fenix/browser/UriOpenedObserverTest.kt
@@ -59,7 +59,7 @@ class UriOpenedObserverTest {
         observer.onSessionRemoved(session)
         verify { session.unregister(observer.singleSessionObserver) }
 
-        verify { settings.setOpenTabsCount(1) }
+        verify { settings.openTabsCount = 1 }
     }
 
     @Test