• Toni Barzic's avatar
    Use hotseat size instead of shelf size when laying out app list · ccb54485
    Toni Barzic authored
    AppListView uses AppListViewDelegate::GetShelfSize() to get the shelf
    size for calculating:
    *   background shield rounded corners.
    *   the app list layout.
    
    For both purposes, app list can assume that the shelf height equals
    non-in-app height (i.e. assume that in-app shelf is not used).
    The former is only used in clamshell mode. For the later, app list will
    not be shown with the in-app shelf (at least in static state).
    
    Though, while transitioning from app window to app list (e.g. while
    dragging the app window), app list view will be laid out while in-app
    shelf shown - if the actual current shelf height is used to calculate
    the layout, the app list grid items might jump up once the transition
    completes (as the shelf height changes and the app list gets laid out
    again with less vertical space available).
    
    This can be avoided if app list view always assumed that the shelf size
    equals the non-in-app size, as done in this cl.
    
    BUG=1013706
    
    Change-Id: I5175781baefe1e92cf8abcd92185b1b3fdf8dc19
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1857200Reviewed-by: default avatarManu Cornet <manucornet@chromium.org>
    Reviewed-by: default avatarAlex Newcomer <newcomer@chromium.org>
    Commit-Queue: Toni Baržić <tbarzic@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#713184}
    ccb54485
shelf_config.h 8.23 KB