Commit 9ab5fbd3 authored by Maksim Ivanov's avatar Maksim Ivanov Committed by Commit Bot

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

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

Bug: 1122844
Change-Id: I0d24fdc82118ff3d62609500d859507742a8cb03
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2383653Reviewed-by: default avatarMarijn Kruisselbrink <mek@chromium.org>
Commit-Queue: Maksim Ivanov <emaxx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#803184}
parent bdb56ebb
...@@ -305,11 +305,12 @@ void AppCacheInternalsUI::Proxy::OnResponseInfoLoaded( ...@@ -305,11 +305,12 @@ void AppCacheInternalsUI::Proxy::OnResponseInfoLoaded(
std::unique_ptr<AppCacheResponseReader> reader = std::unique_ptr<AppCacheResponseReader> reader =
appcache_service_->storage()->CreateResponseReader( appcache_service_->storage()->CreateResponseReader(
GURL(response_enquiry.manifest_url), response_enquiry.response_id); GURL(response_enquiry.manifest_url), response_enquiry.response_id);
AppCacheResponseReader* const reader_ptr = reader.get();
reader->ReadData(response_data.get(), amount_to_read, reader_ptr->ReadData(response_data.get(), amount_to_read,
base::BindOnce(&Proxy::OnResponseDataReadComplete, this, base::BindOnce(&Proxy::OnResponseDataReadComplete,
response_enquiry, response_info, this, response_enquiry, response_info,
std::move(reader), response_data)); std::move(reader), response_data));
} else { } else {
OnResponseDataReadComplete(response_enquiry, nullptr, nullptr, nullptr, -1); OnResponseDataReadComplete(response_enquiry, nullptr, nullptr, nullptr, -1);
} }
......
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