• Sky Malice's avatar
    [Feed] Trigger refresh after sign in and out. · 92eb7422
    Sky Malice authored
    Previously the scheduler was being directly called by the
    ChromeBrowsingDataRemoverDelegate when history was deleted. When this
    happened, we cleared internal state and suppressed refreshes. This
    sometimes coincided with when the FeedAppLifecycle cleared all stored
    articles, but not very well. Particularly, when:
    * Signing in
    * Signing out
    * Clear the cache
    
    So instead, the scheduler is notified by the FeedAppLifecycle exactly
    when all articles are cleared. A boolean is also passed along to inform
    the scheduler which of the two states are we in. Did something get cleared
    and we should suppress refreshing? Or do we now have no data and need to
    refresh immediately.
    
    Bug: 905959
    Change-Id: I25c57022a0da52cc42746d265beae1c55f84d3ec
    Reviewed-on: https://chromium-review.googlesource.com/c/1355994
    Commit-Queue: Sky Malice <skym@chromium.org>
    Reviewed-by: default avatarFilip Gorski <fgorski@chromium.org>
    Reviewed-by: default avatarMartin Šrámek <msramek@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#612848}
    92eb7422
feed_scheduler_bridge.cc 4.56 KB