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

Unified Diff: chrome/browser/chromeos/drive/change_list_processor.cc

Issue 119823002: drive: Show files shared to me from other account's shared folder. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years 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/change_list_processor.cc
diff --git a/chrome/browser/chromeos/drive/change_list_processor.cc b/chrome/browser/chromeos/drive/change_list_processor.cc
index 0fd13239d7a0cd23b566a9f4699d32f27524e8a6..1d16e7f4ba2189ed378cca4c34e783ef9b6f0be1 100644
--- a/chrome/browser/chromeos/drive/change_list_processor.cc
+++ b/chrome/browser/chromeos/drive/change_list_processor.cc
@@ -248,8 +248,11 @@ FileError ChangeListProcessor::ApplyEntryMap(
FileError error = resource_metadata_->GetIdByResourceId(
parent_resource_id, &parent_local_id);
if (error != FILE_ERROR_OK) {
hashimoto 2013/12/20 07:41:56 nit; Could you emit an error for errors other than
kinaba 2013/12/20 07:51:47 Sounds good. Reverted the treatment of other error
- LOG(ERROR) << "Failed to get local ID: " << parent_resource_id
- << ", error = " << FileErrorToString(error);
+ // See crbug.com/326043. In some complicated situations, parent folder
+ // for shared entries may be accessible (and hence its resource id is
+ // included), but not in the change/file list.
+ // In such a case, clear the parent and move it to drive/other.
+ parent_resource_id_map_[it->first] = "";
break;
}
ResourceEntry parent_entry;
« no previous file with comments | « chrome/browser/chromeos/drive/change_list_processor.h ('k') | chrome/browser/chromeos/drive/change_list_processor_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698