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

Unified Diff: chrome/browser/chromeos/drive/file_system/copy_operation.cc

Issue 22885006: Remove TransferFileFromRemoteToLocal, which are no longer used. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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/drive/file_system/copy_operation.cc
diff --git a/chrome/browser/chromeos/drive/file_system/copy_operation.cc b/chrome/browser/chromeos/drive/file_system/copy_operation.cc
index af4435b74f81d3f9632a814b9458058c639746ab..3ec30b5926b6818249b11771f5d76448acaf7945 100644
--- a/chrome/browser/chromeos/drive/file_system/copy_operation.cc
+++ b/chrome/browser/chromeos/drive/file_system/copy_operation.cc
@@ -27,16 +27,6 @@ namespace file_system {
namespace {
-// Copies a file from |src_file_path| to |dest_file_path| on the local
-// file system using base::CopyFile.
-// Returns FILE_ERROR_OK on success or FILE_ERROR_FAILED otherwise.
-FileError CopyLocalFileOnBlockingPool(
- const base::FilePath& src_file_path,
- const base::FilePath& dest_file_path) {
- return base::CopyFile(src_file_path, dest_file_path) ?
- FILE_ERROR_OK : FILE_ERROR_FAILED;
-}
-
// Stores a file to the cache and mark it dirty.
FileError StoreAndMarkDirty(internal::FileCache* cache,
const std::string& resource_id,
@@ -126,49 +116,6 @@ void CopyOperation::Copy(const base::FilePath& src_file_path,
callback));
}
-void CopyOperation::TransferFileFromRemoteToLocal(
- const base::FilePath& remote_src_file_path,
- const base::FilePath& local_dest_file_path,
- const FileOperationCallback& callback) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- DCHECK(!callback.is_null());
-
- download_operation_->EnsureFileDownloadedByPath(
- remote_src_file_path,
- ClientContext(USER_INITIATED),
- GetFileContentInitializedCallback(),
- google_apis::GetContentCallback(),
- base::Bind(&CopyOperation::OnGetFileCompleteForTransferFile,
- weak_ptr_factory_.GetWeakPtr(),
- local_dest_file_path,
- callback));
-}
-
-void CopyOperation::OnGetFileCompleteForTransferFile(
- const base::FilePath& local_dest_file_path,
- const FileOperationCallback& callback,
- FileError error,
- const base::FilePath& local_file_path,
- scoped_ptr<ResourceEntry> entry) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- DCHECK(!callback.is_null());
-
- if (error != FILE_ERROR_OK) {
- callback.Run(error);
- return;
- }
-
- // GetFileByPath downloads the file from Drive to a local cache, which is then
- // copied to the actual destination path on the local file system using
- // CopyLocalFileOnBlockingPool.
- base::PostTaskAndReplyWithResult(
- blocking_task_runner_.get(),
- FROM_HERE,
- base::Bind(
- &CopyLocalFileOnBlockingPool, local_file_path, local_dest_file_path),
- callback);
-}
-
void CopyOperation::TransferFileFromLocalToRemote(
const base::FilePath& local_src_file_path,
const base::FilePath& remote_dest_file_path,

Powered by Google App Engine
This is Rietveld 408576698