Fixup inputs and outputs in DnsConfigService

Updating everything to match the latest style guide preferences, mostly
using references much more liberally and converting output params to
returns wherever reasonable.

Also switching from copies to moves in a couple places that I happened
to notice, and a couple more random cleanups to make Tricium happy.

Change-Id: I999f511835e70e61ee8c0e7c5dd7e6b7f2396aea
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2626786
Reviewed-by: Matt Menke <[email protected]>
Commit-Queue: Eric Orth <[email protected]>
Cr-Commit-Position: refs/heads/master@{#844119}
diff --git a/net/dns/dns_config_watcher_mac.h b/net/dns/dns_config_watcher_mac.h
index 9151a52..86dbe5a 100644
--- a/net/dns/dns_config_watcher_mac.h
+++ b/net/dns/dns_config_watcher_mac.h
@@ -17,7 +17,7 @@
   bool Watch(const base::RepeatingCallback<void(bool succeeded)>& callback);
 
   // Returns false iff a valid config could not be determined.
-  static bool CheckDnsConfig(bool* out_unhandled_options);
+  static bool CheckDnsConfig(bool& out_unhandled_options);
 
  private:
   NotifyWatcherMac watcher_;