Reimplements net::AddressList without struct addrinfo.

content/browser/renderer_host/pepper_message_filter.cc

Index: content/browser/renderer_host/pepper_message_filter.cc
diff --git a/content/browser/renderer_host/pepper_message_filter.cc b/content/browser/renderer_host/pepper_message_filter.cc
index 9babcd1b8ba656c1b7585e1504fcd4e4cc8688e8..7eaffaab3073ba2128529291d63bdc0177cab738 100644
--- a/content/browser/renderer_host/pepper_message_filter.cc
+++ b/content/browser/renderer_host/pepper_message_filter.cc
@@ -561,10 +561,9 @@ void PepperMessageFilter::OnHostResolverResolveLookupFinished(
                                     bound_info.plugin_dispatcher_id,
                                     bound_info.host_resolver_id);
   } else {
-    std::string canonical_name;
-    addresses.GetCanonicalName(&canonical_name);
+    const std::string& canonical_name = addresses.get_canonical_name();
     scoped_ptr<ppapi::NetAddressList> net_address_list(
-        ppapi::CreateNetAddressListFromAddrInfo(addresses.head()));
+        ppapi::CreateNetAddressListFromAddressList(addresses));

[eroman, 2012/05/04 01:08:41]

This sounds weird. I don't know about this code, but seems unusual. I wander if
pepper really needs these wrappers. I'm OK with it for this refactor if the
OWNER signs off on it.

     if (!net_address_list.get()) {
       SendHostResolverResolveACKError(bound_info.routing_id,
                                       bound_info.plugin_dispatcher_id,