summaryrefslogtreecommitdiff
path: root/telecomm/java
Commit message (Collapse)AuthorAgeFilesLines
* Resolve cross account user icon validation.Pranav Madapurmath2025-07-081-6/+28
| | | | | | | | | | | | | | | Resolves a vulnerability found with the cross account user icon validation in StatusHint and TelecomServiceImpl (when registering a phone account). The reporter found that an uri formatted as `userId%` isn't parsed properly with the existing reference to Uri.encodedUserInfo. Bug: 376461551 Bug: 376259166 Flag: EXEMPT bugfix Test: atest TelecomServiceImplTest (cherry picked from https://googleplex-android-review.googlesource.com/q/commit:81c9a17787743df280adac58ab5f74c084d058e1) Merged-In: I25614ead889501f4553ed2b42b366e09a47b0c9f Change-Id: I25614ead889501f4553ed2b42b366e09a47b0c9f
* Merge "Resolve StatusHints image exploit across user." into sc-v2-dev am: ↵Pranav Madapurmath2023-06-142-4/+61
| | | | | | | | | | | | | e371b3018f Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/23465066 Fixes: 285211549 Fixes: 280797684 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> (cherry picked from https://googleplex-android-review.googlesource.com/q/commit:3fc6dd50937d23c854fde540380c51fd451b1c55) Merged-In: Idd360f69fc9e5a9f32fd3ca76ec0440c8bb12cf4 Change-Id: Idd360f69fc9e5a9f32fd3ca76ec0440c8bb12cf4
* Merge "enforce stricter rules when registering phoneAccounts" into ↵Thomas Stuart2023-03-011-0/+19
|\ | | | | | | | | | | | | | | | | sc-qpr1-dev am: 6f0bdf90e2 am: 77e501084b Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/21318377 Change-Id: If4adcabd1fe285ac49dae0de283153daa72d6ed1 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| * enforce stricter rules when registering phoneAccountsThomas Stuart2023-02-271-0/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - include disable accounts when looking up accounts for a package to check if the limit is reached (10) - put a new limit of 10 supported schemes - put a new limit of 256 characters per scheme - put a new limit of 256 characters per address - ensure the Icon can write to memory w/o throwing an exception bug: 259064622 bug: 256819769 Test: cts + unit Change-Id: Ia7d8d00d9de0fb6694ded6a80c40bd55d7fdf7a7 Merged-In: Ia7d8d00d9de0fb6694ded6a80c40bd55d7fdf7a7
| * [automerge] switch TelecomManager List getters to ParceledListSlice 2p: ↵Presubmit Automerger Backend2022-07-252-12/+13
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | d54a48f42a Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/19369770 Bug: 236263294 Change-Id: Ibb1d5a32c345eed74667a9dedf9179fae961adea Merged-In: I025245b2a6f8cfaca86f268851a9d8f0817e07dd
| | * switch TelecomManager List getters to ParceledListSliceThomas Stuart2022-07-252-12/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It was shown that given a large phoneAccountHandles that are over 1 mb, a TransactionTooLarge exception can be silently thrown causing an empty list to be returned. In order to prevent this behavior, all Lists that return a PhoneAccountHandle or PhoneAccount have been switched to ParceledListSlice. bug: 236263294 Test: atest android.telecom.cts.PhoneAccountRegistrarTest #testRegisterPhoneAccountHandleWithFieldOverLimit Change-Id: I025245b2a6f8cfaca86f268851a9d8f0817e07dd Merged-In: I025245b2a6f8cfaca86f268851a9d8f0817e07dd
* | | switch TelecomManager List getters to ParceledListSliceThomas Stuart2022-07-262-14/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It was shown that given a large phoneAccountHandles that are over 1 mb, a TransactionTooLarge exception can be silently thrown causing an empty list to be returned. In order to prevent this behavior, all Lists that return a PhoneAccountHandle or PhoneAccount have been switched to ParceledListSlice. bug: 236263294 Test: manual #1 - bug exists without fix, manual #2 - bug is fixed with patch, 4 new CTS tests in android.telecom.cts.PhoneAccountRegistrarTest Change-Id: I025245b2a6f8cfaca86f268851a9d8f0817e07dd Merged-In: I025245b2a6f8cfaca86f268851a9d8f0817e07dd
* | | Merge "Update InCallService API documentation to clarify role and binding ↵Tyler Gunn2022-06-231-7/+18
|\ \ \ | | | | | | | | | | | | req." into tm-dev
| * | | Update InCallService API documentation to clarify role and binding req.Tyler Gunn2022-06-221-7/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update InCallService API docs to clarify what happens if an app no longer fulfils the requirement of the dialer role. Also update documentation to clearly indicate when the platform may show a warning to the user due to a null binding. Test: Docs only change. Fixes: 236864749 Change-Id: Ief6225513a7aefefa9c029a02cc1a3511b236d44
* | | | Support comparison of byte array values in areBundlesEqualGrace Jia2022-05-121-1/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, telecom framework will send tons of unnecessary Call#onDetailsChanged to notify update between Call.Details with same byte[] value. Since we didn't support the proper way to compare these value, onDetailsChanged might be called multiple times meaninglessly. Bug: 232180843 Test: CallDetailsTest Change-Id: I739412da62f303cb00dc84cb0a6d7773d0a0aeaa
* | | | Merge "Un-obscure the Telephony phone account ID." into tm-devTreeHugger Robot2022-05-071-8/+25
|\ \ \ \
| * | | | Un-obscure the Telephony phone account ID.Tyler Gunn2022-05-061-8/+25
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With the upcoming changes in T to move telephony phone account handles to use subid instead of ICCID, the id of telephony phone account handles no longer needs to be Log.pii-ed. These phone accounts have been hard to diagnose in most bug reports due to the obfuscation. Bug: 226972613 Test: Manual testing; place calls and verify Telephony account handles are now visible. Change-Id: Icae7917e7063c753c8597c401ad7adc1d9052775
* | | | Merge "fix log spewing coming from Telecom#LogUtils" into tm-devThomas Stuart2022-05-031-0/+19
|\ \ \ \
| * | | | fix log spewing coming from Telecom#LogUtilsThomas Stuart2022-04-301-0/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | LogUtils was causing a ~7k line log spew in the utomation tests. new var created in Telecom Log Frameworks file that can gate ceratin lines causing log spew. bug: 224667124 Test: manual, looked through test logs and log spew lines removed Change-Id: I5aac180832c0c26af201d1286727cc3e9c67d207
* | | | | Revert "Un-obscure the Telephony phone account ID."Philip Quinn2022-05-021-25/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit f601004255158ae5ff5b6e2ee2d7740090f8379d. Reason for revert: b/231244396 Change-Id: Ibc922d80fd85c7a8060d9a8ec667345643dce614
* | | | | Un-obscure the Telephony phone account ID.Tyler Gunn2022-04-261-8/+25
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With the upcoming changes in T to move telephony phone account handles to use subid instead of ICCID, the id of telephony phone account handles no longer needs to be Log.pii-ed. These phone accounts have been hard to diagnose in most bug reports due to the obfuscation. Bug: 226972613 Test: Manual testing; place calls and verify Telephony account handles are now visible. Change-Id: I5ff66571658c2b81b83e3f7d1936e3e805598769
* | | | Merge "Add telecom log marking shell command." am: 5b04c63c64 am: b984978ad7 ↵Tyler Gunn2022-04-131-0/+5
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | am: 5f44a37baa Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/2061586 Change-Id: I8b7c0ed95bb77dd0761eebafa2deb645ec728163 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| * | | Merge "Add telecom log marking shell command." am: 5b04c63c64 am: b984978ad7Tyler Gunn2022-04-131-0/+5
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/2061586 Change-Id: Ie11aa20e65536009fab4663bcc362d62facde961 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| | * | | Add telecom log marking shell command.Tyler Gunn2022-04-121-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We often get bugs where testers have executed a LONG list of test steps. It becomes extremely difficult to know where in the logs corresponds to which test script step. This adds a new telecom shell command: adb shell telecom log-mark Testing 1 2 3 This command adds a USER_LOG_MARK event log item to all ongoing calls and to the general telecom logs with the specified message. Test: Tested using the new shell command to mark the telecom logs in and out of calls. Fixes: 228895073 Change-Id: Ic11edc777e57a4fc8748a42c931379faf7cea671
| * | | | Merge "limit TelecomManager#registerPhoneAccount to 10; api doc update" into ↵Tyler Gunn2022-04-071-0/+5
| |\ \ \ \ | | |/ / / | |/| / / | | |/ / | | | | | | | | | | | | | | | | | | | | qt-dev am: 08f8988bc3 am: 8193edd104 am: 74f9b7f432 am: 6b36ce6895 am: e518ef6409 am: 4636aea564 am: d4e7e2336a Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/16908080 Change-Id: Ibf4807763da204c162015da29468328bb1a7e5a9 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| | * | Merge "limit TelecomManager#registerPhoneAccount to 10; api doc update" into ↵Tyler Gunn2022-04-071-0/+5
| | |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | qt-dev am: 08f8988bc3 am: 8193edd104 am: 74f9b7f432 am: 6b36ce6895 am: e518ef6409 Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/16908080 Change-Id: Ie04277d055fb481438470087819c529a7392903b Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| | | * Merge "limit TelecomManager#registerPhoneAccount to 10; api doc update" into ↵Tyler Gunn2022-04-071-0/+5
| | | |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | qt-dev am: 08f8988bc3 am: 8193edd104 am: 74f9b7f432 am: 6b36ce6895 Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/16908080 Change-Id: Ifc65d6af6a1623ab2c4d6d25629ebad376cb5728 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| | | | * Merge "limit TelecomManager#registerPhoneAccount to 10; api doc update" into ↵Tyler Gunn2022-04-071-0/+5
| | | | |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | qt-dev am: 08f8988bc3 am: 8193edd104 am: 74f9b7f432 Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/16908080 Change-Id: I9264ab8f95bec38b438024cc7a65a6b6b720e6fd Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| | | | | * Merge "limit TelecomManager#registerPhoneAccount to 10; api doc update" into ↵Tyler Gunn2022-04-071-0/+5
| | | | | |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | qt-dev am: 08f8988bc3 am: 8193edd104 Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/16908080 Change-Id: I73cf3414a8bd2067fa673fe198244c86eb2726c7 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| | | | | | * Merge "limit TelecomManager#registerPhoneAccount to 10; api doc update" into ↵Tyler Gunn2022-04-061-0/+5
| | | | | | |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | qt-dev am: 08f8988bc3 Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/16908080 Change-Id: I7843acc6edf27ed2d8eece6dc2de7a05aa925b9b Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| | | | | | | * limit TelecomManager#registerPhoneAccount to 10; api doc updateThomas Stuart2022-03-071-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bug: 209814693 Bug: 217934478 Test: CTS Change-Id: I8e4425a4e7de716f86b1f1f56ea605d93f357a57 Merged-In: I8e4425a4e7de716f86b1f1f56ea605d93f357a57
* | | | | | | | Merge "Add new TelecomManager method to check if a self-mgd app is in a ↵Tyler Gunn2022-04-042-0/+34
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | call." into tm-dev
| * | | | | | | | Add new TelecomManager method to check if a self-mgd app is in a call.Tyler Gunn2022-03-252-0/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adding a new method which NotificationManager will use to check if a VOIP app is currently engaged in any self-managed calls. Structured to accept a package name and userhandle rather than a PhoneAccountHandle so that NotificationManager doesn't need to query all phone account handles to find the right one to pass in. NotificationMgr is part of the system service, so this @hide method is sufficient. We can consider making this a system API in the future when we mainline Telecom. Test: Manual test with Telecom test app to validate method behavior. Test: Added new unit test to exercise backing functionality for new method. Bug: 226959797 Change-Id: I74c0568ad2ec308fe50cada7315e3a677363cccc
* | | | | | | | | rename renounce fine location dataSooraj Sasindran2022-03-311-4/+5
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | rename renounce fine location data to include location data Bug: 210923726 Test: cts Merged-In: Ic2ce84f51bf394031903ae180788216cfa910fc0 Change-Id: Ic2ce84f51bf394031903ae180788216cfa910fc0
* | | | | | | | [API Review] onCreateUnknownConnectionEtienne Ruffieux2022-03-081-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cherry-picked onCreateUnknownConnection made system API from internal master, added precision on SystemAPI annotation for client modules libraries as per API review. Bug: 219820274 Tag: #feature Test: manual Change-Id: I1479201705a3ecce6fe69c2e78d30ce0b31cc3d8
* | | | | | | | Merge "Update API documentation for onRttInitiationFailure." am: 7ee20f2830 ↵Treehugger Robot2022-03-042-3/+24
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 5badbebbf9 am: 2e658ce6fe Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1990510 Change-Id: I5c53a4d30a9dd1143d4bb723df69041bf2a355b2
| * | | | | | | Update API documentation for onRttInitiationFailure.Tyler Gunn2022-02-172-3/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add intdef for reason code. Clarifying when this callback can be expected and how to handle RTT state when requested at the start of a call. Test: Docs only change Bug: 216608696 Change-Id: I86e46fbb2b0693564a41d1ca0d9f8c180d775c1a
| * | | | | | | Merge changes from topic "phonenumber"Meng Wang2022-01-071-0/+4
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * changes: Update javadocs based on API council feedback. Deprecate get/setLine1Number in favor of get/setPhoneNumber
| | * | | | | | | Deprecate get/setLine1Number in favor of get/setPhoneNumberMeng Wang2022-01-051-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 193539271 Test: make Change-Id: I09bcdca9d44d6b6a6d4ca00b6d0bba4c45a39bac Merged-In: I09bcdca9d44d6b6a6d4ca00b6d0bba4c45a39bac (cherry picked from commit 8665b5105ecc4da80dd60296bac7f3e630edeef5)
* | | | | | | | | Merge "Revert "Add API for external call audio route."" into tm-devTreeHugger Robot2022-03-033-35/+5
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Revert "Add API for external call audio route."Grace Jia2022-02-283-35/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Revert "Add CTS coverage for new external audio route API." Revert submission 16308028-external-audio Reason for revert: Don't want to launch this in T now Reverted Changes: I6634c752a:Add CTS coverage for new external audio route API.... Ibd1c2880e:Add API for external call audio route. I3e262a913:Implement new external audio route. Change-Id: I3a3eab1151b825523499058040c14696ea2887e9
* | | | | | | | | | decouple TelecomManager#getSelfManagedPhoneAccounts (2/3)Thomas Stuart2022-02-282-16/+41
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Recently TelecomManager#getSelfManagedPhoneAccounts was updated to allow callingPackages to get their PA's. However, API council noted there was ambiguity with what was returned from the method. Thus the method has been decoupled into 2 methods. bug: 216640181 Test: CTS Change-Id: Idbcb4740f5a5d7fdf5bedb9d9f6173eef0cb3f17
* | | | | | | | | Revert "Add API for cross device calling."Grace Jia2022-02-0718-1075/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Revert "Add cts tests for new cross device calling api" Revert submission 16308056-cross-api Reason for revert: Don't want to launch this in T now Reverted Changes: I731226c0b:Add cts tests for new cross device calling api I1a3aa6c1a:Add API for cross device calling. I40f78925b:Implement new TelecomManager API for cross device ... Change-Id: I687d94235b159d8daa20fe75de08394adc39f8c7
* | | | | | | | | Merge "limit TelecomManager#registerPhoneAccount to 10; api doc update"Thomas Stuart2022-02-041-0/+5
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | limit TelecomManager#registerPhoneAccount to 10; api doc updateThomas Stuart2022-02-011-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bug: 209814693 Test: CTS Change-Id: I8e4425a4e7de716f86b1f1f56ea605d93f357a57
* | | | | | | | | | Adding SystemApi for remaining Telecomm errorsRoopa Sattiraju2022-02-011-5/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 206121418 Test: Compile apex Change-Id: I7a8511f48b2e243569cd38e13501124a5b1bb5b2
* | | | | | | | | | API changes requires for Bluetooth apk inside the apex to build.Roopa Sattiraju2022-01-281-6/+19
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 216731966 Test: Compile and Test Change-Id: Ia9c09fb9ca83141419c9f41ef5c248385fb5a9c0
* | | | | | | | | Merge "APIs for SIM call manager voice status"Hunter Knepshield2022-01-261-3/+47
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | APIs for SIM call manager voice statusHunter Knepshield2022-01-241-3/+47
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This CL creates two new PhoneAccount capabilities for the SIM call manager (and potentially telephony itself for PSTN PhoneAccounts) to advertise that they support voice calling, potentially independent of voice service state. With this information, telecom will feed updated back into telephony for plumbing through the standard ServiceState-based APIs. Bug: 205737545 Test: atest android.telecom.cts.PhoneAccountOperationsTest (an additional CTS test for the TelephonyManager API will be added as a follow-up) Change-Id: If7a5b03a7aa630823ac238ef27721e5e6cc596e6
* | | | | | | | | | Merge "TelecomManager#getSelfManagedPA comments update (3/3)"Thomas Stuart2022-01-241-4/+14
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | TelecomManager#getSelfManagedPA comments update (3/3)Thomas Stuart2022-01-241-4/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bug: 209839019 Test: CTS Change-Id: I152e0ee7bd749598a1822f34566b66a50df04863
* | | | | | | | | | | Merge "Add API for cross device calling."Grace Jia2022-01-2418-45/+1075
|\ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / |/| | | | | | | | | |
| * | | | | | | | | | Add API for cross device calling.Grace Jia2022-01-1218-45/+1075
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Test: CTS test Change-Id: I1a3aa6c1ae6d445a2f3b55e5f0d11918da5bed33
* | | | | | | | | | | Merge "Revert "Revert "Migrate unsafe parcel APIs in framework-minus-apex"""TreeHugger Robot2022-01-2010-33/+33
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Revert "Revert "Migrate unsafe parcel APIs in framework-minus-apex""Bernardo Rufino2022-01-1910-33/+33
| | |/ / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 331be9a6431d6489f8d1e1b80cb510d0ee073c50. Reintroducing ag/16366278 since it seems unrelated to b/214053959 (more details on b/214053959#comment55). Original commit message: Migrate unsafe parcel APIs in framework-minus-apex Migrate the following unsafe parcel APIs in framework-minus-apex: * Parcel.readSerializable() * Parcel.readArrayList() * Parcel.readList() * Parcel.readParcelable() * Parcel.readParcelableList() * Parcel.readSparseArray() This CL was generated by applying lint fixes that infer the expected type from the caller code and provide that as the type parameter (ag/16365240). A few observations: * In some classes we couldn't migrate because the class also belonged to another build module whose min SDK wasn't current (as is the case for framework-minus-apex), hence I suppressed the lint check (since I'll eventually submit the lint check to the tree). * In some cases, I needed to do the cast in https://stackoverflow.com/a/1080525/5765705 to make the compiler happy since there isn't another way of providing a class of type Class<MyClassWithGenerics<T>>. * In the readSerializable() case, the new API also requires the class loader, that was inferred to by InferredClass.class.getClassLoader(). * Note that automatic formatting and import rely on running hooked up to the IDE, which wasn't the case here. Bug: 195622897 Change-Id: I272432e6e082a973f7a50492ec35d79c2b577c93 Test: TH passes
generated by cgit v1.2.3 (git 2.25.1) at 2026-01-12 09:12:17 +0000