Media Galleries API - Picasa: Change PMP Parsing to deal with PlatformFile.

chrome/browser/media_galleries/fileapi/picasa/picasa_album_table_reader.h

Index: chrome/browser/media_galleries/fileapi/picasa/picasa_album_table_reader.h
diff --git a/chrome/browser/media_galleries/fileapi/picasa/picasa_album_table_reader.h b/chrome/browser/media_galleries/fileapi/picasa/picasa_album_table_reader.h
index b4ef981d6620d05132931d182bc13e42fda281aa..0e1a7fa6f9a463a9e8c41833d568ed5f1ffeb2d4 100644
--- a/chrome/browser/media_galleries/fileapi/picasa/picasa_album_table_reader.h
+++ b/chrome/browser/media_galleries/fileapi/picasa/picasa_album_table_reader.h
@@ -7,6 +7,7 @@
 
 #include "base/basictypes.h"
 #include "base/files/file_path.h"
+#include "base/platform_file.h"
 #include "base/time.h"
 
 namespace picasa {
@@ -37,10 +38,23 @@ struct AlbumInfo {
   base::FilePath path;
 };
 
+struct PicasaAlbumTableFiles {
+  explicit PicasaAlbumTableFiles(const base::FilePath& directory_path);
+  void CloseAll();

[Lei Zhang, 2013/06/21 02:15:50]

One you start adding methods, classes are preferred. Since there's only 1
method, just write it as a function that takes a PicasaAlbumTableFiles* ?

[tommycli, 2013/06/21 16:14:23]

Sounds good.

+
+  base::PlatformFile indicator_file;
+

[Lei Zhang, 2013/06/21 02:15:50]

nit: why the extra blank line. Is |indicator_file| special in some way?

[tommycli, 2013/06/21 16:14:23]

Yes, indicator_file is a special empty file used to confirm the existence of the
table. I'll add a comment to that effect.

+  base::PlatformFile category_file;
+  base::PlatformFile date_file;
+  base::PlatformFile filename_file;
+  base::PlatformFile name_file;
+  base::PlatformFile token_file;
+  base::PlatformFile uid_file;
+};
+
 class PicasaAlbumTableReader {
  public:
-  // |directory_path| is Picasa's db3 directory where the PMP table is stored.
-  explicit PicasaAlbumTableReader(const base::FilePath& directory_path);
+  explicit PicasaAlbumTableReader(const PicasaAlbumTableFiles& table_files);
   virtual ~PicasaAlbumTableReader();
 
   bool Init();
@@ -49,7 +63,7 @@ class PicasaAlbumTableReader {
   const std::vector<AlbumInfo>& folders() const;
 
  private:
-  const base::FilePath directory_path_;
+  PicasaAlbumTableFiles table_files_;

[Lei Zhang, 2013/06/21 02:15:50]

looks like this can be const.

[tommycli, 2013/06/21 16:14:23]

Done.

 
   bool initialized_;