Commit b6ccba73 authored by Maksim Ivanov's avatar Maksim Ivanov Committed by Commit Bot

Fix useless move() in //content/browser/webauth/authenticator_common.cc

Drop std::move() call that has no effect and only triggers the
"bugprone-use-after-move" clang-tidy warning.

Bug: 1122844
Change-Id: I04d191a6a73964dbd58b61bce722d5885b140c59
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2383021
Commit-Queue: Martin Kreichgauer <martinkr@google.com>
Reviewed-by: default avatarMartin Kreichgauer <martinkr@google.com>
Cr-Commit-Position: refs/heads/master@{#803097}
parent f90be856
...@@ -1038,8 +1038,7 @@ void AuthenticatorCommon::GetAssertion( ...@@ -1038,8 +1038,7 @@ void AuthenticatorCommon::GetAssertion(
base::BindOnce(&AuthenticatorCommon::OnTimeout, base::Unretained(this))); base::BindOnce(&AuthenticatorCommon::OnTimeout, base::Unretained(this)));
ctap_get_assertion_request_ = CreateCtapGetAssertionRequest( ctap_get_assertion_request_ = CreateCtapGetAssertionRequest(
client_data_json_, std::move(options), app_id_, client_data_json_, options, app_id_, browser_context()->IsOffTheRecord());
browser_context()->IsOffTheRecord());
ctap_get_assertion_options_.emplace(); ctap_get_assertion_options_.emplace();
bool is_first = true; bool is_first = true;
......
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