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

Unified Diff: chrome/browser/chromeos/arc/arc_session_manager_unittest.cc

Issue 2574013003: Refactor ArcSessionRunner part 1. (Closed)
Patch Set: Address comments Created 4 years 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/chromeos/arc/arc_session_manager_unittest.cc
diff --git a/chrome/browser/chromeos/arc/arc_session_manager_unittest.cc b/chrome/browser/chromeos/arc/arc_session_manager_unittest.cc
index 32797a755e01bf460bbd9d9b9d71b02f98550750..d49c8936f0a8eba131998a76cb93d82c64b524a3 100644
--- a/chrome/browser/chromeos/arc/arc_session_manager_unittest.cc
+++ b/chrome/browser/chromeos/arc/arc_session_manager_unittest.cc
@@ -31,7 +31,8 @@
#include "chromeos/dbus/fake_session_manager_client.h"
#include "components/arc/arc_bridge_service.h"
#include "components/arc/arc_service_manager.h"
-#include "components/arc/test/fake_arc_bridge_service.h"
+#include "components/arc/arc_session_runner.h"
+#include "components/arc/test/fake_arc_session.h"
#include "components/prefs/pref_service.h"
#include "components/signin/core/account_id/account_id.h"
#include "components/sync/model/fake_sync_change_processor.h"
@@ -73,8 +74,8 @@ class ArcSessionManagerTestBase : public testing::Test {
profile_ = profile_builder.Build();
StartPreferenceSyncing();
- ArcServiceManager::SetArcBridgeServiceForTesting(
- base::MakeUnique<FakeArcBridgeService>());
+ ArcServiceManager::SetArcSessionRunnerForTesting(
+ base::MakeUnique<ArcSessionRunner>(base::Bind(FakeArcSession::Create)));
arc_service_manager_ = base::MakeUnique<ArcServiceManager>(nullptr);
arc_session_manager_ = base::MakeUnique<ArcSessionManager>(
arc_service_manager_->arc_bridge_service());
@@ -99,9 +100,8 @@ class ArcSessionManagerTestBase : public testing::Test {
protected:
Profile* profile() { return profile_.get(); }
- FakeArcBridgeService* bridge_service() {
- return static_cast<FakeArcBridgeService*>(
- arc_service_manager_->arc_bridge_service());
+ ArcBridgeService* bridge_service() {
+ return arc_service_manager_->arc_bridge_service();
}
ArcSessionManager* arc_session_manager() {
return arc_session_manager_.get();
« no previous file with comments | « chrome/browser/chromeos/arc/arc_session_manager_browsertest.cc ('k') | chrome/browser/policy/policy_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698