From 013f16c77e24fc18e2cf1c74af51aa9763cc4345 Mon Sep 17 00:00:00 2001 From: mayurikhin Date: Tue, 6 Aug 2024 21:32:53 +0530 Subject: [PATCH 1/2] Merge remote-tracking branch 'origin/playback_latency_metrics' into playback_latency_metrics --- .../samples/socialite/ui/home/timeline/TimelineViewModel.kt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/src/main/java/com/google/android/samples/socialite/ui/home/timeline/TimelineViewModel.kt b/app/src/main/java/com/google/android/samples/socialite/ui/home/timeline/TimelineViewModel.kt index b9403163..dc2800f5 100644 --- a/app/src/main/java/com/google/android/samples/socialite/ui/home/timeline/TimelineViewModel.kt +++ b/app/src/main/java/com/google/android/samples/socialite/ui/home/timeline/TimelineViewModel.kt @@ -52,7 +52,7 @@ class TimelineViewModel @Inject constructor( // Width/Height ratio of the current media item, used to properly size the Surface var videoRatio by mutableStateOf(null) - var timeAtPlayerPrepare : Long = 0 + var timeAtPlayerPrepare: Long = 0 private val videoSizeListener = object : Player.Listener { override fun onVideoSizeChanged(videoSize: VideoSize) { @@ -69,7 +69,7 @@ class TimelineViewModel @Inject constructor( private val firstFrameRenderedListener = object : Player.Listener { override fun onRenderedFirstFrame() { super.onRenderedFirstFrame() - val timeToFirstRenderFrame = System.currentTimeMillis() - timeAtPlayerPrepare + val timeToFirstRenderFrame = System.currentTimeMillis() - timeAtPlayerPrepare // Use this value in future to compare performance with and without preload manager } } From 269d509147e8f1002177b56a8703cecc15f7c440 Mon Sep 17 00:00:00 2001 From: mayurikhin Date: Wed, 7 Aug 2024 13:46:38 +0530 Subject: [PATCH 2/2] Merge remote-tracking branch 'origin/playback_latency_metrics' into playback_latency_metrics --- .../samples/socialite/ui/home/timeline/TimelineViewModel.kt | 3 +++ 1 file changed, 3 insertions(+) diff --git a/app/src/main/java/com/google/android/samples/socialite/ui/home/timeline/TimelineViewModel.kt b/app/src/main/java/com/google/android/samples/socialite/ui/home/timeline/TimelineViewModel.kt index dc2800f5..dd1991e7 100644 --- a/app/src/main/java/com/google/android/samples/socialite/ui/home/timeline/TimelineViewModel.kt +++ b/app/src/main/java/com/google/android/samples/socialite/ui/home/timeline/TimelineViewModel.kt @@ -18,6 +18,7 @@ package com.google.android.samples.socialite.ui.home.timeline import android.content.Context import android.net.Uri +import android.util.Log import androidx.annotation.OptIn import androidx.compose.runtime.getValue import androidx.compose.runtime.mutableStateOf @@ -53,6 +54,7 @@ class TimelineViewModel @Inject constructor( var videoRatio by mutableStateOf(null) var timeAtPlayerPrepare: Long = 0 + private val TAG = TimelineViewModel::class.java.simpleName private val videoSizeListener = object : Player.Listener { override fun onVideoSizeChanged(videoSize: VideoSize) { @@ -71,6 +73,7 @@ class TimelineViewModel @Inject constructor( super.onRenderedFirstFrame() val timeToFirstRenderFrame = System.currentTimeMillis() - timeAtPlayerPrepare // Use this value in future to compare performance with and without preload manager + Log.d(TAG, "First frame rendered in $timeToFirstRenderFrame ms") } }