drive: Add drive/trash

chrome/browser/chromeos/drive/search_metadata.cc

Index: chrome/browser/chromeos/drive/search_metadata.cc
diff --git a/chrome/browser/chromeos/drive/search_metadata.cc b/chrome/browser/chromeos/drive/search_metadata.cc
index 505a1d087f725cbe64ec27b1dcaf959f080255dc..2026a6ab0a90bd140ab0df738eb1df3d24a5bbb7 100644
--- a/chrome/browser/chromeos/drive/search_metadata.cc
+++ b/chrome/browser/chromeos/drive/search_metadata.cc
@@ -118,9 +118,11 @@ bool IsEligibleEntry(const ResourceEntry& entry,
     return cache_entry.is_present();
   }
 
-  // Exclude "drive", "drive/root", and "drive/other".
+  // Exclude "drive", "drive/root", "drive/other", "drive/trash" and its
+  // children.
   if (it->GetID() == util::kDriveGrandRootLocalId ||
-      entry.parent_local_id() == util::kDriveGrandRootLocalId) {
+      entry.parent_local_id() == util::kDriveGrandRootLocalId ||
+      entry.parent_local_id() == util::kDriveTrashDirLocalId) {

[kinaba, 2013/11/12 02:10:27]

Is looking only the parent sufficient (i.e., what happens if the user removed a
directory?)

[hashimoto, 2013/11/12 04:08:13]

Hmm, recursively looking up parent does not look efficient.
Maybe we should use a boolean flag for deleted entries instead of drive/trash...

     return false;
   }