Adding platform support check for NetworkObserver

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 "base/metrics/histogram_macros.h"
 #include "base/strings/string_util.h"
@@ skipping 783 matching lines @@
     }
   }
   return result;
 }
 
 // static
 NetworkChangeNotifier* NetworkChangeNotifier::CreateMock() {
   return new MockNetworkChangeNotifier();
 }
 
+bool NetworkChangeNotifier::IsNetworkObserverSupported() {
+#if defined(OS_ANDROID)
+  // Notifications for NetworkObserver only implemented for Android
+  // versions >= L.
+  if (base::android::BuildInfo::GetInstance()->sdk_int() >=
+      base::android::SDK_VERSION_LOLLIPOP) {
+    return true;

[Ryan Hamilton, 2015/11/19 02:19:04]

FWIW, you can just do:

return base::android::BuildInfo::GetInstance()->sdk_int() >=
       base::android::SDK_VERSION_LOLLIPOP;

[Jana, 2015/11/19 03:25:19]

Done.

+  }
+#endif
+  return false;
+}
+
 void NetworkChangeNotifier::AddIPAddressObserver(IPAddressObserver* observer) {
   if (g_network_change_notifier)
     g_network_change_notifier->ip_address_observer_list_->AddObserver(observer);
 }
 
 void NetworkChangeNotifier::AddConnectionTypeObserver(
     ConnectionTypeObserver* observer) {
   if (g_network_change_notifier) {
     g_network_change_notifier->connection_type_observer_list_->AddObserver(
         observer);
@@ skipping 331 matching lines @@
 
 NetworkChangeNotifier::DisableForTest::~DisableForTest() {
   DCHECK(!g_network_change_notifier);
   g_network_change_notifier = network_change_notifier_;
 }
 
 void NetworkChangeNotifier::DNSObserver::OnInitialDNSConfigRead() {
 }
 
 }  // namespace net