Commit 6af764ec authored by Maksim Ivanov's avatar Maksim Ivanov Committed by Chromium LUCI CQ

Fix use-after-move in //c/b/ui/app_list/arc/

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

Bug: 1122844
Change-Id: I5100597a1444b530b2f3fef58af7493499ba12de
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2556637
Auto-Submit: Maksim Ivanov <emaxx@chromium.org>
Commit-Queue: Long Cheng <lgcheng@google.com>
Reviewed-by: default avatarLong Cheng <lgcheng@google.com>
Cr-Commit-Position: refs/heads/master@{#834536}
parent bf8e828c
...@@ -386,8 +386,10 @@ void ArcUsbHostPermissionManager::MaybeProcessNextPermissionRequest() { ...@@ -386,8 +386,10 @@ void ArcUsbHostPermissionManager::MaybeProcessNextPermissionRequest() {
*current_request.usb_device_entry())) { *current_request.usb_device_entry())) {
OnUsbPermissionReceived(std::move(current_request), true); OnUsbPermissionReceived(std::move(current_request), true);
} else { } else {
const base::string16 device_name =
current_request.usb_device_entry()->device_name;
ShowUsbAccessPermissionDialog( ShowUsbAccessPermissionDialog(
profile_, app_id, current_request.usb_device_entry()->device_name, profile_, app_id, device_name,
base::BindOnce(&ArcUsbHostPermissionManager::OnUsbPermissionReceived, base::BindOnce(&ArcUsbHostPermissionManager::OnUsbPermissionReceived,
weak_ptr_factory_.GetWeakPtr(), weak_ptr_factory_.GetWeakPtr(),
std::move(current_request))); std::move(current_request)));
......
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