PicasaAlbumTableReader for Media Galleries API

webkit/fileapi/media/picasa/album_info.h

Index: webkit/fileapi/media/picasa/album_info.h
diff --git a/webkit/fileapi/media/picasa/album_info.h b/webkit/fileapi/media/picasa/album_info.h
new file mode 100644
index 0000000000000000000000000000000000000000..e23fcf122a88522a75b18812cc8e01bbd0d7f93f
--- /dev/null
+++ b/webkit/fileapi/media/picasa/album_info.h
@@ -0,0 +1,31 @@
+// 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.
+
+#ifndef WEBKIT_FILEAPI_MEDIA_PICASA_PICASA_ALBUM_INFO_H_
+#define WEBKIT_FILEAPI_MEDIA_PICASA_PICASA_ALBUM_INFO_H_
+
+#include <string>
+
+#include "base/basictypes.h"
+#include "base/files/file_path.h"
+#include "base/time.h"
+
+namespace picasaimport {
+

[vandebo (ex-Chrome), 2013/04/06 01:10:30]

I think this can all go into pmp_albumdata_table_reader.h

[tommycli, 2013/04/08 17:17:29]

Done.

+enum AlbumType {
+  PICASA_USER_ALBUM,
+  PICASA_FOLDER
+};
+
+struct AlbumInfo {
+  AlbumType category;

[vandebo (ex-Chrome), 2013/04/06 01:10:30]

type?

[tommycli, 2013/04/08 17:17:29]

I'll make two different structs as discussed in next file.

+  base::Time timestamp;

[vandebo (ex-Chrome), 2013/04/06 01:10:30]

Do you plan on using timestamp?  If not, omit it.

[tommycli, 2013/04/08 17:17:29]

Used as first layer of de-duping identical names.

+  base::FilePath path;
+  std::string name;
+  std::string uid;
+};
+
+}  // namespace picasaimport
+
+#endif  // WEBKIT_FILEAPI_MEDIA_PICASA_PICASA_ALBUM_INFO_H_