• Clemens Arbesser's avatar
    Reland "[Autofill Assistant] Allow multiple chips in the header." · ae596325
    Clemens Arbesser authored
    This is a reland of 81959047
    
    There was a missing notification in our bottom bar coordinator, which
    led to the bottom sheet not resizing properly. No idea why this passed
    CQ last time, as the test should have failed every time.
    
    Original change's description:
    > [Autofill Assistant] Allow multiple chips in the header.
    >
    > This CL allows the header to support a full carousel.
    >
    > What it does:
    >  - Move header chip logic into native.
    >  - Use RecyclerView instead of hard-coded single chip view.
    >  - Potentially fixes an issue where the status message does not stretch
    > across the whole available width.
    >  - Chips in the header now support change animations.
    >
    > There is an issue caused by a layout resize when switching between PEEK
    > and non-PEEK mode. This is further discussed in the linked bug. The
    > workaround is to delay the layout resize until the sheet state has
    > settled.
    >
    > Bug: b/160856358
    > Change-Id: Id7617ed41119b85fdaa00ebaff7e856984b74b0a
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2288567
    > Commit-Queue: Clemens Arbesser <arbesser@google.com>
    > Reviewed-by: Mathias Carlen <mcarlen@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#796376}
    
    Bug: b/160856358
    Change-Id: If3acb7cc6fabcbab7ce9bc82c0664c53110b5cd3
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2350419Reviewed-by: default avatarMathias Carlen <mcarlen@chromium.org>
    Commit-Queue: Clemens Arbesser <arbesser@google.com>
    Cr-Commit-Position: refs/heads/master@{#797600}
    ae596325
ui_controller_android.cc 71.2 KB