Convert plain C-style casts to use CFCastStrict and GetValueFromDictionary template

net/proxy/proxy_config_service_mac.cc

Index: net/proxy/proxy_config_service_mac.cc
diff --git a/net/proxy/proxy_config_service_mac.cc b/net/proxy/proxy_config_service_mac.cc
index 2fd6ee00af687e0272de16f65bb4b31c5e5c5a82..d462188bc96fc4c57690b879dbacaeb78a2194b0 100644
--- a/net/proxy/proxy_config_service_mac.cc
+++ b/net/proxy/proxy_config_service_mac.cc
@@ -8,6 +8,7 @@
 #include <SystemConfiguration/SystemConfiguration.h>
 
 #include "base/logging.h"
+#include "base/mac/foundation_util.h"
 #include "base/mac/mac_util.h"

[wtc, 2011/11/16 06:23:33]

Can you see if we can remove this #include line now?

 #include "base/mac/scoped_cftyperef.h"
 #include "base/message_loop.h"
@@ -28,8 +29,8 @@ const int kPollIntervalSec = 5;
 bool GetBoolFromDictionary(CFDictionaryRef dict,
                            CFStringRef key,
                            bool default_value) {
-  CFNumberRef number = (CFNumberRef)base::mac::GetValueFromDictionary(
-      dict, key, CFNumberGetTypeID());
+  CFNumberRef number = base::mac::CFCastStrict<CFNumberRef>(
+      base::mac::GetValueFromDictionary(dict, key, CFNumberGetTypeID()));

[Mark Mentovai, 2011/11/11 17:31:50]

I would only consider this a temporary measure. GetValueFromDictionary now
requires you to specify the type in two different ways: once as the third
argument and once in the cast as you store the value in an appropriately-typed
variable. That’s not cool. I think that GetValueFromDictionary should be
templatized to return the proper type.

   if (!number)
     return default_value;
 
@@ -60,10 +61,11 @@ void GetCurrentProxyConfig(ProxyConfig* config) {
   if (GetBoolFromDictionary(config_dict.get(),
                             kSCPropNetProxiesProxyAutoConfigEnable,
                             false)) {
-    CFStringRef pac_url_ref = (CFStringRef)base::mac::GetValueFromDictionary(
-        config_dict.get(),
-        kSCPropNetProxiesProxyAutoConfigURLString,
-        CFStringGetTypeID());
+    CFStringRef pac_url_ref = base::mac::CFCastStrict<CFStringRef>(
+        base::mac::GetValueFromDictionary(
+            config_dict.get(),
+            kSCPropNetProxiesProxyAutoConfigURLString,
+            CFStringGetTypeID()));
     if (pac_url_ref)
       config->set_pac_url(GURL(base::SysCFStringRefToUTF8(pac_url_ref)));
   }
@@ -129,16 +131,16 @@ void GetCurrentProxyConfig(ProxyConfig* config) {
 
   // proxy bypass list
 
-  CFArrayRef bypass_array_ref =
-      (CFArrayRef)base::mac::GetValueFromDictionary(
+  CFArrayRef bypass_array_ref = base::mac::CFCastStrict<CFArrayRef>(
+      base::mac::GetValueFromDictionary(
           config_dict.get(),
           kSCPropNetProxiesExceptionsList,
-          CFArrayGetTypeID());
+          CFArrayGetTypeID()));
   if (bypass_array_ref) {
     CFIndex bypass_array_count = CFArrayGetCount(bypass_array_ref);
     for (CFIndex i = 0; i < bypass_array_count; ++i) {
-      CFStringRef bypass_item_ref =
-          (CFStringRef)CFArrayGetValueAtIndex(bypass_array_ref, i);
+      CFStringRef bypass_item_ref = base::mac::CFCast<CFStringRef>(
+              CFArrayGetValueAtIndex(bypass_array_ref, i));
       if (CFGetTypeID(bypass_item_ref) != CFStringGetTypeID()) {

[Mark Mentovai, 2011/11/11 17:31:50]

This is wrong now. bypass_item_ref will either be a pointer to a CFString, or it
will be NULL if the value in the array was not a CFString. This check can become

      if (!bypass_item_ref) {

         LOG(WARNING) << "Expected value for item " << i
                      << " in the kSCPropNetProxiesExceptionsList"