commit 7a42d3ad83cbb3e7f98964eb876b8b21bd46816a
author Erik Kline <ek@google.com> Wed Feb 15 00:38:50 2017 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Wed Feb 15 00:38:50 2017 +0000
tree 417bae67a42fa6304478f10bbfb62f650a2eb6a9
parent 63445824d698cddf788b5a4de5627f319f501e87
parent 14f7faf78e048b7209551cc32ac50b2673e15409

Merge "Split chooseUpstreamType() into two separate functions"

services/core/java/com/android/server/connectivity/Tethering.java

diff --git a/services/core/java/com/android/server/connectivity/Tethering.java b/services/core/java/com/android/server/connectivity/Tethering.java
index 91a66a8..63e66a2 100644
--- a/services/core/java/com/android/server/connectivity/Tethering.java
+++ b/services/core/java/com/android/server/connectivity/Tethering.java
@@ -1057,9 +1057,13 @@
             }
 
             protected void chooseUpstreamType(boolean tryCell) {
+                final int upstreamType = findPreferredUpstreamType(tryCell);
+                setUpstreamByType(upstreamType);
+            }
+
+            protected int findPreferredUpstreamType(boolean tryCell) {
                 final ConnectivityManager cm = getConnectivityManager();
                 int upType = ConnectivityManager.TYPE_NONE;
-                String iface = null;
 
                 updateConfiguration(); // TODO - remove?
 
@@ -1117,7 +1121,13 @@
                         break;
                 }
 
+                return upType;
+            }
+
+            protected void setUpstreamByType(int upType) {
+                final ConnectivityManager cm = getConnectivityManager();
                 Network network = null;
+                String iface = null;
                 if (upType != ConnectivityManager.TYPE_NONE) {
                     LinkProperties linkProperties = cm.getLinkProperties(upType);
                     if (linkProperties != null) {