[go: nahoru, domu]

blob: 753f9df0e292d6add8963e829de629f9139e9bed [file] [log] [blame]
// Copyright 2013 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
#include "chrome/browser/ui/app_list/app_list_service.h"
#include <memory>
#include <utility>
#include "base/command_line.h"
#include "base/files/file_path.h"
#include "base/macros.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/profiles/profiles_state.h"
#include "chrome/browser/ui/app_list/app_list_service_impl.h"
#include "chrome/browser/ui/app_list/test/fake_profile.h"
#include "chrome/browser/ui/app_list/test/fake_profile_store.h"
#include "chrome/common/chrome_constants.h"
#include "chrome/common/pref_names.h"
#include "components/prefs/pref_registry_simple.h"
#include "components/prefs/pref_service.h"
#include "components/prefs/pref_service_factory.h"
#include "components/prefs/testing_pref_store.h"
#include "testing/gtest/include/gtest/gtest.h"
#include "ui/app_list/app_list_switches.h"
class TestingAppListServiceImpl : public AppListServiceImpl {
public:
TestingAppListServiceImpl(const base::CommandLine& command_line,
PrefService* local_state,
std::unique_ptr<ProfileStore> profile_store)
: AppListServiceImpl(command_line, local_state, std::move(profile_store)),
showing_for_profile_(NULL),
destroy_app_list_call_count_(0) {}
Profile* showing_for_profile() const {
return showing_for_profile_;
}
int destroy_app_list_call_count() const {
return destroy_app_list_call_count_;
}
void PerformStartupChecks(Profile* profile) {
AppListServiceImpl::PerformStartupChecks(profile);
}
// AppListService overrides:
Profile* GetCurrentAppListProfile() override {
// We don't return showing_for_profile_ here because that is only defined if
// the app list is visible.
return NULL;
}
void CreateForProfile(Profile* requested_profile) override {}
void ShowForProfile(Profile* requested_profile) override {
showing_for_profile_ = requested_profile;
RecordAppListLaunch();
}
void ShowForCustomLauncherPage(Profile* profile) override {}
void HideCustomLauncherPage() override {}
void DismissAppList() override { showing_for_profile_ = NULL; }
bool IsAppListVisible() const override { return !!showing_for_profile_; }
gfx::NativeWindow GetAppListWindow() override { return NULL; }
AppListControllerDelegate* GetControllerDelegate() override { return NULL; }
// AppListServiceImpl overrides:
void DestroyAppList() override { ++destroy_app_list_call_count_; }
private:
Profile* showing_for_profile_;
int destroy_app_list_call_count_;
DISALLOW_COPY_AND_ASSIGN(TestingAppListServiceImpl);
};
class AppListServiceUnitTest : public testing::Test {
public:
AppListServiceUnitTest() {}
void SetUp() override {
SetupWithCommandLine(base::CommandLine(base::CommandLine::NO_PROGRAM));
}
protected:
void SetupWithCommandLine(const base::CommandLine& command_line) {
user_data_dir_ = base::FilePath(FILE_PATH_LITERAL("udd"));
profile1_.reset(
new FakeProfile("p1", user_data_dir_.AppendASCII("profile1")));
profile2_.reset(
new FakeProfile("p2", user_data_dir_.AppendASCII("profile2")));
PrefRegistrySimple* pref_registry = new PrefRegistrySimple;
AppListService::RegisterPrefs(pref_registry);
profiles::RegisterPrefs(pref_registry);
PrefServiceFactory factory;
factory.set_user_prefs(base::MakeRefCounted<TestingPrefStore>());
local_state_ = factory.Create(pref_registry);
profile_store_ = new FakeProfileStore(user_data_dir_, local_state_.get());
service_.reset(new TestingAppListServiceImpl(
command_line, local_state_.get(),
std::unique_ptr<ProfileStore>(profile_store_)));
}
void EnableAppList() {
service_->EnableAppList(profile1_.get(),
AppListService::ENABLE_VIA_COMMAND_LINE);
}
base::FilePath user_data_dir_;
std::unique_ptr<PrefService> local_state_;
FakeProfileStore* profile_store_;
std::unique_ptr<TestingAppListServiceImpl> service_;
std::unique_ptr<FakeProfile> profile1_;
std::unique_ptr<FakeProfile> profile2_;
DISALLOW_COPY_AND_ASSIGN(AppListServiceUnitTest);
};
TEST_F(AppListServiceUnitTest, EnablingStateIsPersisted) {
EXPECT_FALSE(local_state_->GetBoolean(prefs::kAppLauncherHasBeenEnabled));
EnableAppList();
EXPECT_TRUE(local_state_->GetBoolean(prefs::kAppLauncherHasBeenEnabled));
EXPECT_EQ(profile1_->GetPath(), user_data_dir_.Append(
local_state_->GetFilePath(prefs::kAppListProfile)));
}
TEST_F(AppListServiceUnitTest, ShowingForProfileLoadsAProfile) {
profile_store_->LoadProfile(profile1_.get());
EnableAppList();
service_->Show();
EXPECT_EQ(profile1_.get(), service_->showing_for_profile());
EXPECT_TRUE(service_->IsAppListVisible());
}
TEST_F(AppListServiceUnitTest, RemovedProfileResetsToInitialProfile) {
EnableAppList();
profile_store_->RemoveProfile(profile1_.get());
// kAppListProfile should have been cleared, and therefore GetProfilePath
// should return the initial profile.
EXPECT_EQ("", local_state_->GetString(prefs::kAppListProfile));
base::FilePath initial_profile_path =
user_data_dir_.AppendASCII(chrome::kInitialProfile);
EXPECT_EQ(initial_profile_path,
service_->GetProfilePath(profile_store_->GetUserDataDir()));
}
TEST_F(AppListServiceUnitTest,
RemovedProfileResetsToLastUsedProfileIfExists) {
local_state_->SetString(prefs::kProfileLastUsed, "last-used");
EnableAppList();
profile_store_->RemoveProfile(profile1_.get());
// kAppListProfile should have been set to kProfileLastUsed.
EXPECT_EQ("last-used", local_state_->GetString(prefs::kAppListProfile));
base::FilePath last_used_profile_path =
user_data_dir_.AppendASCII("last-used");
EXPECT_EQ(last_used_profile_path,
service_->GetProfilePath(profile_store_->GetUserDataDir()));
// For this test, the AppListViewDelegate is not created because the
// app list is never shown, so there is nothing to destroy.
EXPECT_EQ(0, service_->destroy_app_list_call_count());
}
TEST_F(AppListServiceUnitTest, RefusesToLoadGuestAppListProfile) {
// Unlikely, but if somehow the user's app_list.profile pref was set to the
// guest profile, make sure we refuse to load it (or it would crash).
local_state_->SetString(
prefs::kAppListProfile,
base::FilePath(chrome::kGuestProfileDir).MaybeAsASCII());
local_state_->SetString(prefs::kProfileLastUsed, "last-used");
base::FilePath last_used_profile_path =
user_data_dir_.AppendASCII("last-used");
EXPECT_EQ(last_used_profile_path,
service_->GetProfilePath(profile_store_->GetUserDataDir()));
}
TEST_F(AppListServiceUnitTest, RefusesToLoadGuestLastUsedProfile) {
// If the user's most recent browser session was a guest session, make sure we
// do not open a guest profile in the launcher (which would crash).
local_state_->SetString(
prefs::kProfileLastUsed,
base::FilePath(chrome::kGuestProfileDir).MaybeAsASCII());
base::FilePath initial_profile_path =
user_data_dir_.AppendASCII(chrome::kInitialProfile);
EXPECT_EQ(initial_profile_path,
service_->GetProfilePath(profile_store_->GetUserDataDir()));
}
TEST_F(AppListServiceUnitTest, SwitchingProfilesPersists) {
profile_store_->LoadProfile(profile1_.get());
profile_store_->LoadProfile(profile2_.get());
EnableAppList();
service_->SetProfilePath(profile2_->GetPath());
service_->Show();
EXPECT_EQ(profile2_.get(), service_->showing_for_profile());
EXPECT_EQ(profile2_->GetPath(),
service_->GetProfilePath(profile_store_->GetUserDataDir()));
service_->SetProfilePath(profile1_->GetPath());
EXPECT_EQ(profile1_->GetPath(),
service_->GetProfilePath(profile_store_->GetUserDataDir()));
}
TEST_F(AppListServiceUnitTest, EnableViaCommandLineFlag) {
base::CommandLine command_line(base::CommandLine::NO_PROGRAM);
command_line.AppendSwitch(app_list::switches::kEnableAppList);
SetupWithCommandLine(command_line);
service_->PerformStartupChecks(profile1_.get());
EXPECT_TRUE(local_state_->GetBoolean(prefs::kAppLauncherHasBeenEnabled));
}
TEST_F(AppListServiceUnitTest, DisableViaCommandLineFlag) {
base::CommandLine command_line(base::CommandLine::NO_PROGRAM);
command_line.AppendSwitch(app_list::switches::kResetAppListInstallState);
SetupWithCommandLine(command_line);
service_->PerformStartupChecks(profile1_.get());
EXPECT_FALSE(local_state_->GetBoolean(prefs::kAppLauncherHasBeenEnabled));
}
TEST_F(AppListServiceUnitTest, UMAPrefStates) {
EXPECT_FALSE(local_state_->GetBoolean(prefs::kAppLauncherHasBeenEnabled));
EXPECT_EQ(AppListService::ENABLE_NOT_RECORDED,
local_state_->GetInteger(prefs::kAppListEnableMethod));
EXPECT_EQ(0, local_state_->GetInt64(prefs::kAppListEnableTime));
service_->EnableAppList(profile1_.get(),
AppListService::ENABLE_FOR_APP_INSTALL);
// After enable, method and time should be recorded.
EXPECT_TRUE(local_state_->GetBoolean(prefs::kAppLauncherHasBeenEnabled));
EXPECT_EQ(AppListService::ENABLE_FOR_APP_INSTALL,
local_state_->GetInteger(prefs::kAppListEnableMethod));
EXPECT_NE(0, local_state_->GetInt64(prefs::kAppListEnableTime));
service_->ShowForProfile(profile1_.get());
// After a regular "show", time should be cleared, so UMA is not re-recorded.
EXPECT_EQ(AppListService::ENABLE_FOR_APP_INSTALL,
local_state_->GetInteger(prefs::kAppListEnableMethod));
EXPECT_EQ(0, local_state_->GetInt64(prefs::kAppListEnableTime));
// A second enable should be a no-op.
service_->EnableAppList(profile1_.get(),
AppListService::ENABLE_FOR_APP_INSTALL);
EXPECT_EQ(AppListService::ENABLE_FOR_APP_INSTALL,
local_state_->GetInteger(prefs::kAppListEnableMethod));
EXPECT_EQ(0, local_state_->GetInt64(prefs::kAppListEnableTime));
// An app install auto-show here should keep the recorded enable method.
service_->ShowForAppInstall(profile1_.get(), "", false);
EXPECT_EQ(AppListService::ENABLE_FOR_APP_INSTALL,
local_state_->GetInteger(prefs::kAppListEnableMethod));
// Clear the enable state, so we can enable again.
local_state_->SetBoolean(prefs::kAppLauncherHasBeenEnabled, false);
service_->EnableAppList(profile1_.get(),
AppListService::ENABLE_FOR_APP_INSTALL);
EXPECT_EQ(AppListService::ENABLE_FOR_APP_INSTALL,
local_state_->GetInteger(prefs::kAppListEnableMethod));
EXPECT_NE(0, local_state_->GetInt64(prefs::kAppListEnableTime));
// An auto-show here should update the enable method to prevent recording it
// as ENABLE_FOR_APP_INSTALL.
service_->ShowForAppInstall(profile1_.get(), "", false);
EXPECT_EQ(AppListService::ENABLE_SHOWN_UNDISCOVERED,
local_state_->GetInteger(prefs::kAppListEnableMethod));
}