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

Unified Diff: chrome/browser/drive/fake_drive_service.cc

Issue 137343008: Make chrome.fileSystem.chooseEntry work on Google Drive directories. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 11 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/drive/fake_drive_service.cc
diff --git a/chrome/browser/drive/fake_drive_service.cc b/chrome/browser/drive/fake_drive_service.cc
index c7638950ba39c0c2139af539eb8c7cd9dc1bc2fe..ae0713ae32e78a93d384fb81cf2dbe201305c5da 100644
--- a/chrome/browser/drive/fake_drive_service.cc
+++ b/chrome/browser/drive/fake_drive_service.cc
@@ -781,8 +781,7 @@ CancelCallback FakeDriveService::CopyResource(
resource_id == current_resource_id) {
// Make a copy and set the new resource ID and the new title.
scoped_ptr<base::DictionaryValue> copied_entry(entry->DeepCopy());
- copied_entry->SetString("gd$resourceId.$t",
- resource_id + "_copied");
kinaba 2014/01/16 03:16:31 This old code breaks test that copies the same fil
+ copied_entry->SetString("gd$resourceId.$t", GetNewResourceId());
copied_entry->SetString("title.$t", new_title);
// Reset parent directory.

Powered by Google App Engine
This is Rietveld 408576698