Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4982)

Unified Diff: chrome/browser/media_galleries/fileapi/picasa/pmp_test_helper.h

Issue 13529028: PicasaAlbumTableReader for Media Galleries API (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@0005-picasa-import-pmp-reader
Patch Set: Update gypi file to exclude test on android. Created 7 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/media_galleries/fileapi/picasa/pmp_test_helper.h
diff --git a/chrome/browser/media_galleries/fileapi/picasa/pmp_test_helper.h b/chrome/browser/media_galleries/fileapi/picasa/pmp_test_helper.h
index d675773bba192191dbbd8148aa016521d6914a5b..5cee53eeed91b5a45c546536394433a00b37ccab 100644
--- a/chrome/browser/media_galleries/fileapi/picasa/pmp_test_helper.h
+++ b/chrome/browser/media_galleries/fileapi/picasa/pmp_test_helper.h
@@ -22,20 +22,20 @@ class PmpColumnReader;
// A helper class used for unit tests only
class PmpTestHelper {
public:
- PmpTestHelper();
+ explicit PmpTestHelper(const std::string& table_name);
bool Init();
base::FilePath GetTempDirPath();
template<class T>
- bool WriteColumnFileFromVector(const std::string& table_name,
- const std::string& column_name,
+ bool WriteColumnFileFromVector(const std::string& column_name,
const PmpFieldType field_type,
const std::vector<T>& elements_vector);
bool InitColumnReaderFromBytes(PmpColumnReader* const reader,
const std::vector<uint8>& data,
+ const PmpFieldType expected_type,
uint32* rows_read);
static std::vector<uint8> MakeHeader(const PmpFieldType field_type,
@@ -47,6 +47,7 @@ class PmpTestHelper {
const std::vector<T>& elems);
private:
+ std::string table_name_;
base::ScopedTempDir temp_dir_;
};

Powered by Google App Engine
This is Rietveld 408576698