Commit 72654def authored by Maksim Ivanov's avatar Maksim Ivanov Committed by Commit Bot

Fix use-after-move in //content/browser/download/

Fix use-after-move (potential) bugs found by the
"bugprone-use-after-move" clang-tidy check.

Bug: 1122844
Change-Id: I1aa8dc85cdb849382d1dc6ece9b03d0c435cf4ae
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2382852Reviewed-by: default avatarMin Qin <qinmin@chromium.org>
Commit-Queue: Maksim Ivanov <emaxx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#803124}
parent 3da3b889
...@@ -1402,8 +1402,8 @@ void DownloadManagerImpl::BeginDownloadInternal( ...@@ -1402,8 +1402,8 @@ void DownloadManagerImpl::BeginDownloadInternal(
std::move(web_contents_getter), url, method, std::move(initiator), std::move(web_contents_getter), url, method, std::move(initiator),
false /* from_download_cross_origin_redirect */, content_initiated, false /* from_download_cross_origin_redirect */, content_initiated,
std::move(on_can_download_checks_done)); std::move(on_can_download_checks_done));
return;
} }
return;
} }
BeginResourceDownloadOnChecksComplete( BeginResourceDownloadOnChecksComplete(
......
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