drive: Show files shared to me from other account's shared folder.

chrome/browser/chromeos/drive/change_list_processor.cc

 // 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.
 
 #include "chrome/browser/chromeos/drive/change_list_processor.h"
 
 #include "base/metrics/histogram.h"
 #include "base/strings/string_number_conversions.h"
 #include "chrome/browser/chromeos/drive/drive.pb.h"
 #include "chrome/browser/chromeos/drive/file_system_util.h"
@@ skipping 229 matching lines @@
         break;
 
       ResourceEntryMap::iterator it_parent =
           entry_map_.find(parent_resource_id);
       if (it_parent == entry_map_.end()) {
         // Current entry's parent is already updated or not going to be updated,
         // get the parent from the local tree.
         std::string parent_local_id;
         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 FILE_ERROR_NOT_FOUND?

[kinaba, 2013/12/20 07:51:47]

Sounds good. Reverted the treatment of other errors than FILE_ERROR_NOT_FOUND.

-          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;
         while (it_parent == entry_map_.end() && !parent_local_id.empty()) {
           error = resource_metadata_->GetResourceEntryById(
               parent_local_id, &parent_entry);
           if (error != FILE_ERROR_OK) {
             LOG(ERROR) << "Failed to get local entry: "
                        << FileErrorToString(error);
             break;
@@ skipping 191 matching lines @@
         resource_metadata_->GetSubDirectoriesRecursively(local_id,
                                                          &sub_directories);
         changed_dirs_.insert(sub_directories.begin(), sub_directories.end());
       }
     }
   }
 }
 
 }  // namespace internal
 }  // namespace drive