Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(671)

Issue 795013003: Certificate Transparency: Remove SSLWatcher's Alpha log. (Closed)

Created:
6 years ago by Eran Messeri
Modified:
6 years ago
Reviewers:
Ryan Sleevi
CC:
chromium-reviews, cbentzel+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Certificate Transparency: Remove SSLWatcher's Alpha log. Per Ryan's recent announcement, SSLWatcher's Alpha log is no longer recognized by Chrome, so remove it. This change also removes the assumption (made in the CertPolicyEnforcer) that in case of having only 2 logs recognized by Chrome, only 2 will be required where the policy currently says 3 would be necessary. BUG=441337 Committed: https://crrev.com/bce7c1b31eabc1ce8b17a0b7d21549049e53e24e Cr-Commit-Position: refs/heads/master@{#308437}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -36 lines) Patch
M chrome/browser/io_thread.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M net/cert/cert_policy_enforcer.h View 2 chunks +1 line, -4 lines 0 comments Download
M net/cert/cert_policy_enforcer.cc View 2 chunks +3 lines, -6 lines 0 comments Download
M net/cert/cert_policy_enforcer_unittest.cc View 3 chunks +2 lines, -15 lines 0 comments Download
M net/cert/ct_known_logs_static.h View 1 chunk +1 line, -9 lines 0 comments Download

Messages

Total messages: 7 (2 generated)
Eran Messeri
6 years ago (2014-12-15 11:22:12 UTC) #2
Ryan Sleevi
lgtm
6 years ago (2014-12-15 21:15:00 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/795013003/1
6 years ago (2014-12-15 21:20:48 UTC) #5
commit-bot: I haz the power
Committed patchset #1 (id:1)
6 years ago (2014-12-15 22:17:28 UTC) #6
commit-bot: I haz the power
6 years ago (2014-12-15 22:19:03 UTC) #7
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/bce7c1b31eabc1ce8b17a0b7d21549049e53e24e
Cr-Commit-Position: refs/heads/master@{#308437}

Powered by Google App Engine
This is Rietveld 408576698