• Toni Barzic's avatar
    Update home arrow visibility when starting to show home launcher · 9c9eb002
    Toni Barzic authored
    Removes HomeScreenDelegate::UpdateAfterHomeLauncherShown, which was only
    updating expand arrow visibility, and move the logic to
    HomeScreenDelegate::OnHomeLauncherTargetPositionChanged.
    The method was used to update expand arrow visibility when home launcher
    is shown. Given that OnHomeLauncherTargetPositionChanged is expected to
    get called during the same sequence, expand arrow visilibity can be
    moved there without losing much, and reduces the number of methods that
    need to be called while using gestures to show/hide home launcher.
    
    Also, updates how app list progress is calculated when app list is
    scaled in tablet mode to ignore app list translation that was added
    during scaling (without this, if arrow is painted while app list view is
    scaled down, it might be slightly bent).
    
    BUG=1022530
    
    Change-Id: Ie284b413bdc576121757c1c4bb1b4280c97e06b8
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1904235Reviewed-by: default avatarXiyuan Xia <xiyuan@chromium.org>
    Reviewed-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@{#714006}
    9c9eb002
home_screen_delegate.h 4.91 KB