Fix NPE for CompanionDeviceActivity.setResultAndFinish()

Fix: 205274846
Test: atest CompanionDeviceManagerTest
Change-Id: Icecd76d6ff5a81afdcdb7d584ae3ecff51ac5dc3
This commit is contained in:
Evan Chen
2021-11-08 20:05:40 +00:00
parent 8f0480504f
commit 0f88f8c9a2
2 changed files with 5 additions and 8 deletions

View File

@@ -78,6 +78,7 @@ public class CompanionDeviceActivity extends Activity {
getWindow().addSystemFlags(SYSTEM_FLAG_HIDE_NON_SYSTEM_OVERLAY_WINDOWS);
sInstance = this;
getService().mActivity = this;
String deviceProfile = getRequest().getDeviceProfile();
String profilePrivacyDisclaimer = emptyIfNull(getRequest()
@@ -141,8 +142,6 @@ public class CompanionDeviceActivity extends Activity {
profileSummary.setVisibility(View.GONE);
}
getService().mActivity = this;
mCancelButton = findViewById(R.id.button_cancel);
mCancelButton.setOnClickListener(v -> cancel());
}
@@ -194,6 +193,7 @@ public class CompanionDeviceActivity extends Activity {
@Override
protected void onDestroy() {
super.onDestroy();
getService().mActivity = null;
if (sInstance == this) {
sInstance = null;
}

View File

@@ -257,12 +257,8 @@ public class CompanionDeviceDiscoveryService extends Service {
if (!mIsScanning) return;
mIsScanning = false;
CompanionDeviceActivity activity = mActivity;
if (activity != null) {
if (activity.mDeviceListView != null) {
activity.mDeviceListView.removeFooterView(activity.mLoadingIndicator);
}
mActivity = null;
if (mActivity != null && mActivity.mDeviceListView != null) {
mActivity.mDeviceListView.removeFooterView(mActivity.mLoadingIndicator);
}
mBluetoothAdapter.cancelDiscovery();
@@ -334,6 +330,7 @@ public class CompanionDeviceDiscoveryService extends Service {
void onCancel() {
if (DEBUG) Log.i(LOG_TAG, "onCancel()");
mActivity = null;
mServiceCallback.cancel(true);
}