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

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

Issue 7014009: Pass net_log parameter properly for ProxyResolvingClientSocket (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 7 months 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
diff --git a/jingle/notifier/base/chrome_async_socket.h b/jingle/notifier/base/chrome_async_socket.h
index a8a97f5ce72634241e68da46fa2160d431a90803..5cf1ca65c1dcf6a7ff3fa97c332d624eedadc7cb 100644
--- a/jingle/notifier/base/chrome_async_socket.h
+++ b/jingle/notifier/base/chrome_async_socket.h
@@ -20,7 +20,6 @@
#include "base/task.h"
#include "net/base/completion_callback.h"
#include "net/base/net_errors.h"
-#include "net/base/net_log.h"
#include "talk/xmpp/asyncsocket.h"
namespace net {
@@ -34,12 +33,10 @@ class ResolvingClientSocketFactory;
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|.
ChromeAsyncSocket(ResolvingClientSocketFactory* client_socket_factory,
size_t read_buf_size,
- size_t write_buf_size,
- net::NetLog* net_log);
+ size_t write_buf_size);
// Does not raise any signals.
virtual ~ChromeAsyncSocket();
@@ -185,7 +182,6 @@ class ChromeAsyncSocket : public buzz::AsyncSocket {
net::CompletionCallbackImpl<ChromeAsyncSocket> ssl_connect_callback_;
scoped_ptr<ResolvingClientSocketFactory> client_socket_factory_;
- net::BoundNetLog bound_net_log_;
// buzz::AsyncSocket state.
buzz::AsyncSocket::State state_;
« no previous file with comments | « no previous file | jingle/notifier/base/chrome_async_socket.cc » ('j') | jingle/notifier/base/proxy_resolving_client_socket.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698