Commit 50e8c4c6 authored by Maksim Ivanov's avatar Maksim Ivanov Committed by Commit Bot

Fix use-after-move in //chrome/browser/profile_resetter/

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

Bug: 1122844
Change-Id: Ie1a0515f544e0832181e85023cfae922df2ee430
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2382593
Commit-Queue: Dominic Battré <battre@chromium.org>
Reviewed-by: default avatarDominic Battré <battre@chromium.org>
Cr-Commit-Position: refs/heads/master@{#803794}
parent 8b18acc9
......@@ -86,11 +86,13 @@ void ResetReportUploader::DispatchReportInternal(
std::unique_ptr<network::SimpleURLLoader> simple_url_loader =
network::SimpleURLLoader::Create(std::move(resource_request),
traffic_annotation);
network::SimpleURLLoader* const simple_url_loader_ptr =
simple_url_loader.get();
simple_url_loader->AttachStringForUpload(request_data,
"application/octet-stream");
auto it = simple_url_loaders_.insert(simple_url_loaders_.begin(),
std::move(simple_url_loader));
it->get()->DownloadToStringOfUnboundedSizeUntilCrashAndDie(
simple_url_loader_ptr->DownloadToStringOfUnboundedSizeUntilCrashAndDie(
url_loader_factory_.get(),
base::BindOnce(&ResetReportUploader::OnSimpleLoaderComplete,
base::Unretained(this), std::move(it)));
......
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