New issue
Advanced search Search tips

Issue 821761 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 4
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Task



Sign in to add a comment

Turn down ZIP unpacker from the component extension list

Project Member Reported by yamaguchi@chromium.org, Mar 14 2018

Issue description

After release of Zip Archiver, we should remove ZIP unpacker from the component extension list at some point.

When the extension is uninstalled, its storage is also removed.
So it will resolve the issue that similar issue with 803752 (force to forget saved passwords).
 
Labels: M-68
Relevant bug is:
Issue 690217 Enable Zip Unpacker for Drive in Files App - Unzip only
(actually this means Zip Archiver extension.)

Note that ZIP unpacker extension is only used for unzipping.
Theoretically it can be removed after Issue 690217 without waiting for Issue 783200 (packing by Zip Archiver).

Comment 2 by sashab@chromium.org, Mar 21 2018

Labels: CrOSFilesFeature-Zip
Labels: -M-68 M-69

Comment 4 by noel@chromium.org, May 10 2018

Status: Assigned (was: Available)
Blockedon: 855197
Labels: -M-69 M-71
Owner: amistry@chromium.org
Blockedon: -855197
Project Member

Comment 9 by bugdroid1@chromium.org, Oct 4

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

commit db5e010661c11f30fd20baa8ca36ac71302b5630
Author: Anand K. Mistry <amistry@chromium.org>
Date: Thu Oct 04 04:24:00 2018

Purge the "Zip Unpacker" component extension.

This leaves 2 mechanisms to view zip files: the old AVFS-based zip file
mounting, and the "Zip Archiver" component extension that is based on
minizip and uses FSP.

Zip archiver is the default, but doesn't yet support zip64 files. AVFS
does support zip64, which is why it remains an option.

BUG= 821761 

Change-Id: I185992a4ca9a4b21024a9898623668351ba6a464
Reviewed-on: https://chromium-review.googlesource.com/c/1242254
Commit-Queue: Anand Mistry <amistry@chromium.org>
Reviewed-by: Ben Wells <benwells@chromium.org>
Reviewed-by: Noel Gordon <noel@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Steven Holte <holte@chromium.org>
Reviewed-by: Tatsuhisa Yamaguchi <yamaguchi@chromium.org>
Reviewed-by: Stuart Langley <slangley@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596500}
[modify] https://crrev.com/db5e010661c11f30fd20baa8ca36ac71302b5630/chrome/browser/about_flags.cc
[modify] https://crrev.com/db5e010661c11f30fd20baa8ca36ac71302b5630/chrome/browser/chromeos/file_manager/file_browser_handlers.cc
[modify] https://crrev.com/db5e010661c11f30fd20baa8ca36ac71302b5630/chrome/browser/chromeos/file_manager/file_tasks.cc
[modify] https://crrev.com/db5e010661c11f30fd20baa8ca36ac71302b5630/chrome/browser/extensions/component_extensions_whitelist/whitelist.cc
[modify] https://crrev.com/db5e010661c11f30fd20baa8ca36ac71302b5630/chrome/browser/extensions/extension_service.cc
[modify] https://crrev.com/db5e010661c11f30fd20baa8ca36ac71302b5630/chrome/browser/extensions/extension_service.h
[modify] https://crrev.com/db5e010661c11f30fd20baa8ca36ac71302b5630/chrome/browser/extensions/extension_service_unittest.cc
[modify] https://crrev.com/db5e010661c11f30fd20baa8ca36ac71302b5630/chrome/browser/extensions/external_component_loader.cc
[modify] https://crrev.com/db5e010661c11f30fd20baa8ca36ac71302b5630/chrome/browser/flag_descriptions.cc
[modify] https://crrev.com/db5e010661c11f30fd20baa8ca36ac71302b5630/chrome/browser/flag_descriptions.h
[modify] https://crrev.com/db5e010661c11f30fd20baa8ca36ac71302b5630/chrome/common/extensions/api/_manifest_features.json
[modify] https://crrev.com/db5e010661c11f30fd20baa8ca36ac71302b5630/chrome/common/extensions/extension_constants.cc
[modify] https://crrev.com/db5e010661c11f30fd20baa8ca36ac71302b5630/chrome/common/extensions/extension_constants.h
[modify] https://crrev.com/db5e010661c11f30fd20baa8ca36ac71302b5630/chromeos/chromeos_switches.cc
[modify] https://crrev.com/db5e010661c11f30fd20baa8ca36ac71302b5630/chromeos/chromeos_switches.h
[modify] https://crrev.com/db5e010661c11f30fd20baa8ca36ac71302b5630/tools/metrics/histograms/histograms.xml
[modify] https://crrev.com/db5e010661c11f30fd20baa8ca36ac71302b5630/ui/file_manager/file_manager/foreground/js/file_tasks.js
[modify] https://crrev.com/db5e010661c11f30fd20baa8ca36ac71302b5630/ui/file_manager/file_manager/foreground/js/file_tasks_unittest.js
[modify] https://crrev.com/db5e010661c11f30fd20baa8ca36ac71302b5630/ui/file_manager/file_manager/foreground/js/navigation_list_model.js
[modify] https://crrev.com/db5e010661c11f30fd20baa8ca36ac71302b5630/ui/file_manager/file_manager/foreground/js/task_controller.js

Status: Fixed (was: Assigned)

Sign in to add a comment