• Dominick Ng's avatar
    Flush InstallableManager state when the manifest URL changes. · ae3d5351
    Dominick Ng authored
    InstallableManager caches the data it fetches as part of the
    installability check. This means that sites which change their manifest,
    or inject a manifest after the check is run are incorrectly classified.
    
    This CL makes InstallableManager listen to the
    WebContentsObserver::DidUpdateWebManifestURL() method, and invalidates
    its internal state when the method fires. This addresses the issue by
    making InstallableManager refetch data when we know that the manifest
    URL is different.
    
    Tests are added to ensure the correct behaviour.
    
    BUG=792299
    
    Change-Id: If73128b7b86bd741c16235c8a8e7ec5f0caeb165
    Reviewed-on: https://chromium-review.googlesource.com/809994
    Commit-Queue: Dominick Ng <dominickn@chromium.org>
    Reviewed-by: default avatarBen Wells <benwells@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#522038}
    ae3d5351
main.js 870 Bytes