Skip to content

Bugfix/rs notification issue usb #1797

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Mar 28, 2022
Merged

Bugfix/rs notification issue usb #1797

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
a9b0b5d
Fix RS exit foreground params
joeygrover Mar 23, 2022
e1ed2b9
Refactor to call extiForeground for every close()
joeygrover Mar 23, 2022
06f3cd8
Fix altTransport timer issues preventing RS close
joeygrover Mar 23, 2022
28eeac1
Fix incorrect spacing PR #1797
joeygrover Mar 28, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix altTransport timer issues preventing RS close
  • Loading branch information
@joeygrover
joeygrover committed Mar 23, 2022
commit 06f3cd8e3f0db0d09438600de53ca4c363e6cbb1
13 changes: 12 additions & 1 deletion android/sdl_android/src/main/java/com/smartdevicelink/transport/SdlRouterService.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1758,7 +1758,7 @@ public boolean shouldServiceRemainOpen(Intent intent) {

} else if (intent != null && TransportConstants.BIND_REQUEST_TYPE_ALT_TRANSPORT.equals(intent.getAction())) {
DebugTool.logInfo(TAG, "Received start intent with alt transport request.");
startAltTransportTimer();
startAltTransportTimer(); //This timer is started to allow the router service to stay open while it waits for the USB transfer to take place
return true;
} else if (!bluetoothAvailable()) {//If bluetooth isn't on...there's nothing to see here
//Bluetooth is off, we should shut down
Expand Down Expand Up @@ -1980,6 +1980,12 @@ public void onTransportDisconnected(TransportRecord record) {
usbSessionMap.clear();
}
}
//In case the USB connection has ended before the timer expired, we should stop it
if(altTransportTimerHandler != null && altTransportTimerRunnable != null) {
altTransportTimerHandler.removeCallbacks(altTransportTimerRunnable);
altTransportTimerHandler = null;
altTransportTimerRunnable = null;
}
break;
case TCP:
if (tcpTransport != null) {
Expand Down Expand Up @@ -2631,6 +2637,11 @@ private void startAltTransportTimer() {
if (Looper.myLooper() == null) {
Looper.prepare();
}

if(altTransportTimerHandler != null && altTransportTimerRunnable != null) {
altTransportTimerHandler.removeCallbacks(altTransportTimerRunnable);
}

altTransportTimerHandler = new Handler(Looper.myLooper());
altTransportTimerRunnable = new Runnable() {
public void run() {
Expand Down