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

Unified Diff: webrtc/base/win32socketserver.h

Issue 2828223002: Delete method MessageQueue::set_socketserver (Closed)
Patch Set: Use AutoSocketServerThread at one more place, missed in prev patchset. Created 3 years, 8 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: webrtc/base/win32socketserver.h
diff --git a/webrtc/base/win32socketserver.h b/webrtc/base/win32socketserver.h
index a9a53c33c15bfb6d4d589be1e1d7a3a8af4b1d0a..cec68ef3fd961f872bb1c48a6cb188cd1b47ef37 100644
--- a/webrtc/base/win32socketserver.h
+++ b/webrtc/base/win32socketserver.h
@@ -93,7 +93,7 @@ class Win32Socket : public AsyncSocket {
class Win32SocketServer : public SocketServer {
public:
- explicit Win32SocketServer(MessageQueue* message_queue);
+ Win32SocketServer();
virtual ~Win32SocketServer();
void set_modeless_dialog(HWND hdlg) {
@@ -138,12 +138,9 @@ class Win32SocketServer : public SocketServer {
class Win32Thread : public Thread {
public:
- Win32Thread() : ss_(this), id_(0) {
- set_socketserver(&ss_);
- }
+ explicit Win32Thread(SocketServer* ss) : Thread(ss), id_(0) {}
virtual ~Win32Thread() {
Stop();
- set_socketserver(nullptr);
}
virtual void Run() {
id_ = GetCurrentThreadId();
@@ -154,7 +151,6 @@ class Win32Thread : public Thread {
PostThreadMessage(id_, WM_QUIT, 0, 0);
}
private:
- Win32SocketServer ss_;
DWORD id_;
};

Powered by Google App Engine
This is Rietveld 408576698