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

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

Issue 445473002: Remove ResourceIdCanonicalizer from DriveServiceInterface (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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 b46845bcb34c05b356c2a0986d759bedf848d536..39cb7b00359e5e2a2e88524943e799e93ed68bfc 100644
--- a/chrome/browser/chromeos/drive/file_system/copy_operation.cc
+++ b/chrome/browser/chromeos/drive/file_system/copy_operation.cc
@@ -279,14 +279,12 @@ CopyOperation::CopyOperation(base::SequencedTaskRunner* blocking_task_runner,
OperationDelegate* delegate,
JobScheduler* scheduler,
internal::ResourceMetadata* metadata,
- internal::FileCache* cache,
- const ResourceIdCanonicalizer& id_canonicalizer)
+ internal::FileCache* cache)
: blocking_task_runner_(blocking_task_runner),
delegate_(delegate),
scheduler_(scheduler),
metadata_(metadata),
cache_(cache),
- id_canonicalizer_(id_canonicalizer),
create_file_operation_(new CreateFileOperation(blocking_task_runner,
delegate,
metadata)),
@@ -459,7 +457,7 @@ void CopyOperation::TransferFileFromLocalToRemoteAfterPrepare(
// GDoc file may contain a resource ID in the old format.
const std::string canonicalized_resource_id =
- id_canonicalizer_.Run(*gdoc_resource_id);
+ util::CanonicalizeResourceId(*gdoc_resource_id);
// Drop the document extension, which should not be in the title.
// TODO(yoshiki): Remove this code with crbug.com/223304.

Powered by Google App Engine
This is Rietveld 408576698