Cleanup network type matching.

chrome/browser/extensions/api/dial/dial_service.cc

Index: chrome/browser/extensions/api/dial/dial_service.cc
diff --git a/chrome/browser/extensions/api/dial/dial_service.cc b/chrome/browser/extensions/api/dial/dial_service.cc
index 605dfdb78d0777169fa85638aebcc229aae71817..40f8cfb9310950a79eb4f220011fdbad346eed39 100644
--- a/chrome/browser/extensions/api/dial/dial_service.cc
+++ b/chrome/browser/extensions/api/dial/dial_service.cc
@@ -189,8 +189,8 @@ void DialServiceImpl::StartDiscovery() {
 
 #if defined(OS_CHROMEOS)
 IPAddressNumber DialServiceImpl::GetBestBindAddressChromeOS() {
-  std::string connection_types[] =
-      {flimflam::kTypeWifi, flimflam::kTypeEthernet};
+  std::string connection_types[] = {
+      flimflam::kTypeWifi, chromeos::NetworkStateHandler::kMatchTypeWired};
   for (uint i = 0; i < arraysize(connection_types); ++i) {

[stevenjb, 2013/08/28 15:51:30]

I find this confusing. Can we move the logic below to a helper method and call
it explicitly for Wifi and Wired? It also seems strange to me that we would
check wifi before wired, do you know why that is, and if so can you add a
comment? If not, maybe investigate?

[pneubeck (no reviews), 2013/09/02 20:41:21]

I will add the author as reviewer.

     IPAddressNumber bind_ip_address;
     const chromeos::NetworkState* state =