Fix chrome.socket.tcp API to use all addresses received from DNS.

Previously chrome.socket.tcp API was using only one IP address it
receives from DNS. Fixed it to pass all addresses to TCPClientSocket so
it attempts to connect to all of them before failing.

BUG=508210

Review URL: https://codereview.chromium.org/1229763002

Cr-Commit-Position: refs/heads/master@{#337962}

1 files changed, 2 insertions, 5 deletions

diff --git a/extensions/browser/api/socket/socket_api.h b/extensions/browser/api/socket/socket_api.h
index d6e4736..9f4fda7 100644
--- a/extensions/browser/api/socket/socket_api.h
+++ b/extensions/browser/api/socket/socket_api.h
@@ -147,19 +147,16 @@ class SocketExtensionWithDnsLookupFunction : public SocketAsyncApiFunction {
   // AsyncApiFunction:
   bool PrePrepare() override;
 
-  void StartDnsLookup(const std::string& hostname);
+  void StartDnsLookup(const net::HostPortPair& host_port_pair);
   virtual void AfterDnsLookup(int lookup_result) = 0;
 
-  std::string resolved_address_;
+  net::AddressList addresses_;
 
  private:
   void OnDnsLookup(int resolve_result);
 
   // Weak pointer to the resource context.
   content::ResourceContext* resource_context_;
-
-  scoped_ptr<net::HostResolver::RequestHandle> request_handle_;
-  scoped_ptr<net::AddressList> addresses_;
 };
 
 class SocketCreateFunction : public SocketAsyncApiFunction {