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

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

Index: chrome/browser/media_galleries/fileapi/picasa/picasa_album_table_reader.cc
diff --git a/chrome/browser/media_galleries/fileapi/picasa/picasa_album_table_reader.cc b/chrome/browser/media_galleries/fileapi/picasa/picasa_album_table_reader.cc
index e32e390ddc68d6c27fda89908b2d48ea76fb4dbd..615e20cd9a1e01bd4d35af9be9f200c089ca5234 100644
--- a/chrome/browser/media_galleries/fileapi/picasa/picasa_album_table_reader.cc
+++ b/chrome/browser/media_galleries/fileapi/picasa/picasa_album_table_reader.cc
@@ -4,6 +4,7 @@
 
 #include "chrome/browser/media_galleries/fileapi/picasa/picasa_album_table_reader.h"
 
+#include <algorithm>
 #include <vector>
 
 #include "base/path_service.h"
@@ -11,7 +12,6 @@
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/media_galleries/fileapi/picasa/pmp_column_reader.h"
 #include "chrome/browser/media_galleries/fileapi/picasa/pmp_constants.h"
-#include "chrome/browser/media_galleries/fileapi/picasa/pmp_table_reader.h"
 
 namespace picasa {
 
@@ -25,6 +25,25 @@ base::Time TimeFromMicrosoftVariantTime(double variant_time) {
   return base::Time::FromLocalExploded(kPicasaVariantTimeEpoch) + variant_delta;
 }
 
+base::PlatformFile OpenPlatformFile(const base::FilePath& directory_path,
+                                    const std::string& suffix) {
+  base::FilePath path = directory_path.Append(base::FilePath::FromUTF8Unsafe(
+      std::string(kPicasaAlbumTableName) + "_" + suffix));
+  int flags = base::PLATFORM_FILE_OPEN | base::PLATFORM_FILE_READ;
+  return base::CreatePlatformFile(path, flags, NULL, NULL);
+}
+
+base::PlatformFile OpenColumnPlatformFile(const base::FilePath& directory_path,
+                                          const std::string& column_name) {
+  return OpenPlatformFile(directory_path, column_name + "." + kPmpExtension);
+}
+
+void ClosePlatformFile(base::PlatformFile* platform_file) {
+  DCHECK(platform_file);
+  if (base::ClosePlatformFile(*platform_file))
+    *platform_file = base::kInvalidPlatformFileValue;
+}
+
 }  // namespace
 
 AlbumInfo::AlbumInfo() {}
@@ -39,10 +58,32 @@ AlbumInfo::AlbumInfo(const std::string& name, const base::Time& timestamp,
 
 AlbumInfo::~AlbumInfo() {}
 
+PicasaAlbumTableFiles::PicasaAlbumTableFiles(
+    const base::FilePath& directory_path) {
+  indicator_file = OpenPlatformFile(directory_path, "0");
+  category_file  = OpenColumnPlatformFile(directory_path, "category");
+  date_file      = OpenColumnPlatformFile(directory_path, "date");
+  filename_file  = OpenColumnPlatformFile(directory_path, "filename");
+  name_file      = OpenColumnPlatformFile(directory_path, "name");
+  token_file     = OpenColumnPlatformFile(directory_path, "token");
+  uid_file       = OpenColumnPlatformFile(directory_path, "uid");
+}
+
+void ClosePicasaAlbumTableFiles(PicasaAlbumTableFiles* table_files) {
+  ClosePlatformFile(&(table_files->indicator_file));
+  ClosePlatformFile(&(table_files->category_file));
+  ClosePlatformFile(&(table_files->date_file));
+  ClosePlatformFile(&(table_files->filename_file));
+  ClosePlatformFile(&(table_files->name_file));
+  ClosePlatformFile(&(table_files->token_file));
+  ClosePlatformFile(&(table_files->uid_file));
+}
+
 PicasaAlbumTableReader::PicasaAlbumTableReader(
-    const base::FilePath& directory_path)
-    : directory_path_(directory_path),
-      initialized_(false) {}
+    const PicasaAlbumTableFiles& table_files)
+    : table_files_(table_files),
+      initialized_(false) {
+}
 
 PicasaAlbumTableReader::~PicasaAlbumTableReader() {}
 
@@ -60,33 +101,41 @@ bool PicasaAlbumTableReader::Init() {
   if (initialized_)
     return true;
 
-  PmpTableReader pmp_reader(kPicasaAlbumTableName, directory_path_);
-
-  const PmpColumnReader* category_column =
-      pmp_reader.AddColumn("category", PMP_TYPE_UINT32);
-  const PmpColumnReader* date_column =
-      pmp_reader.AddColumn("date", PMP_TYPE_DOUBLE64);
-  const PmpColumnReader* filename_column =
-      pmp_reader.AddColumn("filename", PMP_TYPE_STRING);
-  const PmpColumnReader* name_column =
-      pmp_reader.AddColumn("name", PMP_TYPE_STRING);
-  const PmpColumnReader* token_column =
-      pmp_reader.AddColumn("token", PMP_TYPE_STRING);
-  const PmpColumnReader* uid_column =
-      pmp_reader.AddColumn("uid", PMP_TYPE_STRING);
-
-  if (pmp_reader.Columns().size() != 6)
+  if (table_files_.indicator_file == base::kInvalidPlatformFileValue)
     return false;
 
-  for (uint32 i = 0; i < pmp_reader.RowCount(); i++) {
+  PmpColumnReader category_column, date_column, filename_column, name_column,
+                  token_column, uid_column;
+  if (!category_column.ReadFile(table_files_.category_file, PMP_TYPE_UINT32) ||
+      !date_column.ReadFile(table_files_.date_file, PMP_TYPE_DOUBLE64) ||
+      !filename_column.ReadFile(table_files_.filename_file, PMP_TYPE_STRING) ||
+      !name_column.ReadFile(table_files_.name_file, PMP_TYPE_STRING) ||
+      !token_column.ReadFile(table_files_.token_file, PMP_TYPE_STRING) ||
+      !uid_column.ReadFile(table_files_.uid_file, PMP_TYPE_STRING)) {
+    return false;
+  }
+
+  // In the PMP format, columns can be different lengths. The number of rows
+  // in the table is max of all the columns, and short columns are NULL padded.
+  uint32 row_count = 0;
+  row_count = std::max(row_count, category_column.rows_read());
+  row_count = std::max(row_count, date_column.rows_read());
+  row_count = std::max(row_count, filename_column.rows_read());
+  row_count = std::max(row_count, name_column.rows_read());
+  row_count = std::max(row_count, token_column.rows_read());
+  row_count = std::max(row_count, uid_column.rows_read());
+
+  for (uint32 i = 0; i < row_count; i++) {
     uint32 category = kAlbumCategoryInvalid;
     double date = 0;
     std::string name;
     std::string uid;
-    if (!category_column->ReadUInt32(i, &category) ||
-        !date_column->ReadDouble64(i, &date) ||
-        !name_column->ReadString(i, &name) || name.empty() ||
-        !uid_column->ReadString(i, &uid) || uid.empty()) {
+    // PMP tables often contain 'garbage' rows of deleted or auto-generated
+    // album-like entities. We ignore those rows.
+    if (!category_column.ReadUInt32(i, &category) ||
+        !date_column.ReadDouble64(i, &date) ||
+        !name_column.ReadString(i, &name) || name.empty() ||
+        !uid_column.ReadString(i, &uid) || uid.empty()) {
       continue;
     }
 
@@ -95,7 +144,7 @@ bool PicasaAlbumTableReader::Init() {
     switch (category) {
       case kAlbumCategoryAlbum: {
         std::string token;
-        if (!token_column->ReadString(i, &token) || token.empty() ||
+        if (!token_column.ReadString(i, &token) || token.empty() ||
             !StartsWithASCII(token, kAlbumTokenPrefix, false)) {
           continue;
         }
@@ -105,7 +154,7 @@ bool PicasaAlbumTableReader::Init() {
       }
       case kAlbumCategoryFolder: {
         std::string filename;
-        if (!filename_column->ReadString(i, &filename) || filename.empty())
+        if (!filename_column.ReadString(i, &filename) || filename.empty())
           continue;
 
         base::FilePath path =