diff options
| author | 2024-04-24 15:29:57 +0100 | |
|---|---|---|
| committer | 2024-04-24 15:39:29 +0100 | |
| commit | 5a3013b421d71e470c108eae082011809c7d6c3f (patch) | |
| tree | d62af1e86fa68106467c96d8fab780797a3570c6 /libs/input/PointerController.cpp | |
| parent | 8e1ad154a870d73e3030d6a5f0fb752c6a678023 (diff) | |
Fix nullability in MediaController#getPlaybackInfo()
The playback info field is NonNull, so getPlaybackInfo() is artificially
returning a null value if the Binder transaction to system_server fails.
As per go/android-api-guidelines#framework-rethrow-remoteexception we should always be rethrowing
RemoteExceptions to provide a strong signal that the operation failed.
In practice, the returned value is never null, unless system_server has
died. Several CTS tests already assume this NonNull-ness.
Test: atest CtsMediaBetterTogetherTestCases
Bug: 333564788
Change-Id: Id18cce692d8d0b25486da427ef9860eb2f176aef
Diffstat (limited to 'libs/input/PointerController.cpp')
0 files changed, 0 insertions, 0 deletions