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

Issue 20598007: NSS: rename ALPN extension to have _xtn suffix. (Closed)

Created:
7 years, 4 months ago by agl
Modified:
7 years, 4 months ago
Reviewers:
wtc
CC:
chromium-reviews, cbentzel+watch_chromium.org, Ryan Sleevi
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : Update wtc's land that landed in the meantime. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+22 lines, -22 lines) Patch
M net/socket/ssl_client_socket_nss.cc View 1 1 chunk +2 lines, -2 lines 0 comments Download
M net/third_party/nss/patches/alpn.patch View 6 chunks +6 lines, -6 lines 0 comments Download
M net/third_party/nss/ssl/ssl3ext.c View 5 chunks +13 lines, -13 lines 0 comments Download
M net/third_party/nss/ssl/sslt.h View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 5 (0 generated)
agl
7 years, 4 months ago (2013-08-01 20:14:00 UTC) #1
wtc
Patch set 1 LGTM. Note that my CL https://codereview.chromium.org/20799005/ references the old ALPN extension enum ...
7 years, 4 months ago (2013-08-01 22:31:10 UTC) #2
wtc
agl: my CL https://codereview.chromium.org/20799005/ has landed. Could you also change my code to check for ...
7 years, 4 months ago (2013-08-02 15:10:47 UTC) #3
agl
Committed patchset #2 manually as r215290 (presubmit successful).
7 years, 4 months ago (2013-08-02 15:35:38 UTC) #4
agl
7 years, 4 months ago (2013-08-02 15:35:45 UTC) #5
Message was sent while issue was closed.
Done.

Powered by Google App Engine
This is Rietveld 408576698