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

Unified Diff: chrome/browser/ui/ash/launcher/arc_app_launcher_browsertest.cc

Issue 2507073002: Split ArcSessionManager from ArcAuthService. (Closed)
Patch Set: Split ArcSessionManager from ArcAuthService Created 4 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/ui/ash/launcher/arc_app_launcher_browsertest.cc
diff --git a/chrome/browser/ui/ash/launcher/arc_app_launcher_browsertest.cc b/chrome/browser/ui/ash/launcher/arc_app_launcher_browsertest.cc
index 5cf8f9edb4af5e31fd1d77dbb049e4b5040e92df..13d10d2c995e27d06664f482b3860d341417973e 100644
--- a/chrome/browser/ui/ash/launcher/arc_app_launcher_browsertest.cc
+++ b/chrome/browser/ui/ash/launcher/arc_app_launcher_browsertest.cc
@@ -136,10 +136,12 @@ class ArcAppLauncherBrowserTest : public ExtensionBrowserTest {
void SetUpInProcessBrowserTestFixture() override {
ExtensionBrowserTest::SetUpInProcessBrowserTestFixture();
- arc::ArcAuthService::DisableUIForTesting();
+ arc::ArcSessionManager::DisableUIForTesting();
}
- void SetUpOnMainThread() override { arc::ArcAuthService::Get()->EnableArc(); }
+ void SetUpOnMainThread() override {
+ arc::ArcSessionManager::Get()->EnableArc();
+ }
void InstallTestApps(bool multi_app) {
app_host()->OnAppListRefreshed(GetTestAppsList(multi_app));
@@ -177,13 +179,13 @@ class ArcAppLauncherBrowserTest : public ExtensionBrowserTest {
void SendPackageRemoved() { app_host()->OnPackageRemoved(kTestAppPackage); }
void StartInstance() {
- if (auth_service()->profile() != profile())
- auth_service()->OnPrimaryUserProfilePrepared(profile());
+ if (arc_session_manager()->profile() != profile())
+ arc_session_manager()->OnPrimaryUserProfilePrepared(profile());
app_instance_observer()->OnInstanceReady();
}
void StopInstance() {
- auth_service()->Shutdown();
+ arc_session_manager()->Shutdown();
app_instance_observer()->OnInstanceClosed();
}
@@ -200,7 +202,9 @@ class ArcAppLauncherBrowserTest : public ExtensionBrowserTest {
return app_prefs();
}
- arc::ArcAuthService* auth_service() { return arc::ArcAuthService::Get(); }
+ arc::ArcSessionManager* arc_session_manager() {
+ return arc::ArcSessionManager::Get();
+ }
private:
DISALLOW_COPY_AND_ASSIGN(ArcAppLauncherBrowserTest);

Powered by Google App Engine
This is Rietveld 408576698