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

Unified Diff: chrome/browser/browsing_data/browsing_data_remover_unittest.cc

Issue 49773003: ChromeOS: Remove MockDBusThreadManager. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed tests. Created 7 years, 1 month 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: chrome/browser/browsing_data/browsing_data_remover_unittest.cc
diff --git a/chrome/browser/browsing_data/browsing_data_remover_unittest.cc b/chrome/browser/browsing_data/browsing_data_remover_unittest.cc
index d6313d2de20423c7910f453f73af24fcb07c82e2..908cc8c4388fe1ef7a81ba40b03e4f9aa7c8aecc 100644
--- a/chrome/browser/browsing_data/browsing_data_remover_unittest.cc
+++ b/chrome/browser/browsing_data/browsing_data_remover_unittest.cc
@@ -34,8 +34,8 @@
#include "chrome/test/base/testing_profile.h"
#if defined(OS_CHROMEOS)
#include "chromeos/dbus/dbus_thread_manager.h"
+#include "chromeos/dbus/fake_dbus_thread_manager.h"
#include "chromeos/dbus/mock_cryptohome_client.h"
-#include "chromeos/dbus/mock_dbus_thread_manager.h"
#endif
#include "components/autofill/core/browser/autofill_common_test.h"
#include "components/autofill/core/browser/autofill_profile.h"
@@ -1573,10 +1573,14 @@ TEST_F(BrowsingDataRemoverTest, ContentProtectionPlatformKeysRemoval) {
new testing::NiceMock<chromeos::MockUserManager>();
mock_user_manager->SetActiveUser("test@example.com");
chromeos::ScopedUserManagerEnabler user_manager_enabler(mock_user_manager);
- chromeos::MockDBusThreadManager mock_dbus_manager;
- chromeos::DBusThreadManager::InitializeForTesting(&mock_dbus_manager);
+
+ chromeos::FakeDBusThreadManager* fake_dbus_manager =
+ new chromeos::FakeDBusThreadManager;
chromeos::MockCryptohomeClient* cryptohome_client =
- mock_dbus_manager.mock_cryptohome_client();
+ new chromeos::MockCryptohomeClient;
+ fake_dbus_manager->SetCryptohomeClient(
+ scoped_ptr<chromeos::CryptohomeClient>(cryptohome_client));
+ chromeos::DBusThreadManager::InitializeForTesting(fake_dbus_manager);
// Expect exactly one call. No calls means no attempt to delete keys and more
// than one call means a significant performance problem.
« no previous file with comments | « chrome/browser/apps/app_browsertest.cc ('k') | chrome/browser/chromeos/input_method/input_method_manager_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698