DNS: Per-network-type and Finch-variable timeouts

net/tools/gdig/gdig.cc

Index: net/tools/gdig/gdig.cc
diff --git a/net/tools/gdig/gdig.cc b/net/tools/gdig/gdig.cc
index e0af40fbff9920a2d43add5bceeda86d3bf2c085..e0a6fa1c33a94be0b4916052c196aa345b9df5ce 100644
--- a/net/tools/gdig/gdig.cc
+++ b/net/tools/gdig/gdig.cc
@@ -77,7 +77,7 @@ std::string DnsConfigToString(const DnsConfig& dns_config) {
 
   base::StringAppendF(&output, "options ndots:%d\n", dns_config.ndots);
   base::StringAppendF(&output, "options timeout:%d\n",
-                      static_cast<int>(dns_config.timeout.InMilliseconds()));
+                      static_cast<int>(dns_config.timeout.InSeconds()));

[Randy Smith (Not in Mondays), 2016/03/15 20:12:30]

Why?

[Deprecated (see juliatuttle), 2016/03/16 16:45:30]

This was wrong; see the man page for resolv.conf. timeout is in seconds, not
milliseconds.

[Randy Smith (Not in Mondays), 2016/03/16 20:11:07]

Separate CL or mention in CL description?

   base::StringAppendF(&output, "options attempts:%d\n", dns_config.attempts);
   if (dns_config.rotate)
     output.append("options rotate\n");