chromium/src
0
Fork 0
Code Activity

Use Value::Take...() in /pdf

Browse Source 
Refactor to use the new base::Value
TakeString/TakeDict/TakeList methods that transfer ownership
in a clearer way and allow clang-tidy catch misuses.

This CL was uploaded by git cl split.

R=nigi@chromium.org

Bug: 1360570
Change-Id: Ib6931e28ed8fdeb5d0acc987b12c67d12d554c92
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3913062
Reviewed-by: Nigi <nigi@chromium.org>
Commit-Queue: Maksim Ivanov <emaxx@chromium.org>
Auto-Submit: Maksim Ivanov <emaxx@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1051999}
This commit is contained in:
Maksim Ivanov
2022-09-27 19:55:04 +00:00
committed by Chromium LUCI CQ
parent 9d7be578ed
commit 97ed1dfec7
1 changed files with 1 additions and 1 deletions

2
pdf/post_message_receiver.cc

@@ -129,7 +129,7 @@ void PostMessageReceiver::PostMessage(v8::Local<v8::Value> message) {
client_task_runner_->PostTask( client_task_runner_->PostTask(
FROM_HERE, base::BindOnce(&Client::OnMessage, client_, FROM_HERE, base::BindOnce(&Client::OnMessage, client_,
std::move(converted_message->GetDict()))); std::move(*converted_message).TakeDict()));
} }
} // namespace chrome_pdf } // namespace chrome_pdf