Replace Libarchive with MiniZip.

ui/file_manager/zip_archiver/cpp/compressor_io_javascript_stream.h

Index: ui/file_manager/zip_archiver/cpp/compressor_io_javascript_stream.h
diff --git a/ui/file_manager/zip_archiver/cpp/compressor_io_javascript_stream.h b/ui/file_manager/zip_archiver/cpp/compressor_io_javascript_stream.h
index d0a22d854f5e67f6c52e2ec14727aabe69d50560..21ac2a6ee68585e5d00407f9113ce6a7a62bdff6 100644
--- a/ui/file_manager/zip_archiver/cpp/compressor_io_javascript_stream.h
+++ b/ui/file_manager/zip_archiver/cpp/compressor_io_javascript_stream.h
@@ -8,7 +8,6 @@
 #include <pthread.h>
 #include <string>
 
-#include "archive.h"
 #include "ppapi/cpp/instance_handle.h"
 #include "ppapi/cpp/var_array_buffer.h"
 #include "ppapi/utility/completion_callback_factory.h"
@@ -18,6 +17,12 @@
 #include "compressor_stream.h"
 #include "javascript_compressor_requestor_interface.h"
 
+// A namespace with constants used by CompressorArchiveLibarchive.
+namespace compressor_stream_constants {
+// We need at least 256KB for MiniZip.
+const int64_t kMaximumDataChunkSize = 512 * 1024;

[mtomasz, 2017/04/10 07:15:09]

nit: AFAIK initializations should be in .cc file.

[takise, 2017/04/11 06:00:51]

I want to place this constant here because this value is used in several .cc
files.
You mean we should just declare this variable here and assign the value in .cc?

+}  // namespace compressor_archive_constants
+
 class CompressorIOJavaScriptStream : public CompressorStream {
  public:
   CompressorIOJavaScriptStream(
@@ -25,15 +30,21 @@ class CompressorIOJavaScriptStream : public CompressorStream {
 
   virtual ~CompressorIOJavaScriptStream();
 
-  virtual int64_t Write(int64_t bytes_to_read,
-                        const pp::VarArrayBuffer& buffer);
+  // Flushes the data in buffer_. Since minizip sends tons of write requests and
+  // communication between C++ and JS is very expensive, we need to cache data
+  // in buffer_ and send them in a lump.
+  virtual int64_t Flush();
+
+  virtual int64_t Write(int64_t zip_offset,
+                        int64_t zip_length,
+                        const void* zip_buffer);
 
-  virtual void WriteChunkDone(int64_t write_bytes);
+  virtual int64_t WriteChunkDone(int64_t write_bytes);
 
   virtual int64_t Read(int64_t bytes_to_read, char* destination_buffer);
 
-  virtual void ReadFileChunkDone(int64_t read_bytes,
-                                 pp::VarArrayBuffer* buffer);
+  virtual int64_t ReadFileChunkDone(int64_t read_bytes,
+                                    pp::VarArrayBuffer* buffer);
 
 private:
   // A requestor that makes calls to JavaScript to read and write chunks.
@@ -58,6 +69,15 @@ private:
 
   // Stores the data read from JavaScript.
   char* destination_buffer_;
+
+  // The current offset from which buffer_ has data.
+  int64_t buffer_offset_;
+
+  // The size of the data in buffer_.
+  int64_t buffer_data_length_;
+
+  // The buffer that contains cached data.
+  char buffer_[compressor_stream_constants::kMaximumDataChunkSize];
 };
 
 #endif  // COMPRESSOR_IO_JAVSCRIPT_STREAM_H_