gdata: Convert FindEntryByResourceIdSync() to asynchronous.

chrome/browser/chromeos/gdata/gdata_util.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/gdata/gdata_util.h"
 
 #include <string>
 #include <vector>
 #include <utility>
 
@@ skipping 128 matching lines @@
      kReadOnlyFilePermissions));
   cache_paths->push_back(std::make_pair(
       file_system->GetCacheFilePath(resource_id, file_md5,
           GDataCache::CACHE_TYPE_TMP,
           GDataCache::CACHED_FILE_FROM_SERVER),
       kReadOnlyFilePermissions));
 
   callback.Run();
 }
 
+// Invoked upon completeion of FindEntryByResourceIdAsync initiated by

[achuithb, 2012/06/11 20:46:37]

completion

[hshi1, 2012/06/11 20:58:51]

Done.

+// ModifyGDataFileResourceUrl.
+void OnFindEntryByResourceIdAsync(Profile* profile,
+                                  const std::string& resource_id,
+                                  base::PlatformFileError error,
+                                  GDataEntry* entry) {
+  if (error != base::PLATFORM_FILE_OK || !entry || !entry->AsGDataFile())
+    return;
+
+  std::string file_name = entry->AsGDataFile()->file_name();
+  GURL edit_url = GetFileResourceUrl(resource_id, file_name);
+  OpenEditURLUIThread(profile, &edit_url);
+  DVLOG(1) << "OnFindEntryByResourceIdAsync " << edit_url;
+}
+
 }  // namespace
 
 const FilePath& GetGDataMountPointPath() {
   CR_DEFINE_STATIC_LOCAL(FilePath, gdata_mount_path,
       (FilePath::FromUTF8Unsafe(kGDataMountPointPath)));
   return gdata_mount_path;
 }
 
 const std::string& GetGDataMountPointPathAsString() {
   CR_DEFINE_STATIC_LOCAL(std::string, gdata_mount_path_string,
@@ skipping 35 matching lines @@
         base::Bind(&OpenEditURLUIThread, profile, base::Owned(edit_url)));
     *url = GURL();
     return;
   }
 
   // Handle all other gdata files.
   if (file_system->GetCacheDirectoryPath(
           GDataCache::CACHE_TYPE_TMP).IsParent(gdata_cache_path)) {
     const std::string resource_id =
         gdata_cache_path.BaseName().RemoveExtension().AsUTF8Unsafe();
-    GDataEntry* entry = NULL;
-    file_system->FindEntryByResourceIdSync(
-        resource_id, base::Bind(&ReadOnlyFindEntryCallback, &entry));
-
-    std::string file_name;
-    if (entry && entry->AsGDataFile())
-      file_name = entry->AsGDataFile()->file_name();
-
-    *url = gdata::util::GetFileResourceUrl(resource_id, file_name);
-    DVLOG(1) << "ModifyGDataFileResourceUrl " << *url;
+    file_system->FindEntryByResourceIdAsync(resource_id,
+                                            base::Bind(
+                                                &OnFindEntryByResourceIdAsync,
+                                                profile,
+                                                resource_id));
+    *url = GURL();
+    return;
   }
 }
 
 bool IsUnderGDataMountPoint(const FilePath& path) {
   return GetGDataMountPointPath() == path ||
          GetGDataMountPointPath().IsParent(path);
 }
 
 GDataSearchPathType GetSearchPathStatus(const FilePath& path) {
   std::vector<std::string> components;
@@ skipping 173 matching lines @@
   // Assign the extracted extensions to md5 and extra_extension.
   int extension_count = extensions.size();
   *md5 = (extension_count > 0) ? extensions[extension_count - 1] :
                                  std::string();
   *extra_extension = (extension_count > 1) ? extensions[extension_count - 2] :
                                              std::string();
 }
 
 }  // namespace util
 }  // namespace gdata