Commit c37c6812 by bachinger Committed by microkatz

Remove unused argument

PiperOrigin-RevId: 461830378
(cherry picked from commit e8b5aad3)
parent 481205a8
...@@ -413,7 +413,7 @@ import org.checkerframework.checker.initialization.qual.Initialized; ...@@ -413,7 +413,7 @@ import org.checkerframework.checker.initialization.qual.Initialized;
.isPlayerCommandAvailable(controller, Player.COMMAND_GET_TEXT), .isPlayerCommandAvailable(controller, Player.COMMAND_GET_TEXT),
excludeTimeline); excludeTimeline);
} catch (DeadObjectException e) { } catch (DeadObjectException e) {
onDeadObjectException(controller, e); onDeadObjectException(controller);
} catch (RemoteException e) { } catch (RemoteException e) {
// Currently it's TransactionTooLargeException or DeadSystemException. // Currently it's TransactionTooLargeException or DeadSystemException.
// We'd better to leave log for those cases because // We'd better to leave log for those cases because
...@@ -598,7 +598,7 @@ import org.checkerframework.checker.initialization.qual.Initialized; ...@@ -598,7 +598,7 @@ import org.checkerframework.checker.initialization.qual.Initialized;
task.run(cb, seq); task.run(cb, seq);
} }
} catch (DeadObjectException e) { } catch (DeadObjectException e) {
onDeadObjectException(controller, e); onDeadObjectException(controller);
} catch (RemoteException e) { } catch (RemoteException e) {
// Currently it's TransactionTooLargeException or DeadSystemException. // Currently it's TransactionTooLargeException or DeadSystemException.
// We'd better to leave log for those cases because // We'd better to leave log for those cases because
...@@ -635,7 +635,7 @@ import org.checkerframework.checker.initialization.qual.Initialized; ...@@ -635,7 +635,7 @@ import org.checkerframework.checker.initialization.qual.Initialized;
} }
return future; return future;
} catch (DeadObjectException e) { } catch (DeadObjectException e) {
onDeadObjectException(controller, e); onDeadObjectException(controller);
return Futures.immediateFuture(new SessionResult(RESULT_ERROR_SESSION_DISCONNECTED)); return Futures.immediateFuture(new SessionResult(RESULT_ERROR_SESSION_DISCONNECTED));
} catch (RemoteException e) { } catch (RemoteException e) {
// Currently it's TransactionTooLargeException or DeadSystemException. // Currently it's TransactionTooLargeException or DeadSystemException.
...@@ -649,7 +649,7 @@ import org.checkerframework.checker.initialization.qual.Initialized; ...@@ -649,7 +649,7 @@ import org.checkerframework.checker.initialization.qual.Initialized;
} }
/** Removes controller. Call this when DeadObjectException is happened with binder call. */ /** Removes controller. Call this when DeadObjectException is happened with binder call. */
private void onDeadObjectException(ControllerInfo controller, DeadObjectException e) { private void onDeadObjectException(ControllerInfo controller) {
// Note: Only removing from MediaSessionStub and ignoring (legacy) stubs would be fine for // Note: Only removing from MediaSessionStub and ignoring (legacy) stubs would be fine for
// now. Because calls to the legacy stubs doesn't throw DeadObjectException. // now. Because calls to the legacy stubs doesn't throw DeadObjectException.
sessionStub.getConnectedControllersManager().removeController(controller); sessionStub.getConnectedControllersManager().removeController(controller);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment