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

Unified Diff: chrome/browser/extensions/api/media_galleries/media_galleries_apitest.cc

Issue 11027051: MediaFileSystemRegistry unit tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix compile Created 8 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/extensions/api/media_galleries/media_galleries_apitest.cc
diff --git a/chrome/browser/extensions/api/media_galleries/media_galleries_apitest.cc b/chrome/browser/extensions/api/media_galleries/media_galleries_apitest.cc
index 73f35b4c21fe51e90a7363b6a92e0c0137a3c1d7..d02c2a777a4305bb5ba4205f79a7347469cf07dc 100644
--- a/chrome/browser/extensions/api/media_galleries/media_galleries_apitest.cc
+++ b/chrome/browser/extensions/api/media_galleries/media_galleries_apitest.cc
@@ -12,6 +12,7 @@
#include "chrome/browser/extensions/extension_process_manager.h"
#include "chrome/browser/extensions/extension_system.h"
#include "chrome/browser/extensions/platform_app_browsertest_util.h"
+#include "chrome/browser/media_gallery/media_galleries_test_util.h"
#include "chrome/browser/ui/browser.h"
#include "chrome/common/chrome_paths.h"
#include "chrome/common/chrome_switches.h"
@@ -54,73 +55,6 @@ class PlatformAppMediaGalleriesBrowserTest : public PlatformAppBrowserTest {
}
};
-class EnsureMediaDirectoriesExists {
- public:
- EnsureMediaDirectoriesExists() : num_galleries_(0) {
- Init();
- }
-
- int num_galleries() const { return num_galleries_; }
-
- private:
- void Init() {
-#if defined(OS_CHROMEOS) || defined(OS_ANDROID)
- return;
-#elif defined(OS_LINUX)
- // On Linux, the media directories may not exist by default, so override
- // the settings to point to a temp directory with the media directories.
- ASSERT_TRUE(xdg_dir_.CreateUniqueTempDir());
- const FilePath xdg_path = xdg_dir_.path();
- const FilePath music_dir(xdg_path.Append("Megaman"));
- const FilePath pictures_dir(xdg_path.Append("Pitfall"));
- const FilePath videos_dir(xdg_path.Append("VVVV"));
- ASSERT_TRUE(file_util::CreateDirectory(music_dir));
- ASSERT_TRUE(file_util::CreateDirectory(pictures_dir));
- ASSERT_TRUE(file_util::CreateDirectory(videos_dir));
-
- const FilePath config_file(xdg_path.Append("user-dirs.dirs"));
- std::string xdg_user_dir_data = base::StringPrintf(
- "XDG_MUSIC_DIR=\"%s\"\n"
- "XDG_PICTURES_DIR=\"%s\"\n"
- "XDG_VIDEOS_DIR=\"%s\"\n",
- music_dir.value().c_str(),
- pictures_dir.value().c_str(),
- videos_dir.value().c_str());
- ASSERT_TRUE(file_util::WriteFile(config_file,
- xdg_user_dir_data.c_str(),
- xdg_user_dir_data.size()));
-
- scoped_ptr<base::Environment> env(base::Environment::Create());
- env->SetVar("XDG_CONFIG_HOME", xdg_path.value());
-
- num_galleries_ = 3;
-#else
- const int kDirectoryKeys[] = {
- chrome::DIR_USER_MUSIC,
- chrome::DIR_USER_PICTURES,
- chrome::DIR_USER_VIDEOS,
- };
-
- for (size_t i = 0; i < arraysize(kDirectoryKeys); ++i) {
- FilePath path;
- if (PathService::Get(kDirectoryKeys[i], &path) &&
- file_util::DirectoryExists(path)) {
- ++num_galleries_;
- }
- }
- ASSERT_GT(num_galleries_, 0);
-#endif
- }
-
-#if defined(OS_LINUX)
- ScopedTempDir xdg_dir_;
-#endif
-
- int num_galleries_;
-
- DISALLOW_COPY_AND_ASSIGN(EnsureMediaDirectoriesExists);
-};
-
} // namespace
IN_PROC_BROWSER_TEST_F(PlatformAppMediaGalleriesBrowserTest, NoGalleries) {

Powered by Google App Engine
This is Rietveld 408576698