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

Unified Diff: jingle/notifier/base/chrome_async_socket.h

Issue 5958001: The MediatorThread worker thread needs to have a CertVerifier... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Fix typos in xmpp_connection_unittest.cc Created 10 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: jingle/notifier/base/chrome_async_socket.h
===================================================================
--- jingle/notifier/base/chrome_async_socket.h (revision 69414)
+++ jingle/notifier/base/chrome_async_socket.h (working copy)
@@ -25,6 +25,7 @@
#include "talk/xmpp/asyncsocket.h"
namespace net {
+class CertVerifier;
class ClientSocket;
class ClientSocketFactory;
class IOBufferWithSize;
@@ -34,10 +35,11 @@
class ChromeAsyncSocket : public buzz::AsyncSocket {
public:
- // Takes ownership of |client_socket_factory| but not |net_log|.
- // |net_log| may be NULL.
+ // Takes ownership of |client_socket_factory| but not |cert_verifier| and
agl 2010/12/17 16:22:01 s/and/nor/
+ // |net_log|. |cert_verifier| may not be NULL. |net_log| may be NULL.
ChromeAsyncSocket(net::ClientSocketFactory* client_socket_factory,
const net::SSLConfig& ssl_config,
+ net::CertVerifier* cert_verifier,
size_t read_buf_size,
size_t write_buf_size,
net::NetLog* net_log);
@@ -187,6 +189,7 @@
scoped_ptr<net::ClientSocketFactory> client_socket_factory_;
const net::SSLConfig ssl_config_;
+ net::CertVerifier* cert_verifier_;
agl 2010/12/17 16:22:01 I think this can be a const pointer if you like th
net::BoundNetLog bound_net_log_;
// buzz::AsyncSocket state.

Powered by Google App Engine
This is Rietveld 408576698