Use base::IgnoreReturn<bool>( instead of base::IgnoreReturn(base::Callback<bool(void)>(.

chrome/browser/password_manager/native_backend_kwallet_x_unittest.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <algorithm>
 #include <map>
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 478 matching lines @@
 }
 
 TEST_F(NativeBackendKWalletTest, BasicAddLogin) {
   // Pretend that the migration has already taken place.
   profile_.GetPrefs()->SetBoolean(prefs::kPasswordsUseLocalProfileId, true);
 
   NativeBackendKWalletStub backend(42, profile_.GetPrefs());
   EXPECT_TRUE(backend.InitWithBus(mock_session_bus_));
 
   BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-      base::IgnoreReturn(base::Callback<bool(void)>(
-          base::Bind(&NativeBackendKWalletStub::AddLogin,
-                     base::Unretained(&backend), form_google_))));
+      base::IgnoreReturn<bool>(base::Bind(
+          &NativeBackendKWalletStub::AddLogin,
+          base::Unretained(&backend), form_google_)));
 
   RunDBThread();
 
   EXPECT_FALSE(wallet_.hasFolder("Chrome Form Data"));
 
   std::vector<const PasswordForm*> forms;
   forms.push_back(&form_google_);
   ExpectationArray expected;
   expected.push_back(make_pair(std::string(form_google_.signon_realm), forms));
   CheckPasswordForms("Chrome Form Data (42)", expected);
 }
 
 TEST_F(NativeBackendKWalletTest, BasicListLogins) {
   // Pretend that the migration has already taken place.
   profile_.GetPrefs()->SetBoolean(prefs::kPasswordsUseLocalProfileId, true);
 
   NativeBackendKWalletStub backend(42, profile_.GetPrefs());
   EXPECT_TRUE(backend.InitWithBus(mock_session_bus_));
 
   BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-      base::IgnoreReturn(base::Callback<bool(void)>(
-          base::Bind(&NativeBackendKWalletStub::AddLogin,
-                     base::Unretained(&backend), form_google_))));
+      base::IgnoreReturn<bool>(base::Bind(
+          &NativeBackendKWalletStub::AddLogin,
+          base::Unretained(&backend), form_google_)));
 
   std::vector<PasswordForm*> form_list;
   BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-      base::IgnoreReturn(base::Callback<bool(void)>(
-          base::Bind(&NativeBackendKWalletStub::GetAutofillableLogins,
-                     base::Unretained(&backend), &form_list))));
+      base::IgnoreReturn<bool>(base::Bind(
+          &NativeBackendKWalletStub::GetAutofillableLogins,
+          base::Unretained(&backend), &form_list)));
 
   RunDBThread();
 
   // Quick check that we got something back.
   EXPECT_EQ(1u, form_list.size());
   STLDeleteElements(&form_list);
 
   EXPECT_FALSE(wallet_.hasFolder("Chrome Form Data"));
 
   std::vector<const PasswordForm*> forms;
   forms.push_back(&form_google_);
   ExpectationArray expected;
   expected.push_back(make_pair(std::string(form_google_.signon_realm), forms));
   CheckPasswordForms("Chrome Form Data (42)", expected);
 }
 
 TEST_F(NativeBackendKWalletTest, BasicRemoveLogin) {
   // Pretend that the migration has already taken place.
   profile_.GetPrefs()->SetBoolean(prefs::kPasswordsUseLocalProfileId, true);
 
   NativeBackendKWalletStub backend(42, profile_.GetPrefs());
   EXPECT_TRUE(backend.InitWithBus(mock_session_bus_));
 
   BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-      base::IgnoreReturn(base::Callback<bool(void)>(
-          base::Bind(&NativeBackendKWalletStub::AddLogin,
-                     base::Unretained(&backend), form_google_))));
+      base::IgnoreReturn<bool>(base::Bind(
+          &NativeBackendKWalletStub::AddLogin,
+          base::Unretained(&backend), form_google_)));
 
   RunDBThread();
 
   EXPECT_FALSE(wallet_.hasFolder("Chrome Form Data"));
 
   std::vector<const PasswordForm*> forms;
   forms.push_back(&form_google_);
   ExpectationArray expected;
   expected.push_back(make_pair(std::string(form_google_.signon_realm), forms));
   CheckPasswordForms("Chrome Form Data (42)", expected);
 
   BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-      base::IgnoreReturn(base::Callback<bool(void)>(
-          base::Bind(&NativeBackendKWalletStub::RemoveLogin,
-                     base::Unretained(&backend), form_google_))));
+      base::IgnoreReturn<bool>(base::Bind(
+          &NativeBackendKWalletStub::RemoveLogin,
+          base::Unretained(&backend), form_google_)));
 
   RunDBThread();
 
   expected.clear();
   CheckPasswordForms("Chrome Form Data (42)", expected);
 }
 
 TEST_F(NativeBackendKWalletTest, RemoveNonexistentLogin) {
   // Pretend that the migration has already taken place.
   profile_.GetPrefs()->SetBoolean(prefs::kPasswordsUseLocalProfileId, true);
 
   NativeBackendKWalletStub backend(42, profile_.GetPrefs());
   EXPECT_TRUE(backend.InitWithBus(mock_session_bus_));
 
   // First add an unrelated login.
   BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-      base::IgnoreReturn(base::Callback<bool(void)>(
-          base::Bind(&NativeBackendKWalletStub::AddLogin,
-                     base::Unretained(&backend), form_google_))));
+      base::IgnoreReturn<bool>(base::Bind(
+          &NativeBackendKWalletStub::AddLogin,
+          base::Unretained(&backend), form_google_)));
 
   RunDBThread();
 
   EXPECT_FALSE(wallet_.hasFolder("Chrome Form Data"));
 
   std::vector<const PasswordForm*> forms;
   forms.push_back(&form_google_);
   ExpectationArray expected;
   expected.push_back(make_pair(std::string(form_google_.signon_realm), forms));
   CheckPasswordForms("Chrome Form Data (42)", expected);
 
   // Attempt to remove a login that doesn't exist.
   BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-      base::IgnoreReturn(base::Callback<bool(void)>(
-          base::Bind(&NativeBackendKWalletStub::RemoveLogin,
-                     base::Unretained(&backend), form_isc_))));
+      base::IgnoreReturn<bool>(base::Bind(
+          &NativeBackendKWalletStub::RemoveLogin,
+          base::Unretained(&backend), form_isc_)));
 
   // Make sure we can still get the first form back.
   std::vector<PasswordForm*> form_list;
   BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-      base::IgnoreReturn(base::Callback<bool(void)>(
-          base::Bind(&NativeBackendKWalletStub::GetAutofillableLogins,
-                     base::Unretained(&backend), &form_list))));
+      base::IgnoreReturn<bool>(base::Bind(
+          &NativeBackendKWalletStub::GetAutofillableLogins,
+          base::Unretained(&backend), &form_list)));
 
   RunDBThread();
 
   // Quick check that we got something back.
   EXPECT_EQ(1u, form_list.size());
   STLDeleteElements(&form_list);
 
   CheckPasswordForms("Chrome Form Data (42)", expected);
 }
 
 TEST_F(NativeBackendKWalletTest, AddDuplicateLogin) {
   // Pretend that the migration has already taken place.
   profile_.GetPrefs()->SetBoolean(prefs::kPasswordsUseLocalProfileId, true);
 
   NativeBackendKWalletStub backend(42, profile_.GetPrefs());
   EXPECT_TRUE(backend.InitWithBus(mock_session_bus_));
 
   BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-      base::IgnoreReturn(base::Callback<bool(void)>(
-          base::Bind(&NativeBackendKWalletStub::AddLogin,
-                     base::Unretained(&backend), form_google_))));
+      base::IgnoreReturn<bool>(base::Bind(
+          &NativeBackendKWalletStub::AddLogin,
+          base::Unretained(&backend), form_google_)));
   BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-      base::IgnoreReturn(base::Callback<bool(void)>(
-          base::Bind(&NativeBackendKWalletStub::AddLogin,
-                     base::Unretained(&backend), form_google_))));
+      base::IgnoreReturn<bool>(base::Bind(
+          &NativeBackendKWalletStub::AddLogin,
+          base::Unretained(&backend), form_google_)));
 
   RunDBThread();
 
   EXPECT_FALSE(wallet_.hasFolder("Chrome Form Data"));
 
   std::vector<const PasswordForm*> forms;
   forms.push_back(&form_google_);
   ExpectationArray expected;
   expected.push_back(make_pair(std::string(form_google_.signon_realm), forms));
   CheckPasswordForms("Chrome Form Data (42)", expected);
 }
 
 TEST_F(NativeBackendKWalletTest, ListLoginsAppends) {
   // Pretend that the migration has already taken place.
   profile_.GetPrefs()->SetBoolean(prefs::kPasswordsUseLocalProfileId, true);
 
   NativeBackendKWalletStub backend(42, profile_.GetPrefs());
   EXPECT_TRUE(backend.InitWithBus(mock_session_bus_));
 
   BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-      base::IgnoreReturn(base::Callback<bool(void)>(
-          base::Bind(&NativeBackendKWalletStub::AddLogin,
-                     base::Unretained(&backend), form_google_))));
+      base::IgnoreReturn<bool>(base::Bind(
+          &NativeBackendKWalletStub::AddLogin,
+          base::Unretained(&backend), form_google_)));
 
   // Send the same request twice with the same list both times.
   std::vector<PasswordForm*> form_list;
   BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-      base::IgnoreReturn(base::Callback<bool(void)>(
-          base::Bind(&NativeBackendKWalletStub::GetAutofillableLogins,
-                     base::Unretained(&backend), &form_list))));
+      base::IgnoreReturn<bool>(base::Bind(
+          &NativeBackendKWalletStub::GetAutofillableLogins,
+          base::Unretained(&backend), &form_list)));
   BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-      base::IgnoreReturn(base::Callback<bool(void)>(
-          base::Bind(&NativeBackendKWalletStub::GetAutofillableLogins,
-                     base::Unretained(&backend), &form_list))));
+      base::IgnoreReturn<bool>(base::Bind(
+          &NativeBackendKWalletStub::GetAutofillableLogins,
+          base::Unretained(&backend), &form_list)));
 
   RunDBThread();
 
   // Quick check that we got two results back.
   EXPECT_EQ(2u, form_list.size());
   STLDeleteElements(&form_list);
 
   EXPECT_FALSE(wallet_.hasFolder("Chrome Form Data"));
 
   std::vector<const PasswordForm*> forms;
   forms.push_back(&form_google_);
   ExpectationArray expected;
   expected.push_back(make_pair(std::string(form_google_.signon_realm), forms));
   CheckPasswordForms("Chrome Form Data (42)", expected);
 }
 
 // TODO(mdm): add more basic (i.e. non-migration) tests here at some point.
 // (For example tests for storing >1 password per realm pickle.)
 
 TEST_F(NativeBackendKWalletTest, MigrateOneLogin) {
   // Reject attempts to migrate so we can populate the store.
   wallet_.set_reject_local_folders(true);
 
   {
     NativeBackendKWalletStub backend(42, profile_.GetPrefs());
     EXPECT_TRUE(backend.InitWithBus(mock_session_bus_));
 
     BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-        base::IgnoreReturn(base::Callback<bool(void)>(
-            base::Bind(&NativeBackendKWalletStub::AddLogin,
-                       base::Unretained(&backend), form_google_))));
+        base::IgnoreReturn<bool>(base::Bind(
+            &NativeBackendKWalletStub::AddLogin,
+            base::Unretained(&backend), form_google_)));
 
     // Make sure we can get the form back even when migration is failing.
     std::vector<PasswordForm*> form_list;
     BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-        base::IgnoreReturn(base::Callback<bool(void)>(
-            base::Bind(&NativeBackendKWalletStub::GetAutofillableLogins,
-                       base::Unretained(&backend), &form_list))));
+        base::IgnoreReturn<bool>(base::Bind(
+            &NativeBackendKWalletStub::GetAutofillableLogins,
+            base::Unretained(&backend), &form_list)));
 
     RunDBThread();
 
     // Quick check that we got something back.
     EXPECT_EQ(1u, form_list.size());
     STLDeleteElements(&form_list);
   }
 
   EXPECT_FALSE(wallet_.hasFolder("Chrome Form Data (42)"));
 
   std::vector<const PasswordForm*> forms;
   forms.push_back(&form_google_);
   ExpectationArray expected;
   expected.push_back(make_pair(std::string(form_google_.signon_realm), forms));
   CheckPasswordForms("Chrome Form Data", expected);
 
   // Now allow the migration.
   wallet_.set_reject_local_folders(false);
 
   {
     NativeBackendKWalletStub backend(42, profile_.GetPrefs());
     EXPECT_TRUE(backend.InitWithBus(mock_session_bus_));
 
     // Trigger the migration by looking something up.
     std::vector<PasswordForm*> form_list;
     BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-        base::IgnoreReturn(base::Callback<bool(void)>(
-            base::Bind(&NativeBackendKWalletStub::GetAutofillableLogins,
-                       base::Unretained(&backend), &form_list))));
+        base::IgnoreReturn<bool>(base::Bind(
+            &NativeBackendKWalletStub::GetAutofillableLogins,
+            base::Unretained(&backend), &form_list)));
 
     RunDBThread();
 
     // Quick check that we got something back.
     EXPECT_EQ(1u, form_list.size());
     STLDeleteElements(&form_list);
   }
 
   CheckPasswordForms("Chrome Form Data", expected);
   CheckPasswordForms("Chrome Form Data (42)", expected);
 
   // Check that we have set the persistent preference.
   EXPECT_TRUE(
       profile_.GetPrefs()->GetBoolean(prefs::kPasswordsUseLocalProfileId));
 }
 
 TEST_F(NativeBackendKWalletTest, MigrateToMultipleProfiles) {
   // Reject attempts to migrate so we can populate the store.
   wallet_.set_reject_local_folders(true);
 
   {
     NativeBackendKWalletStub backend(42, profile_.GetPrefs());
     EXPECT_TRUE(backend.InitWithBus(mock_session_bus_));
 
     BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-        base::IgnoreReturn(base::Callback<bool(void)>(
-            base::Bind(&NativeBackendKWalletStub::AddLogin,
-                       base::Unretained(&backend), form_google_))));
+        base::IgnoreReturn<bool>(base::Bind(
+            &NativeBackendKWalletStub::AddLogin,
+            base::Unretained(&backend), form_google_)));
 
     RunDBThread();
   }
 
   EXPECT_FALSE(wallet_.hasFolder("Chrome Form Data (42)"));
 
   std::vector<const PasswordForm*> forms;
   forms.push_back(&form_google_);
   ExpectationArray expected;
   expected.push_back(make_pair(std::string(form_google_.signon_realm), forms));
   CheckPasswordForms("Chrome Form Data", expected);
 
   // Now allow the migration.
   wallet_.set_reject_local_folders(false);
 
   {
     NativeBackendKWalletStub backend(42, profile_.GetPrefs());
     EXPECT_TRUE(backend.InitWithBus(mock_session_bus_));
 
     // Trigger the migration by looking something up.
     std::vector<PasswordForm*> form_list;
     BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-        base::IgnoreReturn(base::Callback<bool(void)>(
-            base::Bind(&NativeBackendKWalletStub::GetAutofillableLogins,
-                       base::Unretained(&backend), &form_list))));
+        base::IgnoreReturn<bool>(base::Bind(
+            &NativeBackendKWalletStub::GetAutofillableLogins,
+            base::Unretained(&backend), &form_list)));
 
     RunDBThread();
 
     // Quick check that we got something back.
     EXPECT_EQ(1u, form_list.size());
     STLDeleteElements(&form_list);
   }
 
   CheckPasswordForms("Chrome Form Data", expected);
   CheckPasswordForms("Chrome Form Data (42)", expected);
 
   // Check that we have set the persistent preference.
   EXPECT_TRUE(
       profile_.GetPrefs()->GetBoolean(prefs::kPasswordsUseLocalProfileId));
 
   // Normally we'd actually have a different profile. But in the test just reset
   // the profile's persistent pref; we pass in the local profile id anyway.
   profile_.GetPrefs()->SetBoolean(prefs::kPasswordsUseLocalProfileId, false);
 
   {
     NativeBackendKWalletStub backend(24, profile_.GetPrefs());
     EXPECT_TRUE(backend.InitWithBus(mock_session_bus_));
 
     // Trigger the migration by looking something up.
     std::vector<PasswordForm*> form_list;
     BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-        base::IgnoreReturn(base::Callback<bool(void)>(
-            base::Bind(&NativeBackendKWalletStub::GetAutofillableLogins,
-                       base::Unretained(&backend), &form_list))));
+        base::IgnoreReturn<bool>(base::Bind(
+            &NativeBackendKWalletStub::GetAutofillableLogins,
+            base::Unretained(&backend), &form_list)));
 
     RunDBThread();
 
     // Quick check that we got something back.
     EXPECT_EQ(1u, form_list.size());
     STLDeleteElements(&form_list);
   }
 
   CheckPasswordForms("Chrome Form Data", expected);
   CheckPasswordForms("Chrome Form Data (42)", expected);
   CheckPasswordForms("Chrome Form Data (24)", expected);
 }
 
 TEST_F(NativeBackendKWalletTest, NoMigrationWithPrefSet) {
   // Reject attempts to migrate so we can populate the store.
   wallet_.set_reject_local_folders(true);
 
   {
     NativeBackendKWalletStub backend(42, profile_.GetPrefs());
     EXPECT_TRUE(backend.InitWithBus(mock_session_bus_));
 
     BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-        base::IgnoreReturn(base::Callback<bool(void)>(
-            base::Bind(&NativeBackendKWalletStub::AddLogin,
-                       base::Unretained(&backend), form_google_))));
+        base::IgnoreReturn<bool>(base::Bind(
+            &NativeBackendKWalletStub::AddLogin,
+            base::Unretained(&backend), form_google_)));
 
     RunDBThread();
   }
 
   EXPECT_FALSE(wallet_.hasFolder("Chrome Form Data (42)"));
 
   std::vector<const PasswordForm*> forms;
   forms.push_back(&form_google_);
   ExpectationArray expected;
   expected.push_back(make_pair(std::string(form_google_.signon_realm), forms));
   CheckPasswordForms("Chrome Form Data", expected);
 
   // Now allow migration, but also pretend that the it has already taken place.
   wallet_.set_reject_local_folders(false);
   profile_.GetPrefs()->SetBoolean(prefs::kPasswordsUseLocalProfileId, true);
 
   {
     NativeBackendKWalletStub backend(42, profile_.GetPrefs());
     EXPECT_TRUE(backend.InitWithBus(mock_session_bus_));
 
     // Trigger the migration by adding a new login.
     BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-        base::IgnoreReturn(base::Callback<bool(void)>(
-            base::Bind(&NativeBackendKWalletStub::AddLogin,
-                       base::Unretained(&backend), form_isc_))));
+        base::IgnoreReturn<bool>(base::Bind(
+            &NativeBackendKWalletStub::AddLogin,
+            base::Unretained(&backend), form_isc_)));
 
     // Look up all logins; we expect only the one we added.
     std::vector<PasswordForm*> form_list;
     BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-        base::IgnoreReturn(base::Callback<bool(void)>(
-            base::Bind(&NativeBackendKWalletStub::GetAutofillableLogins,
-                       base::Unretained(&backend), &form_list))));
+        base::IgnoreReturn<bool>(base::Bind(
+            &NativeBackendKWalletStub::GetAutofillableLogins,
+            base::Unretained(&backend), &form_list)));
 
     RunDBThread();
 
     // Quick check that we got the right thing back.
     EXPECT_EQ(1u, form_list.size());
     if (form_list.size() > 0)
       EXPECT_EQ(form_isc_.signon_realm, form_list[0]->signon_realm);
     STLDeleteElements(&form_list);
   }
 
   CheckPasswordForms("Chrome Form Data", expected);
 
   forms[0] = &form_isc_;
   expected.clear();
   expected.push_back(make_pair(std::string(form_isc_.signon_realm), forms));
   CheckPasswordForms("Chrome Form Data (42)", expected);
 }
 
 TEST_F(NativeBackendKWalletTest, DeleteMigratedPasswordIsIsolated) {
   // Reject attempts to migrate so we can populate the store.
   wallet_.set_reject_local_folders(true);
 
   {
     NativeBackendKWalletStub backend(42, profile_.GetPrefs());
     EXPECT_TRUE(backend.InitWithBus(mock_session_bus_));
 
     BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-        base::IgnoreReturn(base::Callback<bool(void)>(
-            base::Bind(&NativeBackendKWalletStub::AddLogin,
-                       base::Unretained(&backend), form_google_))));
+        base::IgnoreReturn<bool>(base::Bind(
+            &NativeBackendKWalletStub::AddLogin,
+            base::Unretained(&backend), form_google_)));
 
     RunDBThread();
   }
 
   EXPECT_FALSE(wallet_.hasFolder("Chrome Form Data (42)"));
 
   std::vector<const PasswordForm*> forms;
   forms.push_back(&form_google_);
   ExpectationArray expected;
   expected.push_back(make_pair(std::string(form_google_.signon_realm), forms));
   CheckPasswordForms("Chrome Form Data", expected);
 
   // Now allow the migration.
   wallet_.set_reject_local_folders(false);
 
   {
     NativeBackendKWalletStub backend(42, profile_.GetPrefs());
     EXPECT_TRUE(backend.InitWithBus(mock_session_bus_));
 
     // Trigger the migration by looking something up.
     std::vector<PasswordForm*> form_list;
     BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-        base::IgnoreReturn(base::Callback<bool(void)>(
-            base::Bind(&NativeBackendKWalletStub::GetAutofillableLogins,
-                       base::Unretained(&backend), &form_list))));
+        base::IgnoreReturn<bool>(base::Bind(
+            &NativeBackendKWalletStub::GetAutofillableLogins,
+            base::Unretained(&backend), &form_list)));
 
     RunDBThread();
 
     // Quick check that we got something back.
     EXPECT_EQ(1u, form_list.size());
     STLDeleteElements(&form_list);
   }
 
   CheckPasswordForms("Chrome Form Data", expected);
   CheckPasswordForms("Chrome Form Data (42)", expected);
 
   // Check that we have set the persistent preference.
   EXPECT_TRUE(
       profile_.GetPrefs()->GetBoolean(prefs::kPasswordsUseLocalProfileId));
 
   // Normally we'd actually have a different profile. But in the test just reset
   // the profile's persistent pref; we pass in the local profile id anyway.
   profile_.GetPrefs()->SetBoolean(prefs::kPasswordsUseLocalProfileId, false);
 
   {
     NativeBackendKWalletStub backend(24, profile_.GetPrefs());
     EXPECT_TRUE(backend.InitWithBus(mock_session_bus_));
 
     // Trigger the migration by looking something up.
     std::vector<PasswordForm*> form_list;
     BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-        base::IgnoreReturn(base::Callback<bool(void)>(
-            base::Bind(&NativeBackendKWalletStub::GetAutofillableLogins,
-                       base::Unretained(&backend), &form_list))));
+        base::IgnoreReturn<bool>(base::Bind(
+            &NativeBackendKWalletStub::GetAutofillableLogins,
+            base::Unretained(&backend), &form_list)));
 
     RunDBThread();
 
     // Quick check that we got something back.
     EXPECT_EQ(1u, form_list.size());
     STLDeleteElements(&form_list);
 
     // There should be three passwords now.
     CheckPasswordForms("Chrome Form Data", expected);
     CheckPasswordForms("Chrome Form Data (42)", expected);
     CheckPasswordForms("Chrome Form Data (24)", expected);
 
     // Now delete the password from this second profile.
     BrowserThread::PostTask(BrowserThread::DB, FROM_HERE,
-        base::IgnoreReturn(base::Callback<bool(void)>(
-            base::Bind(&NativeBackendKWalletStub::RemoveLogin,
-                       base::Unretained(&backend), form_google_))));
+        base::IgnoreReturn<bool>(base::Bind(
+            &NativeBackendKWalletStub::RemoveLogin,
+            base::Unretained(&backend), form_google_)));
 
     RunDBThread();
 
     // The other two copies of the password in different profiles should remain.
     CheckPasswordForms("Chrome Form Data", expected);
     CheckPasswordForms("Chrome Form Data (42)", expected);
     expected.clear();
     CheckPasswordForms("Chrome Form Data (24)", expected);
   }
 }