New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 607078 link

Starred by 5 users

Issue metadata

Status: Fixed
Owner:
Email to this user bounced
Closed: Sep 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Chrome
Pri: 1
Type: Bug

Blocking:
issue 359837
issue 665468



Sign in to add a comment

Migrate Zip Unpacker to a normal component extension

Project Member Reported by mtomasz@chromium.org, Apr 27 2016

Issue description

As a result, we won't need to download it from the Web Store. Also, development will be easier. Finally, it will become available in the Guest mode.
 
Cc: mtomasz@chromium.org

Comment 2 by fukino@chromium.org, Jan 19 2017

Cc: takise@google.com yawano@chromium.org

Comment 3 by fukino@chromium.org, Jan 19 2017

Cc: fukino@chromium.org
Labels: -Pri-3 Pri-2
Tomasz, is there a blocker or bug for this issue?

To add the ability to zip folders, zip unpacker want to use fileManagerPrivate.resolveIsolatedEntries().
And we want to update zip unpacker and file manager at the same time.

Making the zip unpacker a component extension will make things easier.
The blocker was a difficulty to compile NaCl module using Chrome build system. I didn't try, but I remember that no one has done it before. However, maybe it's worth trying. I agree it would make things way way simpler if we could keep the Zip Unpacker extension in Chromium repo.
Blocking: 665468
Project Member

Comment 6 by bugdroid1@chromium.org, Apr 11 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/cac7340db735b9e8228e8b5abf886b20719db262

commit cac7340db735b9e8228e8b5abf886b20719db262
Author: takise <takise@google.com>
Date: Tue Apr 11 07:39:18 2017

Replace Libarchive with MiniZip.

This CL rewrites all the components that depend on libarchive. Since we
plan to add the files related to build such as BUILD.gn in another CL,
GN doesn't compile the code in this CL yet.

BUG= 607078 
TEST=manually tested
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2807063002
Cr-Commit-Position: refs/heads/master@{#463567}

[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/compressor.cc
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/compressor.h
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/compressor_archive.h
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/compressor_archive_libarchive.cc
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/compressor_archive_libarchive.h
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/compressor_io_javascript_stream.cc
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/compressor_io_javascript_stream.h
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/compressor_stream.h
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/javascript_compressor_requestor_interface.h
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/javascript_message_sender_interface.h
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/module.cc
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/request.cc
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/request.h
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/volume.cc
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/volume.h
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/volume_archive.h
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/volume_archive_libarchive.cc
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/volume_archive_libarchive.h
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/volume_reader.h
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/volume_reader_javascript_stream.cc
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/cpp/volume_reader_javascript_stream.h
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/html/passphrase-dialog.html
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/js/app.js
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/js/background.js
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/js/compressor.js
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/js/passphrase-dialog.js
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/js/request.js
[modify] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/js/volume.js
[add] https://crrev.com/cac7340db735b9e8228e8b5abf886b20719db262/ui/file_manager/zip_archiver/module.nmf.txt

Project Member

Comment 7 by bugdroid1@chromium.org, Apr 12 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1f46c3f8167ab959fa80fa2e8b1d9a56e5e8bd46

commit 1f46c3f8167ab959fa80fa2e8b1d9a56e5e8bd46
Author: takise <takise@google.com>
Date: Wed Apr 12 07:56:43 2017

Rename variable, class, function, etc from libarchive to minizip.

This CL renames all the identifiers with libarchive to minizip.
Note that we are going to change the filenames in another CL.

BUG= 607078 
TEST=manually tested
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2815063002
Cr-Commit-Position: refs/heads/master@{#463952}

[modify] https://crrev.com/1f46c3f8167ab959fa80fa2e8b1d9a56e5e8bd46/ui/file_manager/zip_archiver/cpp/compressor.cc
[modify] https://crrev.com/1f46c3f8167ab959fa80fa2e8b1d9a56e5e8bd46/ui/file_manager/zip_archiver/cpp/compressor.h
[modify] https://crrev.com/1f46c3f8167ab959fa80fa2e8b1d9a56e5e8bd46/ui/file_manager/zip_archiver/cpp/compressor_archive_libarchive.cc
[modify] https://crrev.com/1f46c3f8167ab959fa80fa2e8b1d9a56e5e8bd46/ui/file_manager/zip_archiver/cpp/compressor_archive_libarchive.h
[modify] https://crrev.com/1f46c3f8167ab959fa80fa2e8b1d9a56e5e8bd46/ui/file_manager/zip_archiver/cpp/compressor_io_javascript_stream.h
[modify] https://crrev.com/1f46c3f8167ab959fa80fa2e8b1d9a56e5e8bd46/ui/file_manager/zip_archiver/cpp/module.cc
[modify] https://crrev.com/1f46c3f8167ab959fa80fa2e8b1d9a56e5e8bd46/ui/file_manager/zip_archiver/cpp/volume.cc
[modify] https://crrev.com/1f46c3f8167ab959fa80fa2e8b1d9a56e5e8bd46/ui/file_manager/zip_archiver/cpp/volume.h
[modify] https://crrev.com/1f46c3f8167ab959fa80fa2e8b1d9a56e5e8bd46/ui/file_manager/zip_archiver/cpp/volume_archive_libarchive.cc
[modify] https://crrev.com/1f46c3f8167ab959fa80fa2e8b1d9a56e5e8bd46/ui/file_manager/zip_archiver/cpp/volume_archive_libarchive.h
[modify] https://crrev.com/1f46c3f8167ab959fa80fa2e8b1d9a56e5e8bd46/ui/file_manager/zip_archiver/cpp/volume_reader.h
[modify] https://crrev.com/1f46c3f8167ab959fa80fa2e8b1d9a56e5e8bd46/ui/file_manager/zip_archiver/cpp/volume_reader_javascript_stream.cc
[modify] https://crrev.com/1f46c3f8167ab959fa80fa2e8b1d9a56e5e8bd46/ui/file_manager/zip_archiver/cpp/volume_reader_javascript_stream.h
[modify] https://crrev.com/1f46c3f8167ab959fa80fa2e8b1d9a56e5e8bd46/ui/file_manager/zip_archiver/js/app.js
[modify] https://crrev.com/1f46c3f8167ab959fa80fa2e8b1d9a56e5e8bd46/ui/file_manager/zip_archiver/js/compressor.js
[modify] https://crrev.com/1f46c3f8167ab959fa80fa2e8b1d9a56e5e8bd46/ui/file_manager/zip_archiver/js/decompressor.js

Project Member

Comment 8 by bugdroid1@chromium.org, Apr 13 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/77065d9816b34a5cc7913894b8b756e7c3e6fb79

commit 77065d9816b34a5cc7913894b8b756e7c3e6fb79
Author: takise <takise@google.com>
Date: Thu Apr 13 05:28:56 2017

Build ZIP archiver.

This CL adds and changes some files in ui/file_manager to build ZIP archiver.
We are going to add ZIP archiver to the list of component extensions
in another CL, so ZIP archiver still won't show up in the context
menu with this patch.

BUG= 607078 
TEST=manually tested
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2808993003
Cr-Commit-Position: refs/heads/master@{#464309}

[modify] https://crrev.com/77065d9816b34a5cc7913894b8b756e7c3e6fb79/ui/file_manager/BUILD.gn
[modify] https://crrev.com/77065d9816b34a5cc7913894b8b756e7c3e6fb79/ui/file_manager/file_manager_resources.grd
[add] https://crrev.com/77065d9816b34a5cc7913894b8b756e7c3e6fb79/ui/file_manager/zip_archiver/BUILD.gn
[add] https://crrev.com/77065d9816b34a5cc7913894b8b756e7c3e6fb79/ui/file_manager/zip_archiver/cpp/BUILD.gn
[modify] https://crrev.com/77065d9816b34a5cc7913894b8b756e7c3e6fb79/ui/file_manager/zip_archiver/manifest.json

Project Member

Comment 9 by bugdroid1@chromium.org, Apr 13 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/fca615ce366c3d28d9a0a48522313a3710306be1

commit fca615ce366c3d28d9a0a48522313a3710306be1
Author: findit-for-me <findit-for-me@appspot.gserviceaccount.com>
Date: Thu Apr 13 06:00:58 2017

Revert of Build ZIP archiver in ui/file_manager. (patchset #5 id:80001 of https://codereview.chromium.org/2808993003/ )

Reason for revert:

Findit(https://goo.gl/kROfz5) identified CL at revision 464309 as the
culprit for failures in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtLzc3MDY1ZDk4MTZiMzRhNWNjNzkxMzg5NGI4Yjc1NmU3YzNlNmZiNzkM

Original issue's description:
> Build ZIP archiver.
>
> This CL adds and changes some files in ui/file_manager to build ZIP archiver.
> We are going to add ZIP archiver to the list of component extensions
> in another CL, so ZIP archiver still won't show up in the context
> menu with this patch.
>
> BUG= 607078 
> TEST=manually tested
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
>
> Review-Url: https://codereview.chromium.org/2808993003
> Cr-Commit-Position: refs/heads/master@{#464309}
> Committed: https://chromium.googlesource.com/chromium/src/+/77065d9816b34a5cc7913894b8b756e7c3e6fb79

TBR=mtomasz@chromium.org,fukino@chromium.org,hashimoto@chromium.org,takise@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 607078 

Review-Url: https://codereview.chromium.org/2811183006
Cr-Commit-Position: refs/heads/master@{#464315}

[modify] https://crrev.com/fca615ce366c3d28d9a0a48522313a3710306be1/ui/file_manager/BUILD.gn
[modify] https://crrev.com/fca615ce366c3d28d9a0a48522313a3710306be1/ui/file_manager/file_manager_resources.grd
[delete] https://crrev.com/cd011686c4e1fb0b82628b8c08fde2f8cc7b9bcb/ui/file_manager/zip_archiver/BUILD.gn
[delete] https://crrev.com/cd011686c4e1fb0b82628b8c08fde2f8cc7b9bcb/ui/file_manager/zip_archiver/cpp/BUILD.gn
[modify] https://crrev.com/fca615ce366c3d28d9a0a48522313a3710306be1/ui/file_manager/zip_archiver/manifest.json

Project Member

Comment 10 by bugdroid1@chromium.org, Apr 13 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/cf1686e9b3003b61af2c2d6e9793bfcb3627e9c9

commit cf1686e9b3003b61af2c2d6e9793bfcb3627e9c9
Author: takise <takise@google.com>
Date: Thu Apr 13 06:49:25 2017

Remove a line that is related to libarchive.

This CL just remove a line which depends on libarchive.

BUG= 607078 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2815083003
Cr-Commit-Position: refs/heads/master@{#464324}

[modify] https://crrev.com/cf1686e9b3003b61af2c2d6e9793bfcb3627e9c9/ui/file_manager/zip_archiver/cpp/volume_archive_libarchive.cc

Project Member

Comment 11 by bugdroid1@chromium.org, Apr 13 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a1644c12446b4d6638ea52d567bc92eef927d61b

commit a1644c12446b4d6638ea52d567bc92eef927d61b
Author: takise <takise@google.com>
Date: Thu Apr 13 08:29:56 2017

Reland: Build ZIP archiver.

This CL adds and changes some files in ui/file_manager to build ZIP archiver.
We are going to add ZIP archiver to the list of component extensions
in another CL, so ZIP archiver still won't show up in the context
menu with this patch.

BUG= 607078 
TEST=manually tested
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2808993003
Cr-Commit-Position: refs/heads/master@{#464309}
Committed: https://chromium.googlesource.com/chromium/src/+/77065d9816b34a5cc7913894b8b756e7c3e6fb79

patch from issue 2808993003 at patchset 80001 (http://crrev.com/2808993003#ps80001)

Review-Url: https://codereview.chromium.org/2814313003
Cr-Commit-Position: refs/heads/master@{#464340}

[modify] https://crrev.com/a1644c12446b4d6638ea52d567bc92eef927d61b/ui/file_manager/BUILD.gn
[modify] https://crrev.com/a1644c12446b4d6638ea52d567bc92eef927d61b/ui/file_manager/file_manager_resources.grd
[add] https://crrev.com/a1644c12446b4d6638ea52d567bc92eef927d61b/ui/file_manager/zip_archiver/BUILD.gn
[add] https://crrev.com/a1644c12446b4d6638ea52d567bc92eef927d61b/ui/file_manager/zip_archiver/cpp/BUILD.gn
[modify] https://crrev.com/a1644c12446b4d6638ea52d567bc92eef927d61b/ui/file_manager/zip_archiver/manifest.json

Project Member

Comment 13 by bugdroid1@chromium.org, Apr 14 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/db5677fefa27566faa217994832aaf16272f42e1

commit db5677fefa27566faa217994832aaf16272f42e1
Author: takise <takise@google.com>
Date: Fri Apr 14 06:19:17 2017

Change the copyright from Chromium OS to Chromium.

BUG= 607078 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2822673002
Cr-Commit-Position: refs/heads/master@{#464690}

[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/Makefile
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/Makefile.package
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/Makefile.pnacl
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/compressor.cc
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/compressor.h
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/compressor_archive.h
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/compressor_archive_minizip.cc
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/compressor_archive_minizip.h
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/compressor_io_javascript_stream.cc
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/compressor_io_javascript_stream.h
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/compressor_stream.h
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/javascript_compressor_requestor_interface.h
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/javascript_message_sender_interface.h
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/javascript_requestor_interface.h
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/module.cc
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/request.cc
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/request.h
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/volume.cc
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/volume.h
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/volume_archive.h
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/volume_archive_minizip.cc
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/volume_archive_minizip.h
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/volume_reader.h
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/volume_reader_javascript_stream.cc
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/cpp/volume_reader_javascript_stream.h
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/gdb_script
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/js/app.js
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/js/background.js
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/js/build-config.js
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/js/compressor-foreground.js
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/js/compressor.js
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/js/decompressor.js
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/js/passphrase-dialog.js
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/js/passphrase-manager.js
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/js/request.js
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/js/volume.js
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/cpp/Makefile
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/cpp/fake_lib_archive.cc
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/cpp/fake_lib_archive.h
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/cpp/fake_volume_reader.cc
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/cpp/fake_volume_reader.h
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/cpp/index.html
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/cpp/index.js
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/cpp/main.cc
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/cpp/request_test.cc
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/cpp/volume_archive_libarchive_read_test.cc
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/cpp/volume_archive_libarchive_test.cc
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/cpp/volume_reader_javascript_stream_test.cc
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/cpp/volume_test.cc
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/js/decompressor_test.js
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/js/integration_specific_archives_tests.js
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/js/integration_test.js
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/js/integration_test_helper.js
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/js/passphrase_manager_test.js
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/js/request_test.js
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/js/test_utils.js
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/js/volume_test.js
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/karma.conf.js
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/run_cpp_tests.sh
[modify] https://crrev.com/db5677fefa27566faa217994832aaf16272f42e1/ui/file_manager/zip_archiver/unpacker-test/run_js_tests.sh

Project Member

Comment 14 by bugdroid1@chromium.org, Apr 20 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2cab8929074f911590ef5a1da345b5519dfc1c81

commit 2cab8929074f911590ef5a1da345b5519dfc1c81
Author: takise <takise@google.com>
Date: Thu Apr 20 08:25:39 2017

Make ZIP archiver a component extension.

This CL adds ZIP archiver to the list of component extensions. Currently,
ZIP archiver is in ui/third_party/zip_archiver. This extension was
originally maintained in https://chromium.googlesource.com/apps/unpacker.

By making it a component extension, we can get the following benefits.
(1) Tighter integration into Files app on Chrome OS.
(2) Availability in guest mode.
(3) Automated tests.
(4) Easier development cycles.

ZIP archiver uses Native Client to wrap MiniZip in C++. Since .pexe file,
which is a binary file that contains modules written in C++, is built in
compile time, we need to manually specify the path of the file in
chrome_component_extension_resource_manager.cc.
Note that we use .pexe.js instead of .pexe because .grd file ignores .pexe files.

BUG= 607078 
TEST=manually tested

Review-Url: https://codereview.chromium.org/2805183002
Cr-Commit-Position: refs/heads/master@{#465947}

[modify] https://crrev.com/2cab8929074f911590ef5a1da345b5519dfc1c81/chrome/browser/extensions/chrome_component_extension_resource_manager.cc
[modify] https://crrev.com/2cab8929074f911590ef5a1da345b5519dfc1c81/chrome/browser/extensions/component_extensions_whitelist/whitelist.cc
[modify] https://crrev.com/2cab8929074f911590ef5a1da345b5519dfc1c81/chrome/browser/extensions/component_loader.cc
[modify] https://crrev.com/2cab8929074f911590ef5a1da345b5519dfc1c81/chrome/browser/extensions/component_loader.h

Cc: yamaguchi@chromium.org
Labels: M-63
Owner: klemenko@google.com
Status: Assigned (was: Available)
Summary: Migrate Zip Unpacker to a normal component extension (was: Make ZIP unpacker a component extension.)
Labels: -Pri-2 Pri-1
Blocking: 359837
Project Member

Comment 18 by bugdroid1@chromium.org, Sep 13 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2107279323b8a0674182e01066de7c0dc449c2e4

commit 2107279323b8a0674182e01066de7c0dc449c2e4
Author: Klemen Kozjek <klemenko@google.com>
Date: Wed Sep 13 09:48:32 2017

Zip Archiver: Set/Read last modified date of a file/folder

This CL adds an implementation for setting and reading last modified
date of a file/folder in a zip file. As a result, last modified date of a
file will not be default value anymore.

BUG= 607078 

Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Idfcb3249360aa7c6d8ca8004a39ce38813bf4932
Reviewed-on: https://chromium-review.googlesource.com/664499
Commit-Queue: Klemen Kozjek <klemenko@google.com>
Reviewed-by: Yuki Awano <yawano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#501577}
[modify] https://crrev.com/2107279323b8a0674182e01066de7c0dc449c2e4/chrome/browser/resources/chromeos/zip_archiver/cpp/compressor_archive_minizip.cc
[modify] https://crrev.com/2107279323b8a0674182e01066de7c0dc449c2e4/chrome/browser/resources/chromeos/zip_archiver/cpp/volume_archive_minizip.cc

Status: Started (was: Assigned)
Project Member

Comment 20 by bugdroid1@chromium.org, Sep 15 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6a2af82f83de5e17b7c794de9a8c51793a324749

commit 6a2af82f83de5e17b7c794de9a8c51793a324749
Author: Klemen Kozjek <klemenko@google.com>
Date: Fri Sep 15 06:00:56 2017

Zip Archiver: Skip zip file mount entry state maintenance in guest mode

As of now, when we mount a zip file we maintain mounted entries through
retainEntry private method and use that information to restore them upon
device restart or session login. Because retainEntry method does not work
under incognito mode we return an error message if retainEntry is called.
Incognito mode does not retain changes made in its session and therefore
we skip calls that would otherwise call retainEntry method.

As a result, Zip Archiver works in incognito/guest mode.

Bug:  607078 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I9220eea4ed9ea8b9e873698d44cd6e167194b071
Reviewed-on: https://chromium-review.googlesource.com/663002
Commit-Queue: Klemen Kozjek <klemenko@google.com>
Reviewed-by: Tomasz Mikolajewski <mtomasz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#502171}
[modify] https://crrev.com/6a2af82f83de5e17b7c794de9a8c51793a324749/chrome/browser/resources/chromeos/zip_archiver/js/app.js
[modify] https://crrev.com/6a2af82f83de5e17b7c794de9a8c51793a324749/extensions/browser/api/file_system/file_system_api.cc

Project Member

Comment 21 by bugdroid1@chromium.org, Sep 22 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4316c85fdbc3c8874d0283d54ba63d7af17b38d9

commit 4316c85fdbc3c8874d0283d54ba63d7af17b38d9
Author: Klemen Kozjek <klemenko@google.com>
Date: Fri Sep 22 07:38:55 2017

Zip Archiver: Remember a password if checkbox is checked

As a result, dialog for password will appear only one time if checkbox
to remember the password is checked.

Bug:  607078 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: If1a1d2386dc4f11b9562072dfa46779c468b9b95
Reviewed-on: https://chromium-review.googlesource.com/666936
Reviewed-by: Yuki Awano <yawano@chromium.org>
Commit-Queue: Klemen Kozjek <klemenko@google.com>
Cr-Commit-Position: refs/heads/master@{#503678}
[modify] https://crrev.com/4316c85fdbc3c8874d0283d54ba63d7af17b38d9/chrome/browser/resources/chromeos/zip_archiver/cpp/volume_archive_minizip.cc
[modify] https://crrev.com/4316c85fdbc3c8874d0283d54ba63d7af17b38d9/chrome/browser/resources/chromeos/zip_archiver/cpp/volume_archive_minizip.h
[modify] https://crrev.com/4316c85fdbc3c8874d0283d54ba63d7af17b38d9/chrome/browser/resources/chromeos/zip_archiver/cpp/volume_reader.h
[modify] https://crrev.com/4316c85fdbc3c8874d0283d54ba63d7af17b38d9/chrome/browser/resources/chromeos/zip_archiver/cpp/volume_reader_javascript_stream.cc
[modify] https://crrev.com/4316c85fdbc3c8874d0283d54ba63d7af17b38d9/chrome/browser/resources/chromeos/zip_archiver/cpp/volume_reader_javascript_stream.h
[modify] https://crrev.com/4316c85fdbc3c8874d0283d54ba63d7af17b38d9/chrome/browser/resources/chromeos/zip_archiver/js/app.js
[modify] https://crrev.com/4316c85fdbc3c8874d0283d54ba63d7af17b38d9/chrome/browser/resources/chromeos/zip_archiver/js/decompressor.js
[modify] https://crrev.com/4316c85fdbc3c8874d0283d54ba63d7af17b38d9/chrome/browser/resources/chromeos/zip_archiver/js/passphrase-manager.js
[modify] https://crrev.com/4316c85fdbc3c8874d0283d54ba63d7af17b38d9/chrome/browser/resources/chromeos/zip_archiver/js/volume.js

Project Member

Comment 22 by bugdroid1@chromium.org, Sep 25 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b3d554f873f245695b68a078c3eb35d835e2f152

commit b3d554f873f245695b68a078c3eb35d835e2f152
Author: Klemen Kozjek <klemenko@google.com>
Date: Mon Sep 25 10:42:23 2017

Zip Archiver: Add a support for AES encryption/decryption

This CL adds third party files to build script for minizip library that are required for
AES encryption/decryption implementation. As a result, Zip Archiver is
able to open zip files encrypted with AES.

Bug:  607078 
Change-Id: I8c526de62c139be606fb0cbf90aecb948084345e
Reviewed-on: https://chromium-review.googlesource.com/662983
Reviewed-by: Yuki Awano <yawano@chromium.org>
Commit-Queue: Klemen Kozjek <klemenko@google.com>
Cr-Commit-Position: refs/heads/master@{#504024}
[modify] https://crrev.com/b3d554f873f245695b68a078c3eb35d835e2f152/DEPS
[modify] https://crrev.com/b3d554f873f245695b68a078c3eb35d835e2f152/third_party/minizip/BUILD.gn
[modify] https://crrev.com/b3d554f873f245695b68a078c3eb35d835e2f152/third_party/minizip/README.chromium

Project Member

Comment 23 by bugdroid1@chromium.org, Sep 27 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7f85c5535b180c2042af8d41cbef3cdf16e88e4f

commit 7f85c5535b180c2042af8d41cbef3cdf16e88e4f
Author: Klemen Kozjek <klemenko@google.com>
Date: Wed Sep 27 08:21:02 2017

Zip Archiver: Check if |location| is undefined

Sometimes when we click 'Cancel' in the password dialog for a password
protected zip file, |location| in canExecute of rename command can be
undefined. As a result, it checks if |location| is undefined before
accessing its properties.

Bug:  607078 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I9cac007ecc31d35ea94b385641c56a6fe415cf14
Reviewed-on: https://chromium-review.googlesource.com/686087
Commit-Queue: Klemen Kozjek <klemenko@google.com>
Reviewed-by: Yuki Awano <yawano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504600}
[modify] https://crrev.com/7f85c5535b180c2042af8d41cbef3cdf16e88e4f/ui/file_manager/file_manager/foreground/js/file_manager_commands.js

Project Member

Comment 24 by bugdroid1@chromium.org, Sep 27 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3a813a85bab20fa6c0fc5591fcea34598bd6a1e4

commit 3a813a85bab20fa6c0fc5591fcea34598bd6a1e4
Author: Klemen Kozjek <klemenko@google.com>
Date: Wed Sep 27 11:26:26 2017

Zip Archiver: Add translations

This CL removes translations in .json files and replaces them with .grdp.
It also exposes an API to load and retrieve translation in HTML views.

As a result, views do not have hardcoded strings anymore.

Bug:  607078 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I70a08b9c96dad6cca2f82ce5d5df13ce3fb89126
Reviewed-on: https://chromium-review.googlesource.com/666878
Commit-Queue: Klemen Kozjek <klemenko@google.com>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Reviewed-by: Yuki Awano <yawano@chromium.org>
Reviewed-by: Naoki Fukino <fukino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504627}
[modify] https://crrev.com/3a813a85bab20fa6c0fc5591fcea34598bd6a1e4/chrome/app/chromeos_strings.grdp
[modify] https://crrev.com/3a813a85bab20fa6c0fc5591fcea34598bd6a1e4/chrome/browser/chromeos/extensions/file_manager/private_api_strings.cc
[modify] https://crrev.com/3a813a85bab20fa6c0fc5591fcea34598bd6a1e4/chrome/browser/extensions/component_loader.cc
[modify] https://crrev.com/3a813a85bab20fa6c0fc5591fcea34598bd6a1e4/chrome/browser/extensions/component_loader.h
[modify] https://crrev.com/3a813a85bab20fa6c0fc5591fcea34598bd6a1e4/chrome/browser/resources/chromeos/zip_archiver/BUILD.gn
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/Makefile
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/Makefile.package
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/Makefile.pnacl
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/ar/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/bg/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/ca/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/cs/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/da/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/de/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/el/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/en/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/en_GB/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/es/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/es_419/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/et/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/fa/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/fi/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/fil/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/fr/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/he/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/hi/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/hr/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/hu/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/id/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/it/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/ja/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/ko/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/lt/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/lv/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/ms/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/nl/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/no/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/pl/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/pt_BR/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/pt_PT/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/ro/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/ru/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/sk/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/sl/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/sr/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/sv/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/th/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/tr/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/uk/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/vi/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/zh_CN/messages.json
[delete] https://crrev.com/b69f62c9bdc40e7ce1bf59e117ced99a08e6cb9d/chrome/browser/resources/chromeos/zip_archiver/_locales/zh_TW/messages.json
[modify] https://crrev.com/3a813a85bab20fa6c0fc5591fcea34598bd6a1e4/chrome/browser/resources/chromeos/zip_archiver/css/passphrase-dialog.css
[modify] https://crrev.com/3a813a85bab20fa6c0fc5591fcea34598bd6a1e4/chrome/browser/resources/chromeos/zip_archiver/html/passphrase-dialog.html
[modify] https://crrev.com/3a813a85bab20fa6c0fc5591fcea34598bd6a1e4/chrome/browser/resources/chromeos/zip_archiver/html/passphrase.html
[modify] https://crrev.com/3a813a85bab20fa6c0fc5591fcea34598bd6a1e4/chrome/browser/resources/chromeos/zip_archiver/js/app.js
[modify] https://crrev.com/3a813a85bab20fa6c0fc5591fcea34598bd6a1e4/chrome/browser/resources/chromeos/zip_archiver/js/background.js
[add] https://crrev.com/3a813a85bab20fa6c0fc5591fcea34598bd6a1e4/chrome/browser/resources/chromeos/zip_archiver/js/main.js
[modify] https://crrev.com/3a813a85bab20fa6c0fc5591fcea34598bd6a1e4/chrome/browser/resources/chromeos/zip_archiver/manifest.json

Status: Fixed (was: Started)
Hi,

change https://chromium-review.googlesource.com/c/chromium/src/+/662983 breaks non x86 builds.

x86 related flags are added for all arches:

if (!is_nacl) {
    cflags += [
        "-maes",
        "-msse3",
        "-msse4.1",
    ]
}

For MIPS build there are clang errors:

clang: error: argument unused during compilation: '-maes' [-Werror,-Wunused-command-line-argument]
clang: error: argument unused during compilation: '-msse3' [-Werror,-Wunused-command-line-argument]
clang: error: argument unused during compilation: '-msse4.1' [-Werror,-Wunused-command-line-argument]
Project Member

Comment 27 by bugdroid1@chromium.org, Oct 16 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/fb17a43dad3cb05697cc601a7a5f8b283b8a5c5e

commit fb17a43dad3cb05697cc601a7a5f8b283b8a5c5e
Author: Milko Leporis <milko.leporis@imgtec.com>
Date: Mon Oct 16 10:01:42 2017

Fix minizip build issue for non x64 platforms

Build issue was introduced by change
https://chromium-review.googlesource.com/c/chromium/src/+/662983

Bug:  607078 
Change-Id: Iffa609f09858affc53af09575ae98d706ee778fa
Reviewed-on: https://chromium-review.googlesource.com/718102
Reviewed-by: Yuki Awano <yawano@chromium.org>
Commit-Queue: Yuki Awano <yawano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#509007}
[modify] https://crrev.com/fb17a43dad3cb05697cc601a7a5f8b283b8a5c5e/third_party/minizip/BUILD.gn

Sign in to add a comment