Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(384)

Unified Diff: components/password_manager/content/browser/credential_manager_impl_unittest.cc

Issue 2867743002: Use OnceCallback on Mojo interfaces in //components/password_manager (Closed)
Patch Set: . Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/password_manager/content/browser/credential_manager_impl_unittest.cc
diff --git a/components/password_manager/content/browser/credential_manager_impl_unittest.cc b/components/password_manager/content/browser/credential_manager_impl_unittest.cc
index 9563fc28c69add8b0a0376b41162f8824a27c484..e47c5958190f2e2bc14112473286682656a6c44d 100644
--- a/components/password_manager/content/browser/credential_manager_impl_unittest.cc
+++ b/components/password_manager/content/browser/credential_manager_impl_unittest.cc
@@ -8,6 +8,7 @@
#include <string>
#include <tuple>
+#include <utility>
#include <vector>
#include "base/bind.h"
@@ -350,21 +351,21 @@ class CredentialManagerImplTest : public content::RenderViewHostTestHarness {
// Helpers for testing CredentialManagerImpl methods.
void CallStore(const CredentialInfo& info,
- const CredentialManagerImpl::StoreCallback& callback) {
- cm_service_impl_->Store(info, callback);
+ CredentialManagerImpl::StoreCallback callback) {
+ cm_service_impl_->Store(info, std::move(callback));
}
void CallRequireUserMediation(
- const CredentialManagerImpl::RequireUserMediationCallback& callback) {
- cm_service_impl_->RequireUserMediation(callback);
+ CredentialManagerImpl::RequireUserMediationCallback callback) {
+ cm_service_impl_->RequireUserMediation(std::move(callback));
}
void CallGet(bool zero_click_only,
bool include_passwords,
const std::vector<GURL>& federations,
- const CredentialManagerImpl::GetCallback& callback) {
+ CredentialManagerImpl::GetCallback callback) {
cm_service_impl_->Get(zero_click_only, include_passwords, federations,
- callback);
+ std::move(callback));
}
protected:

Powered by Google App Engine
This is Rietveld 408576698