 Chromium Code Reviews
 Chromium Code Reviews Issue 2807063002:
  Replace Libarchive with MiniZip.  (Closed)
    
  
    Issue 2807063002:
  Replace Libarchive with MiniZip.  (Closed) 
  | Index: ui/file_manager/zip_archiver/cpp/volume_archive.h | 
| diff --git a/ui/file_manager/zip_archiver/cpp/volume_archive.h b/ui/file_manager/zip_archiver/cpp/volume_archive.h | 
| index cd7aac90bbf916db0c93dfe6efb07000c47a1c7a..a55aaf0e17534c2bca2da3396a3a4720c7d3fb67 100644 | 
| --- a/ui/file_manager/zip_archiver/cpp/volume_archive.h | 
| +++ b/ui/file_manager/zip_archiver/cpp/volume_archive.h | 
| @@ -31,16 +31,20 @@ class VolumeArchive { | 
| // archive file. | 
| virtual bool Init(const std::string& encoding) = 0; | 
| - // Gets the next header. In case of failure the error message can be | 
| - // obtained with VolumeArchive::error_message(). | 
| - virtual Result GetNextHeader() = 0; | 
| - virtual Result GetNextHeader(const char** path_name, | 
| - int64_t* size, | 
| - bool* is_directory, | 
| - time_t* modification_time) = 0; | 
| + // Gets the next header. If path_name is set to NULL, then there are no more | 
| + // available headers. Returns true if reading next header was successful. | 
| + // In case of failure the error message can be obtained with | 
| + // VolumeArchive::error_message(). | 
| + virtual VolumeArchive::Result GetCurrentFileInfo( | 
| + std::string* path_name, | 
| + int64_t* size, | 
| + bool* is_directory, | 
| + time_t* modification_time) = 0; | 
| + | 
| + virtual VolumeArchive::Result GoToNextFile() = 0; | 
| // Seeks to the |index|-th header. | 
| 
mtomasz
2017/04/10 07:15:09
nit: Please update comment.
 
takise
2017/04/11 06:00:51
Done.
 | 
| - virtual bool SeekHeader(int64_t index) = 0; | 
| + virtual bool SeekHeader(const std::string& path_name) = 0; | 
| // Gets data from offset to offset + length for the file reached with | 
| // VolumeArchive::GetNextHeader. The data is stored in an internal buffer |