Chromium Code Reviews
Help | Chromium Project | Sign in
(226)

Issue 651055: Make dynamic access to SSLSetSessionOption static (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 1 month ago by Mark Mentovai
Modified:
2 years, 11 months ago
Reviewers:
wtc, Jens Alfke
CC:
chromium-reviews_chromium.org
Visibility:
Public.

Description

Make dynamic access to SSLSetSessionOption static.

It's not often that you can make something both dynamic and static.

BUG=16831
TEST=none

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=39485

Patch Set 1 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Lint Patch
M net/socket/ssl_client_socket_mac.cc View 1 chunk +1 line, -1 line 0 comments 0 errors Download
Commit:

Messages

Total messages: 4
Mark Mentovai
4 years, 1 month ago #1
Jens Alfke
LTGM. There shouldn't be any threading issues here, right? Even in case of a race ...
4 years, 1 month ago #2
Mark Mentovai
snej@chromium.org wrote: > LTGM. There shouldn't be any threading issues here, right? Even in case ...
4 years, 1 month ago #3
wtc
4 years, 1 month ago #4
Re: thread safety: In Chrome, SSLClientSocket is always used
on the IO thread.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1275:d14800f88434