commit 7b1bb962f680700e220759104b231e3eb48777af
author Claudio DeSouza <claudiomdsjr@gmail.com> Wed Feb 21 17:27:16 2018
committer Commit Bot <commit-bot@chromium.org> Wed Feb 21 17:27:16 2018
tree 2c16c09e3ae6912c52ac22622b91048782c498a8
parent 7212bc1a0c35da48ee0065a865f45ebb6e1ef4c6

Remove some use of base::Passed

As proposed on chromium-dev recently, there seem to be no reason to keep
base::Passed around. In most cases where base::Passed is used, OnceCallback
should be preferred, specially because base::Passed is going to transfer
ownership of whatever was passed on its following run, and mostly behaving as
the intended purpose for OnceCallback.

This CL provides a good number of changes for the most straight forward cases
that do not involve changing any public interface. The code changed is mainly
from //base, //component, //crypto, and //device.

Original forum discussion can be found here: https://groups.google.com/a/chromium.org/d/msgid/chromium-dev/abdff810-4359-41a5-8aa9-cd596c784bf5%40chromium.org?utm_medium=email&utm_source=footer

Bug: 812523
Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester;master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I7b8c95cee091475602dca240fca9fa843218c563
Reviewed-on: https://chromium-review.googlesource.com/923262
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Max Morin <maxmorin@chromium.org>
Commit-Queue: Max Morin <maxmorin@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#538143}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 591a997f9b42806d3af3dec8b5fd742dfd1d1386

nss_util.cc

diff --git a/nss_util.cc b/nss_util.cc
index e5a6d6f..b5aabd3 100644
--- a/nss_util.cc
+++ b/nss_util.cc
@@ -325,7 +325,7 @@
                        system_slot_id, tpm_args_ptr),
         base::BindOnce(&NSSInitSingleton::OnInitializedTPMTokenAndSystemSlot,
                        base::Unretained(this),  // NSSInitSingleton is leaky
-                       callback, base::Passed(&tpm_args)));
+                       callback, std::move(tpm_args)));
     initializing_tpm_token_ = true;
   }
 
@@ -488,7 +488,7 @@
                        slot_id, tpm_args_ptr),
         base::BindOnce(&NSSInitSingleton::OnInitializedTPMForChromeOSUser,
                        base::Unretained(this),  // NSSInitSingleton is leaky
-                       username_hash, base::Passed(&tpm_args)));
+                       username_hash, std::move(tpm_args)));
   }
 
   void OnInitializedTPMForChromeOSUser(
@@ -538,7 +538,7 @@
       DVLOG(2) << "empty username_hash";
       if (!callback.is_null()) {
         base::ThreadTaskRunnerHandle::Get()->PostTask(
-            FROM_HERE, base::Bind(callback, base::Passed(ScopedPK11Slot())));
+            FROM_HERE, base::BindOnce(callback, ScopedPK11Slot()));
       }
       return ScopedPK11Slot();
     }