diff options
author | Chalard Jean <jchalard@google.com> | 2019-04-01 17:12:06 -0700 |
---|---|---|
committer | android-build-merger <android-build-merger@google.com> | 2019-04-01 17:12:06 -0700 |
commit | 06dea648f3dbb13d8f46b420f7de2b86cff27ccf (patch) | |
tree | 539f8a38b59e5930787320547f821814b32f2988 | |
parent | 73267ed72ff83262a53a10046a3d5c83e452b916 (diff) | |
parent | 27059069fd9f19e4286cca43b0bc4a014f799dbd (diff) |
Merge "Cleanup SystemReady in the network stack"
am: e0ef7ce4f7
Change-Id: I1a55078b74d5f94afde94da0dc5f6e1d2af10de3
-rw-r--r-- | src/com/android/server/NetworkStackService.java | 6 | ||||
-rw-r--r-- | src/com/android/server/connectivity/NetworkMonitor.java | 15 |
2 files changed, 0 insertions, 21 deletions
diff --git a/src/com/android/server/NetworkStackService.java b/src/com/android/server/NetworkStackService.java index 63f057c..a0a90fd 100644 --- a/src/com/android/server/NetworkStackService.java +++ b/src/com/android/server/NetworkStackService.java @@ -303,12 +303,6 @@ public class NetworkStackService extends Service { } @Override - public void notifySystemReady() { - checkNetworkStackCallingPermission(); - mNm.notifySystemReady(); - } - - @Override public void notifyNetworkConnected(LinkProperties lp, NetworkCapabilities nc) { checkNetworkStackCallingPermission(); mNm.notifyNetworkConnected(lp, nc); diff --git a/src/com/android/server/connectivity/NetworkMonitor.java b/src/com/android/server/connectivity/NetworkMonitor.java index c000fc6..6f31f9b 100644 --- a/src/com/android/server/connectivity/NetworkMonitor.java +++ b/src/com/android/server/connectivity/NetworkMonitor.java @@ -298,8 +298,6 @@ public class NetworkMonitor extends StateMachine { // Avoids surfacing "Sign in to network" notification. private boolean mDontDisplaySigninNotification = false; - private volatile boolean mSystemReady = false; - private final State mDefaultState = new DefaultState(); private final State mValidatedState = new ValidatedState(); private final State mMaybeNotifyState = new MaybeNotifyState(); @@ -434,15 +432,6 @@ public class NetworkMonitor extends StateMachine { } /** - * Send a notification to NetworkMonitor indicating that the system is ready. - */ - public void notifySystemReady() { - // No need to run on the handler thread: mSystemReady is volatile and read only once on the - // isCaptivePortal() thread. - mSystemReady = true; - } - - /** * Send a notification to NetworkMonitor indicating that the network is now connected. */ public void notifyNetworkConnected(LinkProperties lp, NetworkCapabilities nc) { @@ -1592,10 +1581,6 @@ public class NetworkMonitor extends StateMachine { */ private void sendNetworkConditionsBroadcast(boolean responseReceived, boolean isCaptivePortal, long requestTimestampMs, long responseTimestampMs) { - if (!mSystemReady) { - return; - } - Intent latencyBroadcast = new Intent(NetworkMonitorUtils.ACTION_NETWORK_CONDITIONS_MEASURED); if (mNetworkCapabilities.hasTransport(TRANSPORT_WIFI)) { |