Allow missing socketType in SocketInfo if socket is not connected

chrome/common/extensions/api/socket.idl

Index: chrome/common/extensions/api/socket.idl
diff --git a/chrome/common/extensions/api/socket.idl b/chrome/common/extensions/api/socket.idl
index 4bc60680e971f66962846c70791350475f193b8d..43c073275d6d305d559ac96038a6aed10a701a65 100644
--- a/chrome/common/extensions/api/socket.idl
+++ b/chrome/common/extensions/api/socket.idl
@@ -110,7 +110,7 @@ namespace socket {
 
   callback SetNoDelayCallback = void (boolean result);
 
-  callback GetInfoCallback = void (SocketInfo result);
+  callback GetInfoCallback = void (optional SocketInfo result);

[asargent_no_longer_on_chrome, 2013/05/29 21:31:07]

You can actually leave this as non-optional, since for callbacks we guarantee
the arguments will be sent to the callback as specified *unless* there is an
error. So for example in the chrome.tabs.get method I mentioned before, the
"Tab" argument to the callback is not optional but is omitted when an error
occurs. 

We could probably do a better job of documenting our error handling strategy,
because this is how most of our APIs work.

[Bei Zhang, 2013/05/29 22:14:51]

Done.

 
   callback GetNetworkCallback = void (NetworkInterface[] result);