Remove androix.navigation dependence from :service module (#1668)

* Remove androix.navigation-fragment dependence

* Replace the heavier androidx.preference with androidx.fragment which is what's needed

* Remove unused dependencies from libs.version.toml

* Revert reflection based usage of NavHost

Author: surbhiia

gradle/libs.versions.toml

@@ -32,7 +32,6 @@ androidx-navigation-runtime-ktx = { group = "androidx.navigation", name = "navig
 androidx-core = "androidx.core:core:1.18.0"
 androidx-annotation = "androidx.annotation:annotation:1.9.1"
 androidx-lifecycle-process = "androidx.lifecycle:lifecycle-process:2.9.4"
-androidx-preference-ktx = "androidx.preference:preference-ktx:1.2.1"
 androidx-fragment = { module = "androidx.fragment:fragment", version.ref="fragment" }
 
 findbugs-jsr305 = "com.google.code.findbugs:jsr305:3.0.2"

services/build.gradle.kts

@@ -19,7 +19,7 @@ dependencies {
     implementation(libs.androidx.core)
     implementation(libs.androidx.navigation.runtime.ktx)
     implementation(libs.androidx.lifecycle.process)
-    implementation(libs.androidx.preference.ktx)
+    implementation(libs.androidx.fragment)
 
     testImplementation(libs.androidx.test.core)
     testImplementation(libs.robolectric)

services/src/test/java/io/opentelemetry/android/internal/services/visiblescreen/VisibleScreenTrackerTest.kt

@@ -10,6 +10,7 @@ import android.app.Application
 import android.app.Application.ActivityLifecycleCallbacks
 import androidx.fragment.app.DialogFragment
 import androidx.fragment.app.Fragment
+import androidx.navigation.NavHost
 import androidx.navigation.fragment.NavHostFragment
 import io.mockk.impl.annotations.RelaxedMockK
 import io.mockk.junit5.MockKExtension
@@ -115,6 +116,29 @@ internal class VisibleScreenTrackerTest {
         )
     }
 
+    @Test
+    fun fragmentLifecycle_customNavHostIgnored() {
+        val visibleScreenTracker = this.visibleScreenService
+        val fragment = mockk<Fragment>()
+        val customNavHost = mockk<CustomNavHostFragment>()
+
+        visibleScreenTracker.fragmentResumed(fragment)
+        visibleScreenTracker.fragmentResumed(customNavHost)
+        assertEquals(
+            fragment.javaClass.simpleName,
+            visibleScreenTracker.currentlyVisibleScreen,
+        )
+        assertNull(visibleScreenTracker.previouslyVisibleScreen)
+
+        visibleScreenTracker.fragmentPaused(customNavHost)
+        visibleScreenTracker.fragmentPaused(fragment)
+        assertEquals("unknown", visibleScreenTracker.currentlyVisibleScreen)
+        assertEquals(
+            fragment.javaClass.simpleName,
+            visibleScreenTracker.previouslyVisibleScreen,
+        )
+    }
+
     @Test
     fun fragmentLifecycle_dialogFragment() {
         val visibleScreenTracker = this.visibleScreenService
@@ -174,4 +198,6 @@ internal class VisibleScreenTrackerTest {
 
     private val visibleScreenService: VisibleScreenTracker
         get() = VisibleScreenTrackerImpl(application)
+
+    private abstract class CustomNavHostFragment : Fragment(), NavHost
 }