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

Side by Side 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/media_galleries/fileapi/picasa_finder.h" 5 #include "chrome/browser/media_galleries/fileapi/picasa_finder.h"
6 6
7 #include "base/base_paths.h" 7 #include "base/base_paths.h"
8 #include "base/bind.h" 8 #include "base/bind.h"
9 #include "base/file_util.h" 9 #include "base/file_util.h"
10 #include "base/path_service.h" 10 #include "base/path_service.h"
11 #include "chrome/browser/storage_monitor/storage_info.h" 11 #include "chrome/browser/storage_monitor/storage_info.h"
12 #include "chrome/common/media_galleries/picasa_types.h" 12 #include "chrome/common/media_galleries/picasa_types.h"
13 #include "content/public/browser/browser_thread.h" 13 #include "content/public/browser/browser_thread.h"
14 14
15 namespace picasa { 15 namespace picasa {
16 16
17 namespace { 17 base::FilePath PicasaFinder::GetPicasaDatabasePath() {
18
19 // Returns path of Picasa's DB3 database directory. May only be called on
20 // threads that allow for disk IO, like the FILE thread or MediaTaskRunner.
21 base::FilePath FindPicasaDatabaseOnFileThread() {
22 DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::FILE));
23 base::FilePath path; 18 base::FilePath path;
24 19
25 #if defined(OS_WIN) 20 #if defined(OS_WIN)
26 // TODO(tommycli): Check registry for alternative path. 21 // TODO(tommycli): Check registry for alternative path.
27 if (!PathService::Get(base::DIR_LOCAL_APP_DATA, &path)) 22 if (!PathService::Get(base::DIR_LOCAL_APP_DATA, &path))
28 return base::FilePath(); 23 return base::FilePath();
29 #elif defined(OS_MACOSX) 24 #elif defined(OS_MACOSX)
30 // TODO(tommycli): Check Mac Preferences for alternative path. 25 // TODO(tommycli): Check Mac Preferences for alternative path.
31 if (!PathService::Get(base::DIR_APP_DATA, &path)) 26 if (!PathService::Get(base::DIR_APP_DATA, &path))
32 return base::FilePath(); 27 return base::FilePath();
33 #else 28 #else
34 return base::FilePath(); 29 return base::FilePath();
35 #endif 30 #endif
36 31
37 path = path.AppendASCII("Google").AppendASCII("Picasa2") 32 return path.AppendASCII("Google").AppendASCII("Picasa2").AppendASCII(
38 .AppendASCII(kPicasaDatabaseDirName); 33 kPicasaDatabaseDirName);
34 }
35
36 namespace {
37
38 // Returns path of Picasa's DB3 database directory. May only be called on
39 // threads that allow for disk IO, like the FILE thread or MediaTaskRunner.
40 base::FilePath FindPicasaDatabaseOnFileThread() {
41 DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::FILE));
42 base::FilePath path = PicasaFinder::GetPicasaDatabasePath();
39 43
40 // Verify actual existence 44 // Verify actual existence
41 if (!base::DirectoryExists(path)) 45 if (!base::DirectoryExists(path))
42 path.clear(); 46 path.clear();
43 47
44 return path; 48 return path;
45 } 49 }
46 50
47 void FinishOnOriginalThread(const PicasaFinder::DeviceIDCallback& callback, 51 void FinishOnOriginalThread(const PicasaFinder::DeviceIDCallback& callback,
48 const base::FilePath& database_path) { 52 const base::FilePath& database_path) {
49 if (!database_path.empty()) 53 if (!database_path.empty())
50 callback.Run(StorageInfo::MakeDeviceId(StorageInfo::PICASA, 54 callback.Run(StorageInfo::MakeDeviceId(StorageInfo::PICASA,
51 database_path.AsUTF8Unsafe())); 55 database_path.AsUTF8Unsafe()));
52 } 56 }
53 57
54 } // namespace 58 } // namespace
55 59
56 void PicasaFinder::FindPicasaDatabase( 60 void PicasaFinder::FindPicasaDatabase(
57 const PicasaFinder::DeviceIDCallback& callback) { 61 const PicasaFinder::DeviceIDCallback& callback) {
58 content::BrowserThread::PostTaskAndReplyWithResult( 62 content::BrowserThread::PostTaskAndReplyWithResult(
59 content::BrowserThread::FILE, 63 content::BrowserThread::FILE,
60 FROM_HERE, 64 FROM_HERE,
61 base::Bind(&FindPicasaDatabaseOnFileThread), 65 base::Bind(&FindPicasaDatabaseOnFileThread),
62 base::Bind(&FinishOnOriginalThread, callback)); 66 base::Bind(&FinishOnOriginalThread, callback));
63 } 67 }
64 68
65 } // namespace picasa 69 } // namespace picasa
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698