• Dominick Ng's avatar
    Call ArcAppListPrefs::Observer::{OnPackage{Added,Removed}} from OnPackageListRefreshed. · b2e6942f
    Dominick Ng authored
    ArcAppListPref's browser-side cache of ARC apps may become out of date
    with the ARC container, as the async IPC communication pipe between
    container and browser may be flaky, or be established after the
    container has processed some package installations/removals. This means
    observer classes which follow the OnPackageAdded or OnPackageRemoved
    calls may miss installations/uninstallations.
    
    This CL augments ArcAppListPrefs::OnPackageListRefreshed to additionally
    call observers if it detects packages which have been installed or
    uninstalled which it has not yet seen.
    
    BUG=893927
    
    Change-Id: Icd069fdc89e348918f1185102e566bfc772087e3
    Reviewed-on: https://chromium-review.googlesource.com/c/1345705Reviewed-by: default avatarBen Wells <benwells@chromium.org>
    Reviewed-by: default avatarYury Khmel <khmel@chromium.org>
    Commit-Queue: Dominick Ng <dominickn@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#612055}
    b2e6942f
chrome_app_icon_unittest.cc 14 KB