commit 7111c9145bc6c5f46c6ac129405b9057589647e0
author Erik Kline <ek@google.com> Tue Aug 08 20:53:46 2017 -0700
committer Erik Kline <ek@google.com> Wed Aug 09 13:57:14 2017 -0700
tree cbb0199764228d146855869468dd353c3128237a
parent c7da241b3e083ea70bd21a8f279a93b1b53393b3

Add future consideration for USB rndis notification handling

Test: builds
Bug: 63970368
Merged-In: If6afe1cdce9995251ba53fbf30acf9d6e31f9433
Merged-In: I78e280c2299ec3f4d97138ca648f9629ba82e87f
Merged-In: I891cd82b0e8af1049fa3ab9b0a95a2a9ab2e0b26
Change-Id: Ic9ca133a44123f281bd4cfeac57d426346c2b76d
(cherry picked from commit 1290568110405fd45f13824e67beeb98c30400a4)

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 0288eb6..de54144 100644
--- a/services/core/java/com/android/server/connectivity/Tethering.java
+++ b/services/core/java/com/android/server/connectivity/Tethering.java
@@ -840,6 +840,16 @@
             // For more explanation, see b/62552150 .
             synchronized (Tethering.this.mPublicSync) {
                 // Always record the state of RNDIS.
+                // TODO: consider:
+                //     final boolean disconnected = !usbConnected;
+                //     if (disconnected) {
+                //         mRndisEnabled = false;
+                //         mUsbTetherRequested = false;
+                //         return;
+                //     }
+                //     final boolean configured = usbConnected && usbConfigured;
+                //     mRndisEnabled = configured ? rndisEnabled : false;
+                //     if (!configured) return;
                 mRndisEnabled = rndisEnabled;
 
                 if (usbConnected && !usbConfigured) {