Revert of Make the simple QUIC client able to resolve hosts. (patchset #7 id:120001 of https://codereview.chromium.org/1004173004/)

Reason for revert:
Build failure: http://build.chromium.org/p/chromium.win/builders/Win%20x64%20Builder%20%28dbg%29/builds/7147/steps/compile/logs/stdio

Original issue's description:
> Make the simple QUIC client able to resolve hosts.
> Implement a SynchronousHostResolver for this purpose.
>
> Committed: https://crrev.com/acfb2f91085d5291ba1c9a04dbe7aea610b13e33
> Cr-Commit-Position: refs/heads/master@{#321688}

[email protected],[email protected]
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

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

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

Author: pavelfeldman

net/BUILD.gn

@@ -1195,7 +1195,6 @@ if (is_linux) {
       ":epoll_server",
       ":quic_base",
       ":net",
-      ":simple_quic_tools",
       "//base",
       "//third_party/boringssl",
     ]
@@ -1273,8 +1272,6 @@ source_set("simple_quic_tools") {
     "tools/quic/quic_simple_client_session.h",
     "tools/quic/quic_simple_client_stream.cc",
     "tools/quic/quic_simple_client_stream.h",
-    "tools/quic/synchronous_host_resolver.cc",
-    "tools/quic/synchronous_host_resolver.h",
   ]
   deps = [
     ":net",

net/net.gyp

@@ -817,8 +817,6 @@
         'tools/quic/quic_simple_client_session.h',
         'tools/quic/quic_simple_client_stream.cc',
         'tools/quic/quic_simple_client_stream.h',
-        'tools/quic/synchronous_host_resolver.cc',
-        'tools/quic/synchronous_host_resolver.h',
       ],
     },
   ],
@@ -1275,7 +1273,6 @@
             '../base/base.gyp:base',
             'net',
             'quic_base',
-            'simple_quic_tools',
           ],
           'sources': [
             'tools/quic/quic_client_bin.cc',

net/tools/quic/quic_client_bin.cc

@@ -47,8 +47,6 @@
 #include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
 #include "net/base/ip_endpoint.h"
-#include "net/base/net_errors.h"
-#include "net/base/net_log.h"
 #include "net/base/privacy_mode.h"
 #include "net/cert/cert_verifier.h"
 #include "net/http/transport_security_state.h"
@@ -59,7 +57,6 @@
 #include "net/tools/epoll_server/epoll_server.h"
 #include "net/tools/quic/quic_client.h"
 #include "net/tools/quic/spdy_utils.h"
-#include "net/tools/quic/synchronous_host_resolver.h"
 #include "url/gurl.h"
 
 using base::StringPiece;
@@ -172,18 +169,14 @@ int main(int argc, char *argv[]) {
   // protocol is required in the URL.
   GURL url(urls[0]);
   string host = FLAGS_host;
+  // TODO(rtenneti): get ip_addr from hostname by doing host resolution.
   if (host.empty()) {
-    host = url.host();
+    LOG(ERROR) << "--host must be specified\n";
+    return 1;
   }
   if (!net::ParseIPLiteralToNumber(host, &ip_addr)) {
-    net::AddressList addresses;
-    int rv = net::tools::SynchronousHostResolver::Resolve(host, &addresses);
-    if (rv != net::OK) {
-      LOG(ERROR) << "Unable to resolve '" << host << "' : "
-                 << net::ErrorToString(rv);
-      return 1;
-    }
-    ip_addr = addresses[0].address();
+    LOG(ERROR) << "--host could not be parsed as an IP address\n";
+    return 1;
   }
 
   string host_port = net::IPAddressToStringWithPort(ip_addr, FLAGS_port);

net/tools/quic/quic_simple_client_bin.cc

@@ -47,8 +47,6 @@
 #include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
 #include "net/base/ip_endpoint.h"
-#include "net/base/net_errors.h"
-#include "net/base/net_log.h"
 #include "net/base/privacy_mode.h"
 #include "net/cert/cert_verifier.h"
 #include "net/http/http_request_info.h"
@@ -59,7 +57,6 @@
 #include "net/quic/quic_utils.h"
 #include "net/spdy/spdy_http_utils.h"
 #include "net/tools/quic/quic_simple_client.h"
-#include "net/tools/quic/synchronous_host_resolver.h"
 #include "url/gurl.h"
 
 using base::StringPiece;
@@ -173,18 +170,14 @@ int main(int argc, char *argv[]) {
   // protocol is required in the URL.
   GURL url(urls[0]);
   string host = FLAGS_host;
+  // TODO(rtenneti): get ip_addr from hostname by doing host resolution.
   if (host.empty()) {
-    host = url.host();
+    LOG(ERROR) << "--host must be specified\n";
+    return 1;
   }
   if (!net::ParseIPLiteralToNumber(host, &ip_addr)) {
-    net::AddressList addresses;
-    int rv = net::tools::SynchronousHostResolver::Resolve(host, &addresses);
-    if (rv != net::OK) {
-      LOG(ERROR) << "Unable to resolve '" << host << "' : "
-                 << net::ErrorToString(rv);
-      return 1;
-    }
-    ip_addr = addresses[0].address();
+    LOG(ERROR) << "--host could not be parsed as an IP address\n";
+    return 1;
   }
 
   string host_port = net::IPAddressToStringWithPort(ip_addr, FLAGS_port);