From 1290568110405fd45f13824e67beeb98c30400a4 Mon Sep 17 00:00:00 2001
From: Erik Kline <ek@google.com>
Date: Tue, 8 Aug 2017 20:53:46 -0700
Subject: [PATCH] Add future consideration for USB rndis notification handling

Test: builds
Bug: 63970368
Change-Id: Ic9ca133a44123f281bd4cfeac57d426346c2b76d
---
 .../com/android/server/connectivity/Tethering.java     | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/services/core/java/com/android/server/connectivity/Tethering.java b/services/core/java/com/android/server/connectivity/Tethering.java
index 015f7f0778088..2b4f4e6a75f7e 100644
--- a/services/core/java/com/android/server/connectivity/Tethering.java
+++ b/services/core/java/com/android/server/connectivity/Tethering.java
@@ -834,6 +834,16 @@ public class Tethering extends BaseNetworkObserver {
             // 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) {