Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(275)

Unified Diff: net/android/java/src/org/chromium/net/NetworkChangeNotifier.java

Issue 2361563003: Non-functional simplification of NetworkChangeNotifierAutoDetect code (Closed)
Patch Set: fix additional reference Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/android/java/src/org/chromium/net/NetworkChangeNotifier.java
diff --git a/net/android/java/src/org/chromium/net/NetworkChangeNotifier.java b/net/android/java/src/org/chromium/net/NetworkChangeNotifier.java
index 7c38979dc21c90ee594622cf6d2ce11d730a8211..20f1013801b8ccf44aed9e2dd7bd892b11d4369d 100644
--- a/net/android/java/src/org/chromium/net/NetworkChangeNotifier.java
+++ b/net/android/java/src/org/chromium/net/NetworkChangeNotifier.java
@@ -77,9 +77,9 @@ public class NetworkChangeNotifier {
@CalledByNative
public int getCurrentConnectionSubtype() {
- return mAutoDetector == null
- ? ConnectionSubtype.SUBTYPE_UNKNOWN
- : mAutoDetector.getCurrentConnectionSubtype(mAutoDetector.getCurrentNetworkState());
+ return mAutoDetector == null ? ConnectionSubtype.SUBTYPE_UNKNOWN
+ : NetworkChangeNotifierAutoDetect.convertToConnectionSubtype(
+ mAutoDetector.getCurrentNetworkState());
}
@CalledByNative
@@ -217,7 +217,8 @@ public class NetworkChangeNotifier {
mContext, policy);
final NetworkChangeNotifierAutoDetect.NetworkState networkState =
mAutoDetector.getCurrentNetworkState();
- updateCurrentConnectionType(mAutoDetector.getCurrentConnectionType(networkState));
+ updateCurrentConnectionType(
+ NetworkChangeNotifierAutoDetect.convertToConnectionType(networkState));
updateCurrentMaxBandwidth(mAutoDetector.getCurrentMaxBandwidthInMbps(networkState));
}
} else {

Powered by Google App Engine
This is Rietveld 408576698