• Tsuyoshi Horo's avatar
    Keep SSLInfo for SXG perfetching · af905953
    Tsuyoshi Horo authored
    Currently SSLInfo is removed by SignedExchangeLoader::OnHTTPExchangeFound() when
    kURLLoadOptionSendSSLInfoWithResponse flag is not set. So this CL changes
    SignedExchangePrefetchHandler to set the flag to store the SSLInfo in
    PrefetchedSignedExchangeCache. And changes InnerResponseURLLoader to clear the
    SSLInfo if the request is not for main frame main resource and
    report_raw_headers flag is not set.
    
    The case of issue 998359 should be covered by
    SignedExchangeRequestHandlerBrowserTest.Simple/3. But this test was not executed
    as expected. The cached SXG in PrefetchedSignedExchangeCache was not correctly
    used for the next navigation. So this CL changes the test to use JS to trigger
    the navigation from renderer, and to use the overridden verification_time in
    PrefetchedSignedExchangeCache.
    
    Bug: 998359
    Change-Id: I614d4a39bddfd55290e7ff632565ed7bf1c75229
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1771902Reviewed-by: default avatarTsuyoshi Horo <horo@chromium.org>
    Reviewed-by: default avatarKunihiko Sakamoto <ksakamoto@chromium.org>
    Reviewed-by: default avatarKinuko Yasuda <kinuko@chromium.org>
    Commit-Queue: Tsuyoshi Horo <horo@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#691643}
    af905953
signed_exchange_browser_test_helper.cc 3.81 KB