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

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

Issue 2734873002: Fix ArcSessionManager state machine, part 2. (Closed)
Patch Set: Address comments. Created 3 years, 9 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: 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 9775813443e0caa26c818f372be943d3a303f9e4..2a401ff5aee17160e23d436d19b6384a5a4761c4 100644
--- a/chrome/browser/chromeos/arc/arc_session_manager_unittest.cc
+++ b/chrome/browser/chromeos/arc/arc_session_manager_unittest.cc
@@ -365,7 +365,7 @@ TEST_F(ArcSessionManagerTest, RemoveDataDir) {
// preference is false for managed user, i.e., data dir is being removed at
// beginning.
arc_session_manager()->SetProfile(profile());
- arc_session_manager()->RemoveArcData();
+ arc_session_manager()->RequestArcDataRemoval();
EXPECT_TRUE(
profile()->GetPrefs()->GetBoolean(prefs::kArcDataRemoveRequested));
EXPECT_EQ(ArcSessionManager::State::REMOVING_DATA_DIR,
@@ -388,7 +388,7 @@ TEST_F(ArcSessionManagerTest, RemoveDataDir) {
EXPECT_EQ(ArcSessionManager::State::ACTIVE, arc_session_manager()->state());
// Request to remove data and stop session manager.
- arc_session_manager()->RemoveArcData();
+ arc_session_manager()->RequestArcDataRemoval();
ASSERT_TRUE(
profile()->GetPrefs()->GetBoolean(prefs::kArcDataRemoveRequested));
EXPECT_EQ(ArcSessionManager::State::ACTIVE, arc_session_manager()->state());

Powered by Google App Engine
This is Rietveld 408576698