Expose connection subtype from NetworkChangeNotifier

net/android/network_change_notifier_android.cc

Index: net/android/network_change_notifier_android.cc
diff --git a/net/android/network_change_notifier_android.cc b/net/android/network_change_notifier_android.cc
index e093a4f5cbe3fcda006aef3d2f652a20a9c01679..cc98d5a339e132b4cb352b255f33bc35995fe2c3 100644
--- a/net/android/network_change_notifier_android.cc
+++ b/net/android/network_change_notifier_android.cc
@@ -160,6 +160,11 @@ NetworkChangeNotifierAndroid::GetCurrentConnectionType() const {
   return delegate_->GetCurrentConnectionType();
 }
 
+NetworkChangeNotifier::ConnectionSubtype
+NetworkChangeNotifierAndroid::GetCurrentConnectionSubtype() const {
+  return delegate_->GetCurrentConnectionSubtype();

[pauljensen, 2017/05/10 16:47:07]

Hmm this is a potential threading violation.  NCN::GetConnectionSubtype() is
advertised as thread-safe, but here we're implementing it by calling a function
that can only be called on the NCN's thread.  In Chrome this will be the UI
thread, in Cronet this will be an obscure initialization thread.

[tbansal1, 2017/05/11 01:08:52]

I will send a CL tomorrow to fix this. Thanks for noticing this.

+}
+
 void NetworkChangeNotifierAndroid::GetCurrentMaxBandwidthAndConnectionType(
     double* max_bandwidth_mbps,
     ConnectionType* connection_type) const {