When talking to a SOCKS v5 proxy, default to sending addresses as raw domains...

net/socket_stream/socket_stream.cc

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 //
 // TODO(ukai): code is similar with http_network_transaction.cc.  We should
 //   think about ways to share code, if possible.
 
 #include "net/socket_stream/socket_stream.h"
 
 #include <string>
@@ skipping 681 matching lines @@
 int SocketStream::DoSOCKSConnect() {
   DCHECK_EQ(kSOCKSProxy, proxy_mode_);
 
   next_state_ = STATE_SOCKS_CONNECT_COMPLETE;
 
   ClientSocket* s = socket_.release();
   HostResolver::RequestInfo req_info(url_.HostNoBrackets(),
                                      url_.EffectiveIntPort());
 
   if (proxy_info_.proxy_server().scheme() == ProxyServer::SCHEME_SOCKS5)
-    s = new SOCKS5ClientSocket(s, req_info, host_resolver_.get());
+    s = new SOCKS5ClientSocket(s, req_info, NULL /*use proxy-side resolving*/);

[wtc, 2009/12/17 23:23:04]

Maybe you can just say "proxy-side resolving" to deal with
the 80-char limit.

   else
     s = new SOCKSClientSocket(s, req_info, host_resolver_.get());
   socket_.reset(s);
   metrics_->OnSOCKSProxy();
   return socket_->Connect(&io_callback_, load_log_);
 }
 
 int SocketStream::DoSOCKSConnectComplete(int result) {
   DCHECK_EQ(kSOCKSProxy, proxy_mode_);
 
@@ skipping 184 matching lines @@
   return context_->proxy_service();
 }
 
 void SocketStream::GetInfoForTracker(
     RequestTracker<SocketStream>::RecentRequestInfo* info) const {
   info->original_url = url_;
   info->load_log = load_log_;
 }
 
 }  // namespace net