Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Cache the Binder call for PlaybackState" into udc-dev am: 5f716d1882

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/23377421

Change-Id: I4b55bb2adb371f2f80c73697b4defefd564be5bc
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Hongwei Wang
2023-05-23 02:46:43 +00:00
committed by Automerger Merge Worker
parent 3596a2ed76 5f716d1882
commit a88af07124
1 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
libs/WindowManager/Shell/src/com/android/wm/shell/pip/PipMediaController.java
View File

@@ -252,13 +252,16 @@ public class PipMediaController {
// It can be removed when min_sdk of the app is set to 31 or greater. // It can be removed when min_sdk of the app is set to 31 or greater.
@SuppressLint("NewApi") @SuppressLint("NewApi")
private List<RemoteAction> getMediaActions() { private List<RemoteAction> getMediaActions() {
if (mMediaController == null || mMediaController.getPlaybackState() == null) { // Cache the PlaybackState since it's a Binder call.
final PlaybackState playbackState;
if (mMediaController == null
|| (playbackState = mMediaController.getPlaybackState()) == null) {
return Collections.emptyList(); return Collections.emptyList();
} }
ArrayList<RemoteAction> mediaActions = new ArrayList<>(); ArrayList<RemoteAction> mediaActions = new ArrayList<>();
boolean isPlaying = mMediaController.getPlaybackState().isActive(); boolean isPlaying = playbackState.isActive();
long actions = mMediaController.getPlaybackState().getActions(); long actions = playbackState.getActions();
// Prev action // Prev action
mPrevAction.setEnabled((actions & PlaybackState.ACTION_SKIP_TO_PREVIOUS) != 0); mPrevAction.setEnabled((actions & PlaybackState.ACTION_SKIP_TO_PREVIOUS) != 0);