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

Issue 308773007: Merge 273573 "Always call talk_base::InitializeSSL() in chromoti..." (Closed)

Created:
6 years, 6 months ago by Sergey Ulanov
Modified:
6 years, 6 months ago
Reviewers:
Sergey Ulanov
CC:
chromium-reviews, chromoting-reviews_chromium.org
Visibility:
Public.

Description

Merge 273573 "Always call talk_base::InitializeSSL() in chromoti..." > Always call talk_base::InitializeSSL() in chromoting plugin. > > crrev.com/272789 added talk_base::InitializeSSL() under USE_NSS ifdef, > but it didn't work because USE_NSS is not defined on Windows and Mac > even when NSS is actually used (duh!). (USE_NSS is used for X509 certs > and Windows and Mac don't use NSS for X509 certs). > > BUG=364689 > TBR=jamiewalch@chromium.org > > Review URL: https://codereview.chromium.org/307813003 TBR=sergeyu@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=274100

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -2 lines) Patch
M remoting/client/plugin/chromoting_instance.cc View 1 chunk +5 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Sergey Ulanov
6 years, 6 months ago (2014-06-01 00:48:35 UTC) #1
Sergey Ulanov
6 years, 6 months ago (2014-06-01 00:49:06 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r274100 (tree was closed).

Powered by Google App Engine
This is Rietveld 408576698