• Dan Harrington's avatar
    Use the final URL in the intent to open an auto-fetch page · ad01f920
    Dan Harrington authored
    This prevents a bug which can occurr for URLs that redirect. The bug is a bit
    subtle, so see the crbug.com/937581 for more details.
    
    We're almost guaranteed that Chrome hasn't seen a redirect for the final URL,
    so this will fix the problem of redirecting before hitting the offline url
    loader request interceptor. We will intercept requests for either the original
    URL or the final URL.
    
    Note that because we're using a different URL in the intent, when a URL is a
    redirect, we will always open the page in a new tab. This is not ideal, but
    is preferable to failing to open the page.
    
    Bug: 937581
    Change-Id: I04f02556e93eba1ca0b2380313f1052c7f8f7cd3
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1505895
    Commit-Queue: Dan H <harringtond@google.com>
    Reviewed-by: default avatarCarlos Knippschild <carlosk@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#638312}
    ad01f920
auto_fetch_notifier.cc 2.5 KB