Allow chrome to handle 407 auth challenges to CONNECT requests

jingle/notifier/base/proxy_resolving_client_socket.cc

Index: jingle/notifier/base/proxy_resolving_client_socket.cc
===================================================================
--- jingle/notifier/base/proxy_resolving_client_socket.cc	(revision 117986)
+++ jingle/notifier/base/proxy_resolving_client_socket.cc	(working copy)
@@ -172,7 +172,10 @@
   // Now that we have resolved the proxy, we need to connect.
   status = net::InitSocketHandleForRawConnect(
       dest_host_port_pair_, network_session_.get(), proxy_info_, ssl_config_,
-      ssl_config_, bound_net_log_, transport_.get(), connect_callback_);
+      ssl_config_, bound_net_log_, transport_.get(), connect_callback_,
+      base::Bind(&ProxyResolvingClientSocket::OnNeedsProxyTunnelAuthCallback,
+                 weak_factory_.GetWeakPtr()));
+
   if (status != net::ERR_IO_PENDING) {
     // Since this method is always called asynchronously. it is OK to call
     // ProcessConnectDone synchronously.
@@ -180,6 +183,15 @@
   }
 }
 
+void ProxyResolvingClientSocket::OnNeedsProxyTunnelAuthCallback(

[akalin, 2012/01/19 22:12:48]

actually, since this function doesn't do much, it can be a free function in an
anonymous namespace in this .cc file.  Then you can revert the change to the .h
file, and you wouldn't need the weak ptr at all.

[Ryan Hamilton, 2012/01/19 23:11:19]

Done.  (Love it!)

+    const net::HttpResponseInfo& response_info,
+    net::HttpAuthController* auth_controller,
+    net::CompletionCallback callback) {
+  // Since have not way to respond, simply invoke the callback and the
+  // request will fail.
+  callback.Run(net::OK);
+}
+
 void ProxyResolvingClientSocket::ProcessConnectDone(int status) {
   if (status != net::OK) {
     // If the connection fails, try another proxy.