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

Issue 558008: Use the USE_NSS macro to help porting to FreeBSD, OpenBSD, etc.... (Closed)

Created:
10 years, 10 months ago by wtc
Modified:
9 years, 7 months ago
CC:
chromium-reviews, brettw+cc_chromium.org, darin (slow to review)
Visibility:
Public.

Description

Use the USE_NSS macro to help porting to FreeBSD, OpenBSD, etc. Fix cpplint nits. R=evan,pvalchev BUG=none TEST=No compilation errors. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=37358

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -15 lines) Patch
M base/test/test_suite.h View 1 chunk +2 lines, -2 lines 0 comments Download
M build/build_config.h View 1 chunk +3 lines, -2 lines 0 comments Download
M net/http/des.cc View 3 chunks +3 lines, -3 lines 0 comments Download
M net/socket/ssl_test_util.cc View 6 chunks +8 lines, -8 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
wtc
10 years, 10 months ago (2010-01-27 23:23:29 UTC) #1
Evan Martin
LGTM (why are you doing this?)
10 years, 10 months ago (2010-01-27 23:40:13 UTC) #2
Peter Valchev
LGTM you got me curious too :)
10 years, 10 months ago (2010-01-27 23:50:08 UTC) #3
wtc
10 years, 10 months ago (2010-01-28 00:02:20 UTC) #4
On 2010/01/27 23:40:13, Evan Martin wrote:
>
> (why are you doing this?)

Because I love the BSDs!

OK, it's part of my work on porting SSLClientSocketNSS to Windows
(http://crbug.com/28744).  I need to make sure EnsureNSPRInit
and EnsureNSSInit are called on the right platforms.

Powered by Google App Engine
This is Rietveld 408576698