Wire NQE Prefs to Profile

net/base/network_change_notifier.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/base/network_change_notifier.h"
 
 #include <limits>
 #include <unordered_set>
 
 #include "base/macros.h"
@@ skipping 20 matching lines @@
 #elif defined(OS_LINUX) && !defined(OS_CHROMEOS)
 #include "net/base/network_change_notifier_linux.h"
 #elif defined(OS_MACOSX)
 #include "net/base/network_change_notifier_mac.h"
 #endif
 
 namespace net {
 
 namespace {
 
+static const char* const kConnectionTypeNames[] = {
+    "CONNECTION_UNKNOWN", "CONNECTION_ETHERNET", "CONNECTION_WIFI",
+    "CONNECTION_2G",      "CONNECTION_3G",       "CONNECTION_4G",
+    "CONNECTION_NONE",    "CONNECTION_BLUETOOTH"};
+
 // The actual singleton notifier.  The class contract forbids usage of the API
 // in ways that would require us to place locks around access to this object.
 // (The prohibition on global non-POD objects makes it tricky to do such a thing
 // anyway.)
 NetworkChangeNotifier* g_network_change_notifier = NULL;
 
 // Class factory singleton.
 NetworkChangeNotifierFactory* g_network_change_notifier_factory = NULL;
 
 class MockNetworkChangeNotifier : public NetworkChangeNotifier {
@@ skipping 623 matching lines @@
 
 // static
 void NetworkChangeNotifier::GetDnsConfig(DnsConfig* config) {
   if (!g_network_change_notifier) {
     *config = DnsConfig();
   } else {
     g_network_change_notifier->network_state_->GetDnsConfig(config);
   }
 }
 
+//  static
+NetworkChangeNotifier::ConnectionType
+NetworkChangeNotifier::StringToConnectionType(
+    const std::string& connection_type) {
+  static_assert(arraysize(kConnectionTypeNames) ==
+                    NetworkChangeNotifier::CONNECTION_LAST + 1,
+                "ConnectionType name count should match");
+  const auto it = std::find(
+      kConnectionTypeNames,

[RyanSturm, 2016/10/12 21:33:34]

Can this be more C++11-ish and have std::find(std::begin(kConnectionTypeNames),
std::end(kConnectionTypeNames), connection_type)

and

it != std::end(kConnectionTypeNames) below.

[tbansal1, 2016/10/12 22:04:01]

Obsolete.

+      kConnectionTypeNames + arraysize(kConnectionTypeNames), connection_type);
+  return it != kConnectionTypeNames + arraysize(kConnectionTypeNames)
+             ? static_cast<NetworkChangeNotifier::ConnectionType>(

[RyanSturm, 2016/10/12 21:33:34]

I'm not convinced this static_cast is a good way to do this, but the other
option is doing a switch/case and I am not totally sold that that is better. I'm
worried about someone re-ordering the enum or the array without re-ordering the
other.

Use your judgement on which approach you want to go with, but I'd prefer
std::distance(std::begin(kConnectionTypeNames), it) instead of plain subtraction
if you stick with the current approach. It will do the same thing, but might be
more clear to read.

[tbansal1, 2016/10/12 22:04:01]

From
https://cs.chromium.org/chromium/src/net/base/network_change_notifier.h?rcl=0...:
"New enum values should only be added to the end of the enum and no values
should be modified or reused, as this is reported via UMA."
Obsolete.

+                   it - kConnectionTypeNames)
+             : NetworkChangeNotifier::CONNECTION_UNKNOWN;
+}
+
 // static
 const char* NetworkChangeNotifier::ConnectionTypeToString(
     ConnectionType type) {
-  static const char* const kConnectionTypeNames[] = {
-    "CONNECTION_UNKNOWN",
-    "CONNECTION_ETHERNET",
-    "CONNECTION_WIFI",
-    "CONNECTION_2G",
-    "CONNECTION_3G",
-    "CONNECTION_4G",
-    "CONNECTION_NONE",
-    "CONNECTION_BLUETOOTH"
-  };
   static_assert(arraysize(kConnectionTypeNames) ==
                     NetworkChangeNotifier::CONNECTION_LAST + 1,
                 "ConnectionType name count should match");
   if (type < CONNECTION_UNKNOWN || type > CONNECTION_LAST) {
     NOTREACHED();
     return "CONNECTION_INVALID";
   }
   return kConnectionTypeNames[type];
 }
 
@@ skipping 462 matching lines @@
 
 NetworkChangeNotifier::DisableForTest::~DisableForTest() {
   DCHECK(!g_network_change_notifier);
   g_network_change_notifier = network_change_notifier_;
 }
 
 void NetworkChangeNotifier::DNSObserver::OnInitialDNSConfigRead() {
 }
 
 }  // namespace net