make SetReceiveBufferSize and SetSendBufferSize return net error codes (instead of bools)

chrome/browser/devtools/adb/android_usb_socket.cc

Index: chrome/browser/devtools/adb/android_usb_socket.cc
diff --git a/chrome/browser/devtools/adb/android_usb_socket.cc b/chrome/browser/devtools/adb/android_usb_socket.cc
index ee9c9df5674dda237ce4fc83fc653d7859715c20..16cc69bee3e15d73f66ca6b7fdb39749a09df988 100644
--- a/chrome/browser/devtools/adb/android_usb_socket.cc
+++ b/chrome/browser/devtools/adb/android_usb_socket.cc
@@ -137,14 +137,14 @@ int AndroidUsbSocket::Write(net::IOBuffer* buffer,
   return net::ERR_IO_PENDING;
 }
 
-bool AndroidUsbSocket::SetReceiveBufferSize(int32 size) {
+int AndroidUsbSocket::SetReceiveBufferSize(int32 size) {
   NOTIMPLEMENTED();
-  return false;
+  return net::ERR_NOT_IMPLEMENTED;
 }
 
-bool AndroidUsbSocket::SetSendBufferSize(int32 size) {
+int AndroidUsbSocket::SetSendBufferSize(int32 size) {
   NOTIMPLEMENTED();
-  return false;
+  return net::ERR_NOT_IMPLEMENTED;
 }
 
 int AndroidUsbSocket::Connect(const net::CompletionCallback& callback) {
@@ -180,7 +180,7 @@ int AndroidUsbSocket::GetPeerAddress(net::IPEndPoint* address) const {
 
 int AndroidUsbSocket::GetLocalAddress(net::IPEndPoint* address) const {
   NOTIMPLEMENTED();
-  return net::ERR_FAILED;
+  return net::ERR_NOT_IMPLEMENTED;

[Ryan Hamilton, 2014/04/08 19:55:11]

One more change in semantics here. Again, almost assuredly not important.

[jar (doing other things), 2014/04/08 23:16:26]

I'm going to hold off (not do) this change unless my next landing fails.  This
shouldn't be able to hurt a debug build since the NOTIMPLEMENTED() protects the
code.

 }
 
 const net::BoundNetLog& AndroidUsbSocket::NetLog() const {