| commit | 90bb3709dc75f7e44914222114752de5bce133d4 | [log] [tgz] |
|---|---|---|
| author | Bernardo Rufino <brufino@google.com> | Tue Dec 07 20:01:45 2021 +0000 |
| committer | Bernardo Rufino <brufino@google.com> | Wed Dec 15 18:21:38 2021 +0000 |
| tree | eb1795c3399812b3d7f5cee7e41fbd718fb64f4f | |
| parent | 17dd9c30d6b22cc7cf4169028fd29ca797f3d7bf [diff] |
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 Test: TH passes Change-Id: I11a27b9bdab7959ee86e90aa1e1cbebd7aaf883c