Merge "Add null checks to browser callbacks" into rvc-dev am: 31a8993720

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

Change-Id: I419b8a9ceee45d7b37e224649cff92b8649209a8
This commit is contained in:
Beth Thibodeau
2020-06-29 19:37:09 +00:00
committed by Automerger Merge Worker

View File

@@ -94,7 +94,7 @@ public class ResumeMediaBrowser {
// a request with EXTRA_RECENT; if they don't, no resume controls
MediaBrowser.MediaItem child = children.get(0);
MediaDescription desc = child.getDescription();
if (child.isPlayable()) {
if (child.isPlayable() && mMediaBrowser != null) {
mCallback.addTrack(desc, mMediaBrowser.getServiceComponent(),
ResumeMediaBrowser.this);
} else {
@@ -129,7 +129,7 @@ public class ResumeMediaBrowser {
@Override
public void onConnected() {
Log.d(TAG, "Service connected for " + mComponentName);
if (mMediaBrowser.isConnected()) {
if (mMediaBrowser != null && mMediaBrowser.isConnected()) {
String root = mMediaBrowser.getRoot();
if (!TextUtils.isEmpty(root)) {
mCallback.onConnected();
@@ -187,7 +187,7 @@ public class ResumeMediaBrowser {
@Override
public void onConnected() {
Log.d(TAG, "Connected for restart " + mMediaBrowser.isConnected());
if (!mMediaBrowser.isConnected()) {
if (mMediaBrowser == null || !mMediaBrowser.isConnected()) {
mCallback.onError();
return;
}
@@ -246,7 +246,7 @@ public class ResumeMediaBrowser {
@Override
public void onConnected() {
Log.d(TAG, "connected");
if (!mMediaBrowser.isConnected()
if (mMediaBrowser == null || !mMediaBrowser.isConnected()
|| TextUtils.isEmpty(mMediaBrowser.getRoot())) {
mCallback.onError();
} else {