Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1661)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/media/remote/AbstractMediaRouteController.java

Issue 1895773003: [Cast, Android] Use elapsedRealtime vs uptimeMillis for end of playback UMA (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@2661
Patch Set: Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/media/remote/AbstractMediaRouteController.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/media/remote/AbstractMediaRouteController.java b/chrome/android/java/src/org/chromium/chrome/browser/media/remote/AbstractMediaRouteController.java
index c6e08b123d67cd3a736e432c79d4d22a602c3b18..dbe38dff69e4e167e82eb97d2d55d64d1ba8dfbe 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/media/remote/AbstractMediaRouteController.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/media/remote/AbstractMediaRouteController.java
@@ -404,7 +404,7 @@ public abstract class AbstractMediaRouteController implements MediaRouteControll
}
private void recordEndOfSessionUMA() {
- long remotePlaybackStoppedTimestampMs = SystemClock.uptimeMillis();
+ long remotePlaybackStoppedTimestampMs = SystemClock.elapsedRealtime();
// There was no media element ever...
if (mMediaElementAttachedTimestampMs == 0) return;
@@ -473,12 +473,12 @@ public abstract class AbstractMediaRouteController implements MediaRouteControll
public void setMediaStateListener(MediaStateListener mediaStateListener) {
if (mMediaStateListener != null && mediaStateListener == null
&& mMediaElementAttachedTimestampMs != 0) {
- mMediaElementDetachedTimestampMs = SystemClock.uptimeMillis();
+ mMediaElementDetachedTimestampMs = SystemClock.elapsedRealtime();
} else if (mMediaStateListener == null && mediaStateListener != null) {
// We're switching the videos so let's record the UMA for the previous one.
if (mMediaElementDetachedTimestampMs != 0) recordEndOfSessionUMA();
- mMediaElementAttachedTimestampMs = SystemClock.uptimeMillis();
+ mMediaElementAttachedTimestampMs = SystemClock.elapsedRealtime();
mMediaElementDetachedTimestampMs = 0;
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698