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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: ipc/ipc_channel_win.cc

Issue 1954543002: Remove handler from MessagePumpForIO::IOContext. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@remove_observers
Patch Set: rebase Created 4 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
« base/message_loop/message_pump_win.cc ('K') | « device/serial/serial_io_handler_win.cc ('k') | mojo/edk/system/channel_win.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ipc/ipc_channel_win.cc
diff --git a/ipc/ipc_channel_win.cc b/ipc/ipc_channel_win.cc
index 404e688a212367303b3d9ccee8eba0a4b1da0957..03c403bf5ce0675ff99bf794a6171f8735a7c2a2 100644
--- a/ipc/ipc_channel_win.cc
+++ b/ipc/ipc_channel_win.cc
@@ -29,8 +29,6 @@
namespace IPC {
ChannelWin::State::State(ChannelWin* channel) : is_pending(false) {
Lei Zhang 2016/05/09 22:41:30 Do we still need the |channel| param?
fdoray 2016/05/10 16:33:17 Done.
- memset(&context.overlapped, 0, sizeof(context.overlapped));
- context.handler = channel;
}
ChannelWin::State::~State() {
« base/message_loop/message_pump_win.cc ('K') | « device/serial/serial_io_handler_win.cc ('k') | mojo/edk/system/channel_win.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698