Index: chrome/android/java/src/org/chromium/chrome/browser/media/remote/NotificationTransportControl.java |
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/media/remote/NotificationTransportControl.java b/chrome/android/java/src/org/chromium/chrome/browser/media/remote/NotificationTransportControl.java |
index 683f119f74306527527f4f044af758c51d363f8d..72c3b1bf16410b97eccb01c316965459a2af2314 100644 |
--- a/chrome/android/java/src/org/chromium/chrome/browser/media/remote/NotificationTransportControl.java |
+++ b/chrome/android/java/src/org/chromium/chrome/browser/media/remote/NotificationTransportControl.java |
@@ -34,10 +34,10 @@ import javax.annotation.Nullable; |
public class NotificationTransportControl |
extends TransportControl implements MediaRouteController.UiListener { |
/** |
- * Service used to transform intent requests triggered from the notification into |
- * {@code Listener} callbacks. Ideally this class should be protected, but public is required |
- * to create as a service. |
- */ |
+ * Service used to transform intent requests triggered from the notification into |
+ * {@code Listener} callbacks. Ideally this class should be protected, but public is required to |
+ * create as a service. |
+ */ |
public static class ListenerService extends Service { |
private static final String ACTION_PREFIX = ListenerService.class.getName() + "."; |
@@ -185,8 +185,8 @@ public class NotificationTransportControl |
case PLAYING: |
showProgress = true; |
showPlayPause = true; |
- contentView.setProgressBar(R.id.progress, videoInfo.durationMillis, |
- videoInfo.currentTimeMillis, false); |
+ contentView.setProgressBar(R.id.progress, (int) videoInfo.durationMillis, |
whywhat
2016/01/05 21:19:24
is this typecast a potential problem for really lo
aberent
2016/01/06 11:13:34
Done (although I think you will have trouble findi
|
+ (int) videoInfo.currentTimeMillis, false); |
contentView.setImageViewResource( |
R.id.playpause, R.drawable.ic_vidcontrol_pause); |
contentView.setContentDescription( |
@@ -198,8 +198,8 @@ public class NotificationTransportControl |
case PAUSED: |
showProgress = true; |
showPlayPause = true; |
- contentView.setProgressBar(R.id.progress, videoInfo.durationMillis, |
- videoInfo.currentTimeMillis, false); |
+ contentView.setProgressBar(R.id.progress, (int) videoInfo.durationMillis, |
+ (int) videoInfo.currentTimeMillis, false); |
contentView.setImageViewResource( |
R.id.playpause, R.drawable.ic_vidcontrol_play); |
contentView.setContentDescription( |
@@ -358,7 +358,6 @@ public class NotificationTransportControl |
return Bitmap.createScaledBitmap(bitmap, width, height, false); |
} |
- |
private NotificationTransportControl(Context context) { |
this.mContext = context; |
mHandler = new Handler(context.getMainLooper()); |
@@ -381,7 +380,7 @@ public class NotificationTransportControl |
} |
@Override |
- public void onDurationUpdated(int durationMillis) { |
+ public void onDurationUpdated(long durationMillis) { |
// Set the progress update interval based on the screen height/width, since there's no point |
// in updating the progress bar more frequently than what the user can see. |
// getDisplayMetrics() is dependent on the current orientation, so we need to get the max |
@@ -415,7 +414,7 @@ public class NotificationTransportControl |
} |
@Override |
- public void onPositionChanged(int positionMillis) { |
+ public void onPositionChanged(long positionMillis) { |
RemoteVideoInfo videoInfo = new RemoteVideoInfo(getVideoInfo()); |
videoInfo.currentTimeMillis = positionMillis; |
setVideoInfo(videoInfo); |