Skip to content

Commit 1bf8120

Browse files
committed
Switch to a single button
1 parent e659596 commit 1bf8120

File tree

4 files changed

+12
-31
lines changed

4 files changed

+12
-31
lines changed

firebase-sessions/test-app/src/main/kotlin/com/google/firebase/testing/sessions/FirstFragment.kt

Lines changed: 10 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -26,19 +26,20 @@ import android.view.LayoutInflater
2626
import android.view.View
2727
import android.view.ViewGroup
2828
import androidx.fragment.app.Fragment
29+
import androidx.lifecycle.lifecycleScope
2930
import com.google.firebase.crashlytics.FirebaseCrashlytics
3031
import com.google.firebase.perf.FirebasePerformance
31-
import com.google.firebase.perf.metrics.Trace as PerfTrace
3232
import com.google.firebase.testing.sessions.databinding.FragmentFirstBinding
33+
import kotlinx.coroutines.Dispatchers
3334
import java.util.Date
3435
import java.util.Locale
36+
import kotlinx.coroutines.launch
37+
import kotlinx.coroutines.delay
3538

3639
/** A simple [Fragment] subclass as the default destination in the navigation. */
3740
class FirstFragment : Fragment() {
3841
val crashlytics = FirebaseCrashlytics.getInstance()
3942
val performance = FirebasePerformance.getInstance()
40-
var performanceTrace: PerfTrace? = null
41-
var performanceTraceIdenticalName: PerfTrace? = null
4243

4344
private var _binding: FragmentFirstBinding? = null
4445

@@ -70,16 +71,12 @@ class FirstFragment : Fragment() {
7071
}
7172
}
7273
binding.createTrace.setOnClickListener {
73-
performanceTrace = performance.newTrace("test_trace")
74-
performanceTraceIdenticalName = performance.newTrace("test_trace")
75-
}
76-
binding.startTrace.setOnClickListener {
77-
performanceTrace?.start()
78-
performanceTraceIdenticalName?.start()
79-
}
80-
binding.stopTrace.setOnClickListener {
81-
performanceTrace?.stop()
82-
performanceTraceIdenticalName?.stop()
74+
viewLifecycleOwner.lifecycleScope.launch(Dispatchers.IO) {
75+
val performanceTrace = performance.newTrace("test_trace")
76+
performanceTrace.start()
77+
delay(1000)
78+
performanceTrace.stop()
79+
}
8380
}
8481
binding.buttonForegroundProcess.setOnClickListener {
8582
if (binding.buttonForegroundProcess.getText().startsWith("Start")) {

firebase-sessions/test-app/src/main/res/layout/fragment_first.xml

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -55,22 +55,6 @@
5555
app:layout_constraintLeft_toLeftOf="parent"
5656
app:layout_constraintTop_toBottomOf="@id/button_anr" />
5757

58-
<Button
59-
android:id="@+id/start_trace"
60-
android:layout_width="wrap_content"
61-
android:layout_height="wrap_content"
62-
android:text="@string/start_trace"
63-
app:layout_constraintLeft_toLeftOf="parent"
64-
app:layout_constraintTop_toBottomOf="@id/button_anr" />
65-
66-
<Button
67-
android:id="@+id/stop_trace"
68-
android:layout_width="wrap_content"
69-
android:layout_height="wrap_content"
70-
android:text="@string/stop_trace"
71-
app:layout_constraintLeft_toLeftOf="parent"
72-
app:layout_constraintTop_toBottomOf="@id/start_trace" />
73-
7458
<Button
7559
android:id="@+id/button_foreground_process"
7660
android:layout_width="wrap_content"

firebase-sessions/test-app/src/main/res/values/strings.xml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,6 @@
2424
<string name="non_fatal_button_text">Non Fatal</string>
2525
<string name="button_anr_text">ANR</string>
2626
<string name="create_trace">Create New Trace: test_trace</string>
27-
<string name="start_trace">Start Trace: test_trace</string>
28-
<string name="stop_trace">Stop Trace: test_trace</string>
2927
<string name="start_foreground_service_text">Start Foreground Service</string>
3028
<string name="start_splitcreen_text">Start splitscreen - Different activity</string>
3129
<string name="start_splitcreen_same_text">Start splitscreen - Same activity</string>

firebase-sessions/test-app/test-app.gradle.kts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,8 @@ dependencies {
8484
implementation("androidx.navigation:navigation-fragment-ktx:2.4.1")
8585
implementation("androidx.navigation:navigation-ui-ktx:2.4.1")
8686
implementation("com.google.android.material:material:1.9.0")
87+
implementation("org.jetbrains.kotlinx:kotlinx-coroutines-android:1.3.9")
88+
implementation("androidx.lifecycle:lifecycle-runtime-ktx:2.4.0")
8789
implementation(libs.androidx.core)
8890

8991
androidTestImplementation("com.google.firebase:firebase-common:21.0.0")

0 commit comments

Comments
 (0)