replace platform conditions with os_posix and os_bsd where applicable

net/base/host_resolver_impl.cc

Index: net/base/host_resolver_impl.cc
diff --git a/net/base/host_resolver_impl.cc b/net/base/host_resolver_impl.cc
index 334695c55652c7c2bf70b91d7d79444512649292..217f1e1329e678e0500a337fbc45cf9a33e165e1 100644
--- a/net/base/host_resolver_impl.cc
+++ b/net/base/host_resolver_impl.cc
@@ -84,9 +84,11 @@ std::vector<int> GetAllGetAddrinfoOSErrors() {
 #if defined(OS_POSIX)
 #if !defined(OS_FREEBSD)
 #if !defined(OS_ANDROID)
-    // EAI_ADDRFAMILY has been declared obsolete in Android's netdb.h.
+    // EAI_ADDRFAMILY has been declared obsolete in Android's and
+    // FreeBSD's netdb.h.
     EAI_ADDRFAMILY,
 #endif
+    // EAI_NODATA has been declared obsolete in FreeBSD's netdb.h.
     EAI_NODATA,
 #endif

[wtc, 2011/11/30 01:08:15]

It seems better to use independent ifdefs rather than nesting
them:

#if !defined(OS_ANDROID) && !defined(OS_FREEBSD)
    // ...
    EAI_ADDRFAMILY,
#endif
#if !defined(OS_FREEBSD)
    // ...
    EAI_NODATA,
#endif

     EAI_AGAIN,