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

Unified Diff: chrome/browser/ui/app_list/arc/arc_app_test.cc

Issue 2723073002: Extract kArcEnabled preference from ArcSessionManager part 2. (Closed)
Patch Set: address comments Created 3 years, 10 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/ui/app_list/arc/arc_app_test.cc
diff --git a/chrome/browser/ui/app_list/arc/arc_app_test.cc b/chrome/browser/ui/app_list/arc/arc_app_test.cc
index d2b90b3aefa6216f3903d879e10b12c5c7e2c19d..a529b2fb59fabe5b89ad35521622a74254b7f56b 100644
--- a/chrome/browser/ui/app_list/arc/arc_app_test.cc
+++ b/chrome/browser/ui/app_list/arc/arc_app_test.cc
@@ -8,6 +8,8 @@
#include "base/memory/ptr_util.h"
#include "base/run_loop.h"
#include "base/strings/stringprintf.h"
+#include "chrome/browser/chromeos/arc/arc_auth_notification.h"
+#include "chrome/browser/chromeos/arc/arc_play_store_enabled_preference_handler.h"
#include "chrome/browser/chromeos/arc/arc_session_manager.h"
#include "chrome/browser/chromeos/arc/arc_util.h"
#include "chrome/browser/chromeos/login/users/fake_chrome_user_manager.h"
@@ -86,8 +88,12 @@ void ArcAppTest::SetUp(Profile* profile) {
base::Bind(arc::FakeArcSession::Create)));
DCHECK(arc::ArcSessionManager::Get());
arc::ArcSessionManager::DisableUIForTesting();
+ arc::ArcAuthNotification::DisableForTesting();
arc_session_manager_->SetProfile(profile_);
- arc_session_manager_->StartPreferenceHandler();
+ arc_play_store_enabled_preference_handler_ =
+ base::MakeUnique<arc::ArcPlayStoreEnabledPreferenceHandler>(
+ profile_, arc_session_manager_.get());
+ arc_play_store_enabled_preference_handler_->Start();
arc_app_list_pref_ = ArcAppListPrefs::Get(profile_);
DCHECK(arc_app_list_pref_);
@@ -166,6 +172,7 @@ void ArcAppTest::CreateFakeAppsAndPackages() {
void ArcAppTest::TearDown() {
app_instance_.reset();
+ arc_play_store_enabled_preference_handler_.reset();
arc_session_manager_.reset();
arc_service_manager_.reset();
if (dbus_thread_manager_initialized_) {
« no previous file with comments | « chrome/browser/ui/app_list/arc/arc_app_test.h ('k') | chrome/browser/ui/ash/launcher/arc_app_launcher_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698