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

Unified Diff: chrome/browser/media_galleries/media_galleries_test_util.cc

Issue 23513059: Media Galleries API Picasa: End-to-end browsertest. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 3 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/media_galleries/media_galleries_test_util.cc
diff --git a/chrome/browser/media_galleries/media_galleries_test_util.cc b/chrome/browser/media_galleries/media_galleries_test_util.cc
index d07e80c59971fb68af280b5bca47f635ccccd8ea..af538a72ed4fa7943d670911d75e363609ac3fec 100644
--- a/chrome/browser/media_galleries/media_galleries_test_util.cc
+++ b/chrome/browser/media_galleries/media_galleries_test_util.cc
@@ -75,6 +75,26 @@ EnsureMediaDirectoriesExists::EnsureMediaDirectoriesExists()
EnsureMediaDirectoriesExists::~EnsureMediaDirectoriesExists() {
}
+base::FilePath EnsureMediaDirectoriesExists::GetFakeAppDataPath() const {
+ DCHECK(fake_dir_.IsValid());
+ return fake_dir_.path().AppendASCII("appdata");
+}
+
+#if defined(OS_WIN)
+base::FilePath EnsureMediaDirectoriesExists::GetFakeLocalAppDataPath() const {
+ DCHECK(fake_dir_.IsValid());
+ return fake_dir_.path().AppendASCII("localappdata");
+}
+#endif
+
+#if defined(OS_WIN) || defined(OS_MACOSX)
+base::FilePath
+EnsureMediaDirectoriesExists::GetFakePicasaFoldersRootPath() const {
+ DCHECK(fake_dir_.IsValid());
+ return fake_dir_.path().AppendASCII("picasa_folders");
+}
+#endif
+
void EnsureMediaDirectoriesExists::Init() {
#if defined(OS_CHROMEOS) || defined(OS_ANDROID)
return;
@@ -83,18 +103,23 @@ void EnsureMediaDirectoriesExists::Init() {
ASSERT_TRUE(fake_dir_.CreateUniqueTempDir());
#if defined(OS_WIN) || defined(OS_MACOSX)
- // This is to make sure the tests don't think iTunes is installed (unless
- // we control it specifically).
- appdir_override_.reset(new base::ScopedPathOverride(
- base::DIR_APP_DATA, fake_dir_.path().AppendASCII("itunes")));
+ // This is to control whether or not tests think iTunes and Picasa are
+ // installed.
+ app_data_override_.reset(new base::ScopedPathOverride(
+ base::DIR_APP_DATA, GetFakeAppDataPath()));
+#if defined(OS_WIN)
+ // Picasa on Windows is in the DIR_LOCAL_APP_DATA directory.
+ local_app_data_override_.reset(new base::ScopedPathOverride(
+ base::DIR_LOCAL_APP_DATA, GetFakeLocalAppDataPath()));
+#endif
#endif
music_override_.reset(new base::ScopedPathOverride(
- chrome::DIR_USER_MUSIC, fake_dir_.path().AppendASCII("music")));
+ chrome::DIR_USER_MUSIC, fake_dir_.path().AppendASCII("music")));
pictures_override_.reset(new base::ScopedPathOverride(
- chrome::DIR_USER_PICTURES, fake_dir_.path().AppendASCII("pictures")));
+ chrome::DIR_USER_PICTURES, fake_dir_.path().AppendASCII("pictures")));
video_override_.reset(new base::ScopedPathOverride(
- chrome::DIR_USER_VIDEOS, fake_dir_.path().AppendASCII("videos")));
+ chrome::DIR_USER_VIDEOS, fake_dir_.path().AppendASCII("videos")));
num_galleries_ = 3;
#endif
}

Powered by Google App Engine
This is Rietveld 408576698