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

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

Issue 1652163002: Refactor Clank cast code to use MediaNotificationManager (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@MediaUi
Patch Set: Respond to final batch of nits Created 4 years, 10 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
Index: chrome/android/java/src/org/chromium/chrome/browser/media/remote/LockScreenTransportControlV18.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/media/remote/LockScreenTransportControlV18.java b/chrome/android/java/src/org/chromium/chrome/browser/media/remote/LockScreenTransportControlV18.java
deleted file mode 100644
index 1e8a608cfa4b72e328aae999d27ad576a6ef0cac..0000000000000000000000000000000000000000
--- a/chrome/android/java/src/org/chromium/chrome/browser/media/remote/LockScreenTransportControlV18.java
+++ /dev/null
@@ -1,80 +0,0 @@
-// Copyright 2014 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-package org.chromium.chrome.browser.media.remote;
-
-import android.annotation.TargetApi;
-import android.content.Context;
-import android.os.Build;
-
-/**
- * An implementation of {@link LockScreenTransportControl} targeting platforms with an API greater
- * than 17. Extends {@link LockScreenTransportControlV16}, adding support for seeking.
- */
-@TargetApi(Build.VERSION_CODES.JELLY_BEAN_MR2)
-// This whole class is based upon RemoteControlClient, which is deprecated (and non-functional) in
-// the Android L SDK. Once the L SDK is released we will add code in LockSceenTransportControl to
-// prevent the use of this class on Android L. See {@link LockScreenTransportControl}.
-@SuppressWarnings("deprecation")
-class LockScreenTransportControlV18 extends LockScreenTransportControlV16 {
-
- private final PlaybackPositionUpdateListener mPlaybackPositionUpdateListener;
- private final GetPlaybackPositionUpdateListener mGetPlaybackPositionUpdateListener;
-
- LockScreenTransportControlV18(Context context) {
- super(context);
- mPlaybackPositionUpdateListener = new PlaybackPositionUpdateListener();
- mGetPlaybackPositionUpdateListener = new GetPlaybackPositionUpdateListener();
- }
-
- @Override
- protected void register() {
- super.register();
- getRemoteControlClient().setPlaybackPositionUpdateListener(mPlaybackPositionUpdateListener);
- getRemoteControlClient().setOnGetPlaybackPositionListener(
- mGetPlaybackPositionUpdateListener);
- }
-
- @Override
- protected void unregister() {
- getRemoteControlClient().setOnGetPlaybackPositionListener(null);
- getRemoteControlClient().setPlaybackPositionUpdateListener(null);
- super.unregister();
- }
-
- @Override
- protected void updatePlaybackState(int state) {
- RemoteVideoInfo videoInfo = getVideoInfo();
- if (videoInfo != null && getRemoteControlClient() != null) {
- getRemoteControlClient().setPlaybackState(state, videoInfo.currentTimeMillis, 1.0f);
- } else {
- super.updatePlaybackState(state);
- }
- }
-
- @Override
- protected int getTransportControlFlags() {
- return super.getTransportControlFlags()
- | android.media.RemoteControlClient.FLAG_KEY_MEDIA_POSITION_UPDATE;
- }
-
- private class GetPlaybackPositionUpdateListener implements
- android.media.RemoteControlClient.OnGetPlaybackPositionListener {
-
- @Override
- public long onGetPlaybackPosition() {
- RemoteVideoInfo videoInfo = getVideoInfo();
- return videoInfo == null ? 0 : videoInfo.currentTimeMillis;
- }
- }
-
- private class PlaybackPositionUpdateListener implements
- android.media.RemoteControlClient.OnPlaybackPositionUpdateListener {
-
- @Override
- public void onPlaybackPositionUpdate(long position) {
- for (Listener listener : getListeners()) listener.onSeek(position);
- }
- }
-}

Powered by Google App Engine
This is Rietveld 408576698