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

Unified Diff: chrome/browser/media_galleries/fileapi/picasa_finder.cc

Issue 23513059: Media Galleries API Picasa: End-to-end browsertest. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: self review 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/fileapi/picasa_finder.cc
diff --git a/chrome/browser/media_galleries/fileapi/picasa_finder.cc b/chrome/browser/media_galleries/fileapi/picasa_finder.cc
index 86b4e12599bded872266e7ee7ac6e550588fe6de..e6f4d9712058e78ba1314039abf4dbaebfe095ec 100644
--- a/chrome/browser/media_galleries/fileapi/picasa_finder.cc
+++ b/chrome/browser/media_galleries/fileapi/picasa_finder.cc
@@ -14,12 +14,7 @@
namespace picasa {
-namespace {
-
-// Returns path of Picasa's DB3 database directory. May only be called on
-// threads that allow for disk IO, like the FILE thread or MediaTaskRunner.
-base::FilePath FindPicasaDatabaseOnFileThread() {
- DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::FILE));
+base::FilePath PicasaFinder::GetPicasaDatabasePath() {
base::FilePath path;
#if defined(OS_WIN)
@@ -34,8 +29,17 @@ base::FilePath FindPicasaDatabaseOnFileThread() {
return base::FilePath();
#endif
- path = path.AppendASCII("Google").AppendASCII("Picasa2")
- .AppendASCII(kPicasaDatabaseDirName);
+ return path.AppendASCII("Google").AppendASCII("Picasa2").AppendASCII(
+ kPicasaDatabaseDirName);
+}
+
+namespace {
+
+// Returns path of Picasa's DB3 database directory. May only be called on
+// threads that allow for disk IO, like the FILE thread or MediaTaskRunner.
+base::FilePath FindPicasaDatabaseOnFileThread() {
+ DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::FILE));
+ base::FilePath path = PicasaFinder::GetPicasaDatabasePath();
// Verify actual existence
if (!base::DirectoryExists(path))

Powered by Google App Engine
This is Rietveld 408576698