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

Unified Diff: net/socket/tcp_client_socket_win.cc

Issue 6658027: UDP sockets implementation for windows. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: - Created 9 years, 9 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: net/socket/tcp_client_socket_win.cc
diff --git a/net/socket/tcp_client_socket_win.cc b/net/socket/tcp_client_socket_win.cc
index 6027003529be88389a575607eefeca7c4232c1e3..7a2d4688cbabb85cbdd376ef14a846c902aca9c0 100644
--- a/net/socket/tcp_client_socket_win.cc
+++ b/net/socket/tcp_client_socket_win.cc
@@ -22,87 +22,12 @@
#include "net/base/network_change_notifier.h"
#include "net/base/sys_addrinfo.h"
#include "net/base/winsock_init.h"
+#include "net/base/winsock_util.h"
namespace net {
namespace {
-// Assert that the (manual-reset) event object is not signaled.
-void AssertEventNotSignaled(WSAEVENT hEvent) {
- DWORD wait_rv = WaitForSingleObject(hEvent, 0);
- if (wait_rv != WAIT_TIMEOUT) {
- DWORD err = ERROR_SUCCESS;
- if (wait_rv == WAIT_FAILED)
- err = GetLastError();
- CHECK(false); // Crash.
- // This LOG statement is unreachable since we have already crashed, but it
- // should prevent the compiler from optimizing away the |wait_rv| and
- // |err| variables so they appear nicely on the stack in crash dumps.
- VLOG(1) << "wait_rv=" << wait_rv << ", err=" << err;
- }
-}
-
-// If the (manual-reset) event object is signaled, resets it and returns true.
-// Otherwise, does nothing and returns false. Called after a Winsock function
-// succeeds synchronously
-//
-// Our testing shows that except in rare cases (when running inside QEMU),
-// the event object is already signaled at this point, so we call this method
-// to avoid a context switch in common cases. This is just a performance
-// optimization. The code still works if this function simply returns false.
-bool ResetEventIfSignaled(WSAEVENT hEvent) {
- // TODO(wtc): Remove the CHECKs after enough testing.
- DWORD wait_rv = WaitForSingleObject(hEvent, 0);
- if (wait_rv == WAIT_TIMEOUT)
- return false; // The event object is not signaled.
- CHECK_EQ(WAIT_OBJECT_0, wait_rv);
- BOOL ok = WSAResetEvent(hEvent);
- CHECK(ok);
- return true;
-}
-
-//-----------------------------------------------------------------------------
-
-int MapWinsockError(int os_error) {
- // There are numerous Winsock error codes, but these are the ones we thus far
- // find interesting.
- switch (os_error) {
- case WSAEACCES:
- return ERR_ACCESS_DENIED;
- case WSAENETDOWN:
- return ERR_INTERNET_DISCONNECTED;
- case WSAETIMEDOUT:
- return ERR_TIMED_OUT;
- case WSAECONNRESET:
- case WSAENETRESET: // Related to keep-alive
- return ERR_CONNECTION_RESET;
- case WSAECONNABORTED:
- return ERR_CONNECTION_ABORTED;
- case WSAECONNREFUSED:
- return ERR_CONNECTION_REFUSED;
- case WSA_IO_INCOMPLETE:
- case WSAEDISCON:
- // WSAEDISCON is returned by WSARecv or WSARecvFrom for message-oriented
- // sockets (where a return value of zero means a zero-byte message) to
- // indicate graceful connection shutdown. We should not ever see this
- // error code for TCP sockets, which are byte stream oriented.
- LOG(DFATAL) << "Unexpected error " << os_error
- << " mapped to net::ERR_UNEXPECTED";
- return ERR_UNEXPECTED;
- case WSAEHOSTUNREACH:
- case WSAENETUNREACH:
- return ERR_ADDRESS_UNREACHABLE;
- case WSAEADDRNOTAVAIL:
- return ERR_ADDRESS_INVALID;
- case ERROR_SUCCESS:
- return OK;
- default:
- LOG(WARNING) << "Unknown error " << os_error
- << " mapped to net::ERR_FAILED";
- return ERR_FAILED;
- }
-}
-
int MapConnectError(int os_error) {
switch (os_error) {
// connect fails with WSAEACCES when Windows Firewall blocks the
« no previous file with comments | « net/net.gyp ('k') | net/udp/datagram_server_socket.h » ('j') | net/udp/udp_socket_win.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698