Add gdata content search

chrome/browser/chromeos/gdata/gdata_files.h

 // 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_GDATA_GDATA_FILES_H_
 #define CHROME_BROWSER_CHROMEOS_GDATA_GDATA_FILES_H_
 #pragma once
 
 #include <map>
 #include <string>
@@ skipping 120 matching lines @@
 
   // The resource id of the parent folder. This piece of information is needed
   // to pair files from change feeds with their directory parents withing the
   // existing file system snapshot (GDataRootDirectory::resource_map_).
   const std::string& parent_resource_id() const { return parent_resource_id_; }
 
   // True if file was deleted. Used only for instances that are generated from
   // delta feeds.
   bool is_deleted() const { return deleted_; }
 
+  // True it the entry is not bound to any file system (i.e. doesn't have a root

[Ben Chan, 2012/05/03 22:47:39]

typo: True it -> True if

[tbarzic, 2012/05/03 23:56:17]

Done.

+  // directory set). E.g. |fake_search_directory| below.
+  // NOTE: GDataRootDirectories will return true here, since they have
+  // themselves as root directories.
+  bool is_detached() const { return root_ == NULL; }
+
   // Returns virtual file path representing this file system entry. This path
   // corresponds to file path expected by public methods of GDataFileSyste
   // class.
   FilePath GetFilePath() const;
 
   // Sets |file_name_| based on the value of |title_| without name
   // de-duplication (see AddEntry() for details on de-duplication).
   virtual void SetFileNameFromTitle();
 
  protected:
@@ skipping 338 matching lines @@
 
   // Serializes/Parses to/from string via proto classes.
   void SerializeToString(std::string* serialized_proto) const;
   bool ParseFromString(const std::string& serialized_proto);
 
   // Converts to/from proto.
   void FromProto(const GDataRootDirectoryProto& proto);
   void ToProto(GDataRootDirectoryProto* proto) const;
 
  private:
+  // Used in |FindEntryByPath| if the path that is being searched for is
+  // pointing to a search result path. The find path parameters should be
+  // modified to point to the actual file system entry bound to the search
+  // result.
+  bool ModifyFindEntryParamsIfSearchPath(
+      const FilePath& file_path,
+      std::vector<FilePath::StringType>* components,
+      GDataDirectory** current_dir,
+      FilePath* directory_path);
+
   ResourceMap resource_map_;
   CacheMap cache_map_;
 
+  // Fake directories that will be returned when searching for content search
+  // paths to make file manager happy when resolving paths. These directories
+  // should never  be used for file operations or storing file entries.

[Ben Chan, 2012/05/03 22:47:39]

nit: remove one extra space in "never  be"

[tbarzic, 2012/05/03 23:56:17]

Done.

+  scoped_ptr<GDataDirectory> fake_search_directory_;

[zel, 2012/05/03 22:57:56]

hmm, what if I make two search requests at the same time? if you want/need to
cache results of search you should probably hold the map<query, GDataDirectory>.
I am also not sure that GDataRootDirectory is the best place for it.

[tbarzic, 2012/05/03 23:56:17]

This is not supposed to keep any data. We only need this so we have something to
return when file manager tries to resolve "gdata/.search/foo" or "gdata/.search"
paths (which don't really exist in actual gdata file system).

But, yeah, for caching results, we'll need a map.

[zel, 2012/05/04 00:00:08]

I am confused. Why would we care about this is there is no search data inside?

+
   base::Time last_serialized_;
   int largest_changestamp_;
   size_t serialized_size_;
 
   DISALLOW_COPY_AND_ASSIGN(GDataRootDirectory);
 };
 
 }  // namespace gdata
 
 #endif  // CHROME_BROWSER_CHROMEOS_GDATA_GDATA_FILES_H_