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

Issue 762173003: policy_templates.json: Document full removal of SSLv3 after M43. (Closed)

Created:
6 years ago by Thiemo Nagel
Modified:
6 years ago
Reviewers:
bartfab (slow)
CC:
chromium-reviews, joaodasilva+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

policy_templates.json: Document full removal of SSLv3 after M43. BUG=436391 Committed: https://crrev.com/1481769622de486860fd291a72ddcad46b0a25e9 Cr-Commit-Position: refs/heads/master@{#306100}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -11 lines) Patch
M components/policy/resources/policy_templates.json View 5 chunks +15 lines, -11 lines 0 comments Download

Messages

Total messages: 8 (2 generated)
Thiemo Nagel
Hi Bartosz, may I ask for a quick review? To quote Saswat: "Every passing day ...
6 years ago (2014-11-28 16:34:10 UTC) #2
bartfab (slow)
LGTM In hindsight, "ssl3" should have been called something like "ssl3_effective_until_chrome_42", analogously to the |ShowModalDialog_EffectiveUntil20150430| ...
6 years ago (2014-11-28 16:50:11 UTC) #3
Thiemo Nagel
I agree. Thank you very much!
6 years ago (2014-11-28 17:06:41 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/762173003/1
6 years ago (2014-11-28 17:07:18 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
6 years ago (2014-11-28 17:51:52 UTC) #7
commit-bot: I haz the power
6 years ago (2014-11-28 17:52:42 UTC) #8
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/1481769622de486860fd291a72ddcad46b0a25e9
Cr-Commit-Position: refs/heads/master@{#306100}

Powered by Google App Engine
This is Rietveld 408576698