| commit | 1a5cb387d6542391d0edf69cdd828e5b7c8a9d65 | [log] [tgz] |
|---|---|---|
| author | Bernardo Rufino <brufino@google.com> | Fri Jan 14 17:35:36 2022 +0000 |
| committer | Bernardo Rufino <brufino@google.com> | Wed Jan 19 11:13:18 2022 +0000 |
| tree | e9901a7364aafc761bc13c0bb16698008e5069ae | |
| parent | d6b531e627ab0fc2dd855220abf545d4df7ce9ae [diff] |
Revert "Revert "Migrate unsafe parcel APIs in framework-minus-apex"" 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