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

Unified Diff: chrome/browser/chromeos/gdata/gdata_directory_service.cc

Issue 10823427: gdata: Change callback type of GDataDirectoryService::GetEntryInfoByResourceId() (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 4 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/chromeos/gdata/gdata_directory_service.cc
diff --git a/chrome/browser/chromeos/gdata/gdata_directory_service.cc b/chrome/browser/chromeos/gdata/gdata_directory_service.cc
index df0192249c344edec92e59b8fe7f9e9e6b4b4c1b..be038c1a9bc582a1e20e0e38ba98bb7b0bec669c 100644
--- a/chrome/browser/chromeos/gdata/gdata_directory_service.cc
+++ b/chrome/browser/chromeos/gdata/gdata_directory_service.cc
@@ -328,25 +328,30 @@ void GDataDirectoryService::GetEntryByResourceIdAsync(
void GDataDirectoryService::GetEntryInfoByResourceId(
const std::string& resource_id,
- const GetEntryInfoCallback& callback) {
+ const GetEntryInfoWithFilePathCallback& callback) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
DCHECK(!callback.is_null());
scoped_ptr<GDataEntryProto> entry_proto;
GDataFileError error = GDATA_FILE_ERROR_FAILED;
+ FilePath drive_file_path;
GDataEntry* entry = GetEntryByResourceId(resource_id);
if (entry) {
entry_proto.reset(new GDataEntryProto);
entry->ToProtoFull(entry_proto.get());
error = GDATA_FILE_OK;
+ drive_file_path = entry->GetFilePath();
} else {
error = GDATA_FILE_ERROR_NOT_FOUND;
}
base::MessageLoopProxy::current()->PostTask(
FROM_HERE,
- base::Bind(callback, error, base::Passed(&entry_proto)));
+ base::Bind(callback,
+ error,
+ drive_file_path,
+ base::Passed(&entry_proto)));
}
void GDataDirectoryService::GetEntryInfoByPath(

Powered by Google App Engine
This is Rietveld 408576698