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

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

 // Copyright 2013 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/media_galleries/fileapi/picasa/pmp_column_reader.h"
 
 #include <cstring>
 
 #include "base/file_util.h"
-#include "base/files/file_path.h"
 #include "base/logging.h"
 #include "base/threading/thread_restrictions.h"
 
 namespace picasa {
 
 namespace {
 
+COMPILE_ASSERT(sizeof(double) == 8, double_must_be_8_bytes_long);
 const int64 kPmpMaxFilesize = 50*1024*1024;  // Arbitrary maximum of 50 MB.
 
 }  // namespace
 
 PmpColumnReader::PmpColumnReader()
     : length_(0),
       field_type_(PMP_TYPE_INVALID),
       rows_(0) {}
 
 PmpColumnReader::~PmpColumnReader() {}
 
-bool PmpColumnReader::Init(const base::FilePath& filepath,
-                           const PmpFieldType expected_type,
-                           uint32* rows_read) {
+bool PmpColumnReader::Init(const base::PlatformFile& file,
+                           const PmpFieldType expected_type) {
   DCHECK(!data_.get());
   base::ThreadRestrictions::AssertIOAllowed();
 
-  if (!file_util::GetFileSize(filepath, &length_))
+  if (file == base::kInvalidPlatformFileValue)
     return false;
 
+  base::PlatformFileInfo info;
+  if (!base::GetPlatformFileInfo(file, &info))
+    return false;
+
+  length_ = info.size;
+
   if (length_ < kPmpHeaderSize || length_ > kPmpMaxFilesize)
     return false;
 
   data_.reset(new uint8[length_]);
 
   char* data_begin = reinterpret_cast<char*>(data_.get());
 
   DCHECK(length_ < kint32max);  // ReadFile expects an int.
 
-  bool success = file_util::ReadFile(filepath, data_begin, length_) &&
-                 ParseData(expected_type, rows_read);
+  bool success = base::ReadPlatformFile(file, 0, data_begin, length_) &&
+                 ParseData(expected_type);
 
   if (!success)
     rows_ = 0;  // If any of the reading or parsing fails, prevent Read* calls.
 
   return success;
 }
 
 bool PmpColumnReader::ReadString(const uint32 row, std::string* result) const {
   DCHECK(data_.get() != NULL);
 
@@ skipping 38 matching lines @@
 bool PmpColumnReader::ReadUInt64(const uint32 row, uint64* result) const {
   DCHECK(data_.get() != NULL);
 
   if (field_type_ != PMP_TYPE_UINT64 || row >= rows_)
     return false;
 
   *result = reinterpret_cast<uint64*>(data_.get() + kPmpHeaderSize)[row];
   return true;
 }
 
-bool PmpColumnReader::ParseData(const PmpFieldType expected_type,
-                                uint32* rows_read) {
+uint32 PmpColumnReader::rows() const {
+  DCHECK(data_.get() != NULL);
+  return rows_;
+}
+
+bool PmpColumnReader::ParseData(const PmpFieldType expected_type) {
   DCHECK(data_.get() != NULL);
   DCHECK_GE(length_, kPmpHeaderSize);
 
   // Check all magic bytes.
   if (memcmp(&kPmpMagic1, &data_[kPmpMagic1Offset], sizeof(kPmpMagic1)) != 0 ||
       memcmp(&kPmpMagic2, &data_[kPmpMagic2Offset], sizeof(kPmpMagic2)) != 0 ||
       memcmp(&kPmpMagic3, &data_[kPmpMagic3Offset], sizeof(kPmpMagic3)) != 0 ||
       memcmp(&kPmpMagic4, &data_[kPmpMagic4Offset], sizeof(kPmpMagic4)) != 0) {
     return false;
   }
@@ skipping 35 matching lines @@
       expected_body_length = static_cast<int64>(rows_) * sizeof(uint8);
       break;
     case PMP_TYPE_UINT64:
       expected_body_length = static_cast<int64>(rows_) * sizeof(uint64);
       break;
     default:
       return false;
       break;
   }
 
-  if (body_length == expected_body_length && rows_read)
-    *rows_read = rows_;
   return body_length == expected_body_length;
 }
 
 int64 PmpColumnReader::IndexStrings() {
   DCHECK(data_.get() != NULL);
   DCHECK_GE(length_, kPmpHeaderSize);
 
   strings_.reserve(rows_);
 
   int64 bytes_parsed = kPmpHeaderSize;
@@ skipping 12 matching lines @@
 
     strings_.push_back(reinterpret_cast<const char*>(data_cursor));
     data_cursor += length_in_bytes;
     bytes_parsed += length_in_bytes;
   }
 
   return bytes_parsed - kPmpHeaderSize;
 }
 
 }  // namespace picasa