• Antonio Gomes's avatar
    Set more URLLoaderCompletionStatus data for successful FileURLLoaderFactory loads · 402dbf5a
    Antonio Gomes authored
    This CL allows SimpleURLLoader's to successfully load file:// URLs, ie when
    using FileURLLoaderFactory.
    
    Prior to this CL, even on the file specified in the file:// URL is written
    successfully, the URLLoaderCompletionStatus instance is passed to the
    URLLoaderClient from FileURLLoader::OnFileWritten, had only with the
    |error_code| set.
    
    SimpleURLLoader checks other URLLoaderCompletionStatus class members,
    (eg |decoded_body_length|) in order to consider the load actually successful.
    
    BUG=773295,844928
    
    Change-Id: I1040775fad2af7f0eb88f81e48bb355d48e5fc31
    Reviewed-on: https://chromium-review.googlesource.com/1159042Reviewed-by: default avatarJohn Abd-El-Malek <jam@chromium.org>
    Commit-Queue: Antonio Gomes <tonikitoo@igalia.com>
    Cr-Commit-Position: refs/heads/master@{#580130}
    402dbf5a
file_url_loader_factory.cc 26.5 KB