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

Issue 775217 link

Starred by 1 user
Status: Fixed
Owner:
waff...@chromium.org
Closed: Oct 2017
Cc:
sorin@chromium.org
█ mab@chromium.org
agl@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Remove CRLSet Component from iOS

Project Member Reported by waff...@chromium.org, Oct 16 2017 
Remove CRLSet Component from iOS

We don't ship it on Android. (It takes disk space, memory, and bandwidth, so agl@ switched it off.)
We think iOS and Android should be aligned.
On iOS, we use the system's networking stack for browsing anyway, thus the CRLSets are less useful.

(In the future, CRLSets might carry a complete set of intermediate revocations and be part of a consistent revocation story with OCSP Must Staple. At that point, maybe we reconsider our stance on mobile.)
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 24 2017 

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

commit ddf9723e3f0b2d7ac4a957cde07b5e972f970dd9
Author: Joshua Pawlicki <waffles@google.com>
Date: Tue Oct 24 15:24:23 2017

No longer install CRLSet on ios.

Bug:  775217 
Change-Id: Ibd6bb27bed7ba209e611a4d327b40b358839eed9
Reviewed-on: https://chromium-review.googlesource.com/722021
Reviewed-by: Ryan Sleevi <rsleevi@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Sorin Jianu <sorin@chromium.org>
Commit-Queue: Joshua Pawlicki <waffles@chromium.org>
Cr-Commit-Position: refs/heads/master@{#511153}
[modify] https://crrev.com/ddf9723e3f0b2d7ac4a957cde07b5e972f970dd9/chrome/browser/BUILD.gn
[modify] https://crrev.com/ddf9723e3f0b2d7ac4a957cde07b5e972f970dd9/chrome/browser/chrome_browser_main.cc
[modify] https://crrev.com/ddf9723e3f0b2d7ac4a957cde07b5e972f970dd9/chrome/browser/chromeos/BUILD.gn
[modify] https://crrev.com/ddf9723e3f0b2d7ac4a957cde07b5e972f970dd9/chrome/browser/chromeos/login/session/user_session_manager.cc
[rename] https://crrev.com/ddf9723e3f0b2d7ac4a957cde07b5e972f970dd9/chrome/browser/component_updater/crl_set_component_installer.cc
[add] https://crrev.com/ddf9723e3f0b2d7ac4a957cde07b5e972f970dd9/chrome/browser/component_updater/crl_set_component_installer.h
[modify] https://crrev.com/ddf9723e3f0b2d7ac4a957cde07b5e972f970dd9/components/component_updater/BUILD.gn
[modify] https://crrev.com/ddf9723e3f0b2d7ac4a957cde07b5e972f970dd9/components/component_updater/DEPS
[delete] https://crrev.com/7d0633dde4a03e0cf3411a04554e73559c835095/components/component_updater/crl_set_component_installer.h
[add] https://crrev.com/ddf9723e3f0b2d7ac4a957cde07b5e972f970dd9/components/component_updater/crl_set_remover.cc
[add] https://crrev.com/ddf9723e3f0b2d7ac4a957cde07b5e972f970dd9/components/component_updater/crl_set_remover.h
[modify] https://crrev.com/ddf9723e3f0b2d7ac4a957cde07b5e972f970dd9/ios/chrome/app/BUILD.gn
[modify] https://crrev.com/ddf9723e3f0b2d7ac4a957cde07b5e972f970dd9/ios/chrome/app/main_controller.mm

Comment 2 by waff...@chromium.org, Oct 24 2017

Status: Fixed (was: Started)
►

Sign in to add a comment