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

Unified Diff: chrome/browser/ui/app_list/extension_app_model_builder_unittest.cc

Issue 282103003: Moved IS_EPHEMERAL flag to extension prefs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix up file header Created 6 years, 7 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/extension_app_model_builder_unittest.cc
diff --git a/chrome/browser/ui/app_list/extension_app_model_builder_unittest.cc b/chrome/browser/ui/app_list/extension_app_model_builder_unittest.cc
index 3b1085f48a36e64c434ed11dba4033516c3b6483..7746eb233629e58e969db1ad735f3d86ae135a0c 100644
--- a/chrome/browser/ui/app_list/extension_app_model_builder_unittest.cc
+++ b/chrome/browser/ui/app_list/extension_app_model_builder_unittest.cc
@@ -12,10 +12,10 @@
#include "base/run_loop.h"
#include "base/values.h"
#include "chrome/browser/extensions/extension_function_test_utils.h"
-#include "chrome/browser/extensions/extension_service_unittest.h"
#include "chrome/browser/extensions/install_tracker.h"
#include "chrome/browser/extensions/install_tracker_factory.h"
#include "chrome/browser/ui/app_list/app_list_controller_delegate_impl.h"
+#include "chrome/browser/ui/app_list/app_list_test_util.h"
#include "chrome/common/chrome_constants.h"
#include "chrome/common/extensions/extension_constants.h"
#include "chrome/common/pref_names.h"
@@ -28,10 +28,6 @@
namespace {
-const char kHostedAppId[] = "dceacbkfkmllgmjmbhgkpjegnodmildf";
-const char kPackagedApp1Id[] = "emfkafnhnpcmabnnkckkchdilgeoekbo";
-const char kPackagedApp2Id[] = "jlklkagmeajbjiobondfhiekepofmljl";
-
// Get a string of all apps in |model| joined with ','.
std::string GetModelContent(app_list::AppListModel* model) {
std::string content;
@@ -103,29 +99,13 @@ const size_t kDefaultAppCount = 3u;
} // namespace
-class ExtensionAppModelBuilderTest : public ExtensionServiceTestBase {
+class ExtensionAppModelBuilderTest : public AppListTestBase {
public:
ExtensionAppModelBuilderTest() {}
virtual ~ExtensionAppModelBuilderTest() {}
virtual void SetUp() OVERRIDE {
- ExtensionServiceTestBase::SetUp();
-
- // Load "app_list" extensions test profile.
- // The test profile has 4 extensions:
- // 1 dummy extension, 2 packaged extension apps and 1 hosted extension app.
- base::FilePath source_install_dir = data_dir_
- .AppendASCII("app_list")
- .AppendASCII("Extensions");
- base::FilePath pref_path = source_install_dir
- .DirName()
- .Append(chrome::kPreferencesFilename);
- InitializeInstalledExtensionService(pref_path, source_install_dir);
- service_->Init();
-
- // There should be 4 extensions in the test profile.
- const extensions::ExtensionSet* extensions = service_->extensions();
- ASSERT_EQ(static_cast<size_t>(4), extensions->size());
+ AppListTestBase::SetUp();
CreateBuilder();
}
« no previous file with comments | « chrome/browser/ui/app_list/extension_app_model_builder.cc ('k') | chrome/browser/ui/app_list/recommended_apps.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698