tree 6ce1d9f660893545508009d548d0832740450608
parent 7509bd2feb952fc9ced34894c54af0e01f1eeab4
author Daniel Santiago Rivera <danysantiago@google.com> 1652987278 -0700
committer Daniel Santiago Rivera <danysantiago@google.com> 1653075765 -0700

Fix the incorrect usage of a wrapper cursor in entity cursor converters.

Two things:
1. EntityRowAdapter should not wrap a cursor unless the indices provided are not the default ones, i.e. only when the indices are remapped for duplicate columns should it wrap the cursor, otherwise the column indices don't change and the generated cursor entity converter already finds them by name.
2. The wrapped cursor in CursorUtil is now a `CursorWrapper` and not a delegate of `AbstractCursor` to avoid risking missing overriding methods, such as getBlob().

Bug: 232947402
Test: DaoWriterTest and AmbiguousColumnResolverTest
Change-Id: If47b5484999653b244da50fbb01d4e954541581c