Commit 9bece2a1 authored by Xing Liu's avatar Xing Liu Committed by Commit Bot

Don't use http cache for offline page download.

Most DownloadManager::DownloadUrl use cases don't use cache because
cache can result in download with a partial file, and it can block
video thumbnail generation when the partial file is too small.

Bug: 909605
Change-Id: I40c8dae00da677d969f1a3c398b758d4e1d230a4
Reviewed-on: https://chromium-review.googlesource.com/c/1357501Reviewed-by: default avatarJian Li <jianli@chromium.org>
Commit-Queue: Xing Liu <xingliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#612807}
parent 8f1d8942
......@@ -287,8 +287,6 @@ void DownloadAsFile(content::WebContents* web_contents, const GURL& url) {
dl_params->set_referrer(referrer.url);
dl_params->set_referrer_policy(
content::Referrer::ReferrerPolicyForUrlRequest(referrer.policy));
dl_params->set_prefer_cache(true);
dl_params->set_prompt(false);
dl_params->set_download_source(download::DownloadSource::OFFLINE_PAGE);
dlm->DownloadUrl(std::move(dl_params));
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment