gdata: Remove use of FindEntryByPathSync() from CopyOnUIThread()

chrome/browser/chromeos/gdata/gdata_file_system.h

 // 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_GDATA_GDATA_FILE_SYSTEM_H_
 #define CHROME_BROWSER_CHROMEOS_GDATA_GDATA_FILE_SYSTEM_H_
 
 #include <map>
 #include <set>
 #include <string>
@@ skipping 718 matching lines @@
       const FileResourceIdMap& file_map,
       GDataEntry* directory_entry);
   void GetAvailableSpaceOnUIThread(const GetAvailableSpaceCallback& callback);
   void AddUploadedFileOnUIThread(UploadMode upload_mode,
                                  const FilePath& virtual_dir_path,
                                  scoped_ptr<DocumentEntry> entry,
                                  const FilePath& file_content_path,
                                  GDataCache::FileOperationType cache_operation,
                                  const base::Closure& callback);
 
+  // Part of CopyOnUIThread(). Called after GetEntryInfoPairByPaths() is
+  // complete.
+  void CopyOnUIThreadAfterGetEntryInfoPair(
+    const FilePath& src_file_path,

[achuithb, 2012/08/09 00:29:28]

Why not get rid of src_file_path and dest_file_path? They are in |result|.

[satorux1, 2012/08/09 08:33:03]

Removed src_file_path.

result.first.path == src_file_path but result.second.path ==
dest_file_path.DirName(), hence cannot remove dest_file_path

+    const FilePath& dest_file_path,
+    const FileOperationCallback& callback,
+    scoped_ptr<EntryInfoPairResult> result);
+
   // Part of RemoveOnUIThread(). Called after GetEntryInfoByPath() is
   // complete.
   void RemoveOnUIThreadAfterGetEntryInfo(
     const FilePath& file_path,
     bool is_recursive,
     const FileOperationCallback& callback,
     GDataFileError error,
     scoped_ptr<GDataEntryProto> entry_proto);
 
   // Part of RequestDirectoryRefreshOnUIThread(). Called after
@@ skipping 68 matching lines @@
   // invalidate the weak pointers before any other members are destroyed.
   base::WeakPtrFactory<GDataFileSystem> ui_weak_ptr_factory_;
   // Unlike other classes, we need this as we need this to redirect a task
   // from IO thread to UI thread.
   base::WeakPtr<GDataFileSystem> ui_weak_ptr_;
 };
 
 }  // namespace gdata
 
 #endif  // CHROME_BROWSER_CHROMEOS_GDATA_GDATA_FILE_SYSTEM_H_