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

Fix use-after-move in //chromeos/services/multidevice_setup/

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

Bug: 1122844
Change-Id: I8ad7887c8903f907699d1fd5f98bf4cb8f6bb42f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2556957Reviewed-by: default avatarDaniel Cheng <dcheng@chromium.org>
Reviewed-by: default avatarJon Mann <jonmann@chromium.org>
Auto-Submit: Maksim Ivanov <emaxx@chromium.org>
Commit-Queue: Maksim Ivanov <emaxx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#831966}
parent 6a21a5c4
...@@ -34,8 +34,9 @@ AndroidSmsAppInstallingStatusObserver::Factory::Create( ...@@ -34,8 +34,9 @@ AndroidSmsAppInstallingStatusObserver::Factory::Create(
AndroidSmsAppHelperDelegate* android_sms_app_helper_delegate, AndroidSmsAppHelperDelegate* android_sms_app_helper_delegate,
PrefService* pref_service) { PrefService* pref_service) {
if (test_factory_) { if (test_factory_) {
test_factory_->CreateInstance(host_status_provider, feature_state_manager, return test_factory_->CreateInstance(
std::move(android_sms_app_helper_delegate)); host_status_provider, feature_state_manager,
std::move(android_sms_app_helper_delegate));
} }
return base::WrapUnique(new AndroidSmsAppInstallingStatusObserver( return base::WrapUnique(new AndroidSmsAppInstallingStatusObserver(
host_status_provider, feature_state_manager, host_status_provider, feature_state_manager,
......
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