Implement SyncSocket for untrusted NaCl builds.

base/sync_socket_nacl.cc

Index: base/sync_socket_nacl.cc
===================================================================
--- base/sync_socket_nacl.cc	(revision 146234)
+++ base/sync_socket_nacl.cc	(working copy)
@@ -18,7 +18,6 @@
 const SyncSocket::Handle SyncSocket::kInvalidHandle = -1;
 
 SyncSocket::SyncSocket() : handle_(kInvalidHandle) {
-  NOTREACHED();
 }
 
 SyncSocket::~SyncSocket() {
@@ -30,28 +29,34 @@
 }
 
 bool SyncSocket::Close() {
- return false;
+  if (handle_ != -1) {

[brettw, 2012/07/13 23:12:55]

Can you use kInvalidHandle instead of -1 in this function?

[bbudge, 2012/07/13 23:44:05]

Done.

+    close(handle_);

[Mark Seaborn, 2012/07/13 23:18:32]

Please check the return value of close()

[bbudge, 2012/07/13 23:44:05]

Done.

+    handle_ = -1;
+  }
+  return true;
 }
 
 size_t SyncSocket::Send(const void* buffer, size_t length) {
-  return 0;
+  return -1;

[brettw, 2012/07/13 23:12:55]

Can you comment why this isn't implemented?

[bbudge, 2012/07/13 23:44:05]

Done.

 }
 
 size_t SyncSocket::Receive(void* buffer, size_t length) {
-  return 0;
+  return read(handle_, buffer, length);
 }
 
 size_t SyncSocket::Peek() {
-  return 0;
+  return -1;
 }
 
-CancelableSyncSocket::CancelableSyncSocket() {}
+CancelableSyncSocket::CancelableSyncSocket() {
+}
+
 CancelableSyncSocket::CancelableSyncSocket(Handle handle)
     : SyncSocket(handle) {
 }
 
 size_t CancelableSyncSocket::Send(const void* buffer, size_t length) {
-  return 0;
+  return -1;
 }
 
 bool CancelableSyncSocket::Shutdown() {