Media Galleries API Picasa: End-to-end browsertest.

chrome/browser/media_galleries/fileapi/picasa_finder.cc

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..70a7db4b9da6e4cf99f4a1d95821b64599796f76 100644
--- a/chrome/browser/media_galleries/fileapi/picasa_finder.cc
+++ b/chrome/browser/media_galleries/fileapi/picasa_finder.cc
@@ -7,6 +7,7 @@
 #include "base/base_paths.h"
 #include "base/bind.h"
 #include "base/file_util.h"
+#include "base/files/file_path.h"
 #include "base/path_service.h"
 #include "chrome/browser/storage_monitor/storage_info.h"
 #include "chrome/common/media_galleries/picasa_types.h"
@@ -34,8 +35,8 @@ base::FilePath FindPicasaDatabaseOnFileThread() {
   return base::FilePath();
 #endif
 
-  path = path.AppendASCII("Google").AppendASCII("Picasa2")
-             .AppendASCII(kPicasaDatabaseDirName);
+  return path.AppendASCII("Google").AppendASCII("Picasa2").AppendASCII(

[vandebo (ex-Chrome), 2013/09/18 15:49:35]

s/return/path =/

[tommycli, 2013/09/19 22:58:45]

Done.

+      kPicasaDatabaseDirName);
 
   // Verify actual existence
   if (!base::DirectoryExists(path))
@@ -53,8 +54,9 @@ void FinishOnOriginalThread(const PicasaFinder::DeviceIDCallback& callback,
 
 }  // namespace
 
-void PicasaFinder::FindPicasaDatabase(
-    const PicasaFinder::DeviceIDCallback& callback) {
+namespace PicasaFinder {

[vandebo (ex-Chrome), 2013/09/18 15:49:35]

remove

[tommycli, 2013/09/19 22:58:45]

Done.

+
+void FindPicasaDatabase(const PicasaFinder::DeviceIDCallback& callback) {
   content::BrowserThread::PostTaskAndReplyWithResult(
       content::BrowserThread::FILE,
       FROM_HERE,
@@ -62,4 +64,6 @@ void PicasaFinder::FindPicasaDatabase(
       base::Bind(&FinishOnOriginalThread, callback));
 }
 
+}  // namespace PicasaFinder
+
 }  // namespace picasa