Don't use same value for socket() and handle(), as they are different concepts.

runtime/bin/eventhandler_win.cc

Index: runtime/bin/eventhandler_win.cc
diff --git a/runtime/bin/eventhandler_win.cc b/runtime/bin/eventhandler_win.cc
index 934c7ca96cdf1cff572e2455e7e8bdcbf0e93f8d..26260a404eff73099b259a0521696b921eeebd62 100644
--- a/runtime/bin/eventhandler_win.cc
+++ b/runtime/bin/eventhandler_win.cc
@@ -432,6 +432,18 @@ void DirectoryWatchHandle::Stop() {
 }
 
 
+bool SocketHandle::CreateCompletionPort(HANDLE completion_port) {
+  completion_port_ = CreateIoCompletionPort(reinterpret_cast<HANDLE>(socket()),

[kasperl, 2014/06/04 14:01:05]

Would it make sense to have a non-virtual DoCreateCompletionPort that takes two
handles (the completion port and the socket/handle) so you can share the code?

[Anders Johnsen, 2014/06/04 14:05:22]

Done.

+                                            completion_port,
+                                            reinterpret_cast<ULONG_PTR>(this),
+                                            0);
+  if (completion_port_ == NULL) {
+    return false;
+  }
+  return true;
+}
+
+
 void SocketHandle::HandleIssueError() {
   int error = WSAGetLastError();
   if (error == WSAECONNRESET) {