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

[LE] Error handling for updating the Broadcast Code and Name

Browse Source 
- Add methods for Broadcast callback to handle the UI in base dialog

 - Override the methods in broadcast dialog to handle the update
   broadcast info error.

Bug: 230473995
Test: build pass
Change-Id: Ieb81c7fc39368f5fc65ed1b1ed0a14c1ff14ba86
This commit is contained in:
changbetty
2022-05-05 13:12:19 +00:00
parent 8d7c5727d2
commit 7fb2ee23d3
2 changed files with 125 additions and 42 deletions
Download Patch File Download Diff File Expand all files Collapse all files

44
packages/SystemUI/src/com/android/systemui/media/dialog/MediaOutputBaseDialog.java
View File

@@ -76,6 +76,7 @@ public abstract class MediaOutputBaseDialog extends SystemUIDialog implements
private static final String PREF_NAME = "MediaOutputDialog";
private static final String PREF_IS_LE_BROADCAST_FIRST_LAUNCH = "PrefIsLeBroadcastFirstLaunch";
private static final boolean DEBUG = true;
private static final int HANDLE_BROADCAST_FAILED_DELAY = 3000;
private final Handler mMainThreadHandler = new Handler(Looper.getMainLooper());
private final RecyclerView.LayoutManager mLayoutManager;
@@ -119,7 +120,7 @@ public abstract class MediaOutputBaseDialog extends SystemUIDialog implements
Log.d(TAG, "onBroadcastStarted(), reason = " + reason
+ ", broadcastId = " + broadcastId);
}
mMainThreadHandler.post(() -> startLeBroadcastDialog());
mMainThreadHandler.post(() -> handleLeBroadcastStarted());
}
@Override
@@ -127,7 +128,8 @@ public abstract class MediaOutputBaseDialog extends SystemUIDialog implements
if (DEBUG) {
Log.d(TAG, "onBroadcastStartFailed(), reason = " + reason);
}
handleLeBroadcastStartFailed();
mMainThreadHandler.postDelayed(() -> handleLeBroadcastStartFailed(),
HANDLE_BROADCAST_FAILED_DELAY);
}
@Override
@@ -137,7 +139,7 @@ public abstract class MediaOutputBaseDialog extends SystemUIDialog implements
Log.d(TAG, "onBroadcastMetadataChanged(), broadcastId = " + broadcastId
+ ", metadata = " + metadata);
}
mMainThreadHandler.post(() -> refresh());
mMainThreadHandler.post(() -> handleLeBroadcastMetadataChanged());
}
@Override
@@ -146,7 +148,7 @@ public abstract class MediaOutputBaseDialog extends SystemUIDialog implements
Log.d(TAG, "onBroadcastStopped(), reason = " + reason
+ ", broadcastId = " + broadcastId);
}
mMainThreadHandler.post(() -> refresh());
mMainThreadHandler.post(() -> handleLeBroadcastStopped());
}
@Override
@@ -154,7 +156,7 @@ public abstract class MediaOutputBaseDialog extends SystemUIDialog implements
if (DEBUG) {
Log.d(TAG, "onBroadcastStopFailed(), reason = " + reason);
}
mMainThreadHandler.post(() -> refresh());
mMainThreadHandler.post(() -> handleLeBroadcastStopFailed());
}
@Override
@@ -163,7 +165,7 @@ public abstract class MediaOutputBaseDialog extends SystemUIDialog implements
Log.d(TAG, "onBroadcastUpdated(), reason = " + reason
+ ", broadcastId = " + broadcastId);
}
mMainThreadHandler.post(() -> refresh());
mMainThreadHandler.post(() -> handleLeBroadcastUpdated());
}
@Override
@@ -172,7 +174,7 @@ public abstract class MediaOutputBaseDialog extends SystemUIDialog implements
Log.d(TAG, "onBroadcastUpdateFailed(), reason = " + reason
+ ", broadcastId = " + broadcastId);
}
mMainThreadHandler.post(() -> refresh());
mMainThreadHandler.post(() -> handleLeBroadcastUpdateFailed());
}
@Override
@@ -384,10 +386,34 @@ public abstract class MediaOutputBaseDialog extends SystemUIDialog implements
Bitmap.createScaledBitmap(bitmap, size, size, false));
}
protected void handleLeBroadcastStartFailed() {
public void handleLeBroadcastStarted() {
startLeBroadcastDialog();
}
public void handleLeBroadcastStartFailed() {
mStopButton.setText(R.string.media_output_broadcast_start_failed);
mStopButton.setEnabled(false);
mMainThreadHandler.postDelayed(() -> refresh(), 3000);
refresh();
}
public void handleLeBroadcastMetadataChanged() {
refresh();
}
public void handleLeBroadcastStopped() {
refresh();
}
public void handleLeBroadcastStopFailed() {
refresh();
}
public void handleLeBroadcastUpdated() {
refresh();
}
public void handleLeBroadcastUpdateFailed() {
refresh();
}
protected void startLeBroadcast() {

123
packages/SystemUI/src/com/android/systemui/media/dialog/MediaOutputBroadcastDialog.java
View File

@@ -61,6 +61,10 @@ public class MediaOutputBroadcastDialog extends MediaOutputBaseDialog {
private ImageView mBroadcastCodeEdit;
private AlertDialog mAlertDialog;
private TextView mBroadcastErrorMessage;
private int mRetryCount = 0;
private String mCurrentBroadcastName;
private String mCurrentBroadcastCode;
private boolean mIsStopbyUpdateBroadcastCode = false;
static final int METADATA_BROADCAST_NAME = 0;
static final int METADATA_BROADCAST_CODE = 1;
@@ -144,8 +148,6 @@ public class MediaOutputBroadcastDialog extends MediaOutputBaseDialog {
//init UI component
mBroadcastQrCodeView = getDialogView().requireViewById(R.id.qrcode_view);
//Set the QR code view
setQrCodeView();
mBroadcastNotify = getDialogView().requireViewById(R.id.broadcast_info);
mBroadcastNotify.setOnClickListener(v -> {
@@ -171,8 +173,16 @@ public class MediaOutputBroadcastDialog extends MediaOutputBaseDialog {
launchBroadcastUpdatedDialog(true, mBroadcastCode.getText().toString());
});
mBroadcastName.setText(getBroadcastMetadataInfo(METADATA_BROADCAST_NAME));
mBroadcastCode.setText(getBroadcastMetadataInfo(METADATA_BROADCAST_CODE));
refreshUi();
}
private void refreshUi() {
setQrCodeView();
mCurrentBroadcastName = getBroadcastMetadataInfo(METADATA_BROADCAST_NAME);
mCurrentBroadcastCode = getBroadcastMetadataInfo(METADATA_BROADCAST_CODE);
mBroadcastName.setText(mCurrentBroadcastName);
mBroadcastCode.setText(mCurrentBroadcastCode);
}
private void inflateBroadcastInfoArea() {
@@ -239,52 +249,99 @@ public class MediaOutputBroadcastDialog extends MediaOutputBaseDialog {
}
if (isBroadcastCode) {
handleBroadcastCodeUpdated(updatedString);
/* If the user wants to update the Broadcast Code, the Broadcast session should be
* stopped then used the new Broadcast code to start the Broadcast.
*/
mIsStopbyUpdateBroadcastCode = true;
mMediaOutputController.setBroadcastCode(updatedString);
if (!mMediaOutputController.stopBluetoothLeBroadcast()) {
handleLeBroadcastStopFailed();
return;
}
} else {
handleBroadcastNameUpdated(updatedString);
/* If the user wants to update the Broadcast Name, we don't need to stop the Broadcast
* session. Only use the new Broadcast name to update the broadcast session.
*/
mMediaOutputController.setBroadcastName(updatedString);
if (!mMediaOutputController.updateBluetoothLeBroadcast()) {
handleLeBroadcastUpdateFailed();
}
}
}
private void handleBroadcastNameUpdated(String name) {
// TODO(b/230473995) Add the retry mechanism and error handling when update fails
String currentName = mMediaOutputController.getBroadcastName();
int retryCount = MAX_BROADCAST_INFO_UPDATE;
mMediaOutputController.setBroadcastName(name);
if (!mMediaOutputController.updateBluetoothLeBroadcast()) {
mMediaOutputController.setBroadcastName(currentName);
handleLeUpdateBroadcastFailed(retryCount);
@Override
public void handleLeBroadcastStarted() {
mRetryCount = 0;
if (mAlertDialog != null) {
mAlertDialog.dismiss();
}
refreshUi();
}
@Override
public void handleLeBroadcastStartFailed() {
mMediaOutputController.setBroadcastCode(mCurrentBroadcastCode);
mRetryCount++;
handleUpdateFailedUi();
}
@Override
public void handleLeBroadcastMetadataChanged() {
refreshUi();
}
@Override
public void handleLeBroadcastUpdated() {
mRetryCount = 0;
if (mAlertDialog != null) {
mAlertDialog.dismiss();
}
refreshUi();
}
@Override
public void handleLeBroadcastUpdateFailed() {
//Change the value in shared preferences back to it original value
mMediaOutputController.setBroadcastName(mCurrentBroadcastName);
mRetryCount++;
handleUpdateFailedUi();
}
@Override
public void handleLeBroadcastStopped() {
if (mIsStopbyUpdateBroadcastCode) {
mIsStopbyUpdateBroadcastCode = false;
mRetryCount = 0;
if (!mMediaOutputController.startBluetoothLeBroadcast()) {
handleLeBroadcastStartFailed();
return;
}
} else {
dismiss();
}
}
private void handleBroadcastCodeUpdated(String newPassword) {
// TODO(b/230473995) Add the retry mechanism and error handling when update fails
String currentPassword = mMediaOutputController.getBroadcastCode();
int retryCount = MAX_BROADCAST_INFO_UPDATE;
if (!mMediaOutputController.stopBluetoothLeBroadcast()) {
mMediaOutputController.setBroadcastCode(currentPassword);
handleLeUpdateBroadcastFailed(retryCount);
return;
}
@Override
public void handleLeBroadcastStopFailed() {
//Change the value in shared preferences back to it original value
mMediaOutputController.setBroadcastCode(mCurrentBroadcastCode);
mRetryCount++;
mMediaOutputController.setBroadcastCode(newPassword);
if (!mMediaOutputController.startBluetoothLeBroadcast()) {
mMediaOutputController.setBroadcastCode(currentPassword);
handleLeUpdateBroadcastFailed(retryCount);
return;
}
mAlertDialog.dismiss();
handleUpdateFailedUi();
}
private void handleLeUpdateBroadcastFailed(int retryCount) {
private void handleUpdateFailedUi() {
final Button positiveBtn = mAlertDialog.getButton(AlertDialog.BUTTON_POSITIVE);
mBroadcastErrorMessage.setVisibility(View.VISIBLE);
if (retryCount < MAX_BROADCAST_INFO_UPDATE) {
if (mRetryCount < MAX_BROADCAST_INFO_UPDATE) {
if (positiveBtn != null) {
positiveBtn.setEnabled(true);
}
mBroadcastErrorMessage.setText(R.string.media_output_broadcast_update_error);
} else {
mRetryCount = 0;
mBroadcastErrorMessage.setText(R.string.media_output_broadcast_last_update_error);
}
}