[net/dns] Watch configd notifications to detect DnsConfig changes on Mac.

net/dns/dns_config_service_posix.cc

Index: net/dns/dns_config_service_posix.cc
diff --git a/net/dns/dns_config_service_posix.cc b/net/dns/dns_config_service_posix.cc
index b3977fceca0ed6112ce0e391d2cbb8f7af2a007a..0fdd1fe0ed515687a0dcb3f3f658a8d297b9aa1f 100644
--- a/net/dns/dns_config_service_posix.cc
+++ b/net/dns/dns_config_service_posix.cc
@@ -16,6 +16,10 @@
 #include "net/dns/file_path_watcher_wrapper.h"
 #include "net/dns/serial_worker.h"
 
+#if defined(OS_MACOSX)
+#include "net/dns/notify_watcher_mac.h"
+#endif
+
 #ifndef _PATH_RESCONF  // Normally defined in <resolv.h>
 #define _PATH_RESCONF "/etc/resolv.conf"
 #endif
@@ -24,7 +28,6 @@ namespace net {
 
 namespace {
 
-const FilePath::CharType* kFilePathConfig = FILE_PATH_LITERAL(_PATH_RESCONF);
 const FilePath::CharType* kFilePathHosts = FILE_PATH_LITERAL("/etc/hosts");
 
 // A SerialWorker that uses libresolv to initialize res_state and converts
@@ -82,8 +85,30 @@ class ConfigReader : public SerialWorker {
 
 namespace internal {
 
+#if defined(OS_MACOSX)
+// Normally defined in dnsinfo.h in configd.

[Mark Mentovai, 2012/04/17 19:53:54]

You can do this:

extern "C" {

// From 10.7.3 configd-395.10/dnsinfo/dnsinfo.h
const char* dns_configuration_notify_key();

}  // extern "C"

Then you can call dns_configuration_notify_key() without hardcoding anything and
without <dnsinfo.h> being exposed. I like treating the Apple call as the
interface better than matching what they’ve hard-coded, even though both are
technically private.

[szym, 2012/04/17 20:14:31]

I wish this worked. Unfortunately, it won't link ("ld: symbol(s) not found").

[Mark Mentovai, 2012/04/17 20:22:20]

szym wrote:
I see, it’s marked “private_extern” in 10.5, which appears to be a bug based on
the AVAILABLE_MAC_OS_X_VERSION_10_4_AND_LATER in its declaration, but that’s
life.

+const char* kDnsNotifyKey =

[Mark Mentovai, 2012/04/17 19:53:54]

If you’re sticking with a const char*, make it static or put it in an anonymous
namespace. There’s no reason to expose it beyond this file (even if in a
namespace called “internal”.)

+    "com.apple.system.SystemConfiguration.dns_configuration";
+
+class DnsConfigServicePosix::ConfigWatcher : public NotifyWatcherMac {
+ public:
+  bool Watch(const base::Callback<void(bool succeeded)>& callback) {
+    return NotifyWatcherMac::Watch(kDnsNotifyKey, callback);
+  }
+};
+#else
+const FilePath::CharType* kFilePathConfig = FILE_PATH_LITERAL(_PATH_RESCONF);

[Mark Mentovai, 2012/04/17 19:53:54]

This, too, can be static or in an anonymous namespace.

+
+class DnsConfigServicePosix::ConfigWatcher : public FilePathWatcherWrapper {
+ public:
+  bool Watch(const base::Callback<void(bool succeeded)>& callback) {
+    return FilePathWatcherWrapper:Watch(FilePath(kFilePathConfig), callback);
+  }
+};
+#endif
+
 DnsConfigServicePosix::DnsConfigServicePosix()
-    : config_watcher_(new FilePathWatcherWrapper()),
+    : config_watcher_(new ConfigWatcher()),
       hosts_watcher_(new FilePathWatcherWrapper()) {
   config_reader_ = new ConfigReader(
       base::Bind(&DnsConfigServicePosix::OnConfigRead,
@@ -106,18 +131,17 @@ void DnsConfigServicePosix::Watch(const CallbackType& callback) {
 
   // Even if watchers fail, we keep the other one as it provides useful signals.
   if (config_watcher_->Watch(
-         FilePath(kFilePathConfig),
-         base::Bind(&DnsConfigServicePosix::OnConfigChanged,
-                    base::Unretained(this)))) {
+          base::Bind(&DnsConfigServicePosix::OnConfigChanged,
+                     base::Unretained(this)))) {
     OnConfigChanged(true);
   } else {
     OnConfigChanged(false);
   }
 
   if (hosts_watcher_->Watch(
-         FilePath(kFilePathHosts),
-         base::Bind(&DnsConfigServicePosix::OnHostsChanged,
-                    base::Unretained(this)))) {
+          FilePath(kFilePathHosts),
+          base::Bind(&DnsConfigServicePosix::OnHostsChanged,
+                     base::Unretained(this)))) {
     OnHostsChanged(true);
   } else {
     OnHostsChanged(false);