diff options
author | Irfan Sheriff <isheriff@google.com> | 2013-04-23 21:57:53 +0000 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2013-04-23 21:57:53 +0000 |
commit | 0aea616acdb228b7a8c061977c0445174f60a1e1 (patch) | |
tree | 85c314a4785ac7d2faa0097003be04a405a8b877 | |
parent | 22a3344cda1a3dcf4f719e0685c5d06155922c29 (diff) | |
parent | e68d858b0faf638045980237922e969516b1213e (diff) | |
download | frameworks_base-0aea616acdb228b7a8c061977c0445174f60a1e1.zip frameworks_base-0aea616acdb228b7a8c061977c0445174f60a1e1.tar.gz frameworks_base-0aea616acdb228b7a8c061977c0445174f60a1e1.tar.bz2 |
Merge "Keep device state always updated" into jb-mr2-dev
-rw-r--r-- | services/java/com/android/server/wifi/WifiService.java | 18 |
1 files changed, 5 insertions, 13 deletions
diff --git a/services/java/com/android/server/wifi/WifiService.java b/services/java/com/android/server/wifi/WifiService.java index 9560199..5cf1966 100644 --- a/services/java/com/android/server/wifi/WifiService.java +++ b/services/java/com/android/server/wifi/WifiService.java @@ -121,8 +121,6 @@ public final class WifiService extends IWifiManager.Stub { * on this */ private WorkSource mScanWorkSource; - private boolean mIsReceiverRegistered = false; - /** * Asynchronous channel to WifiStateMachine */ @@ -272,6 +270,11 @@ public final class WifiService extends IWifiManager.Stub { } } }, new IntentFilter(WifiManager.SCAN_RESULTS_AVAILABLE_ACTION)); + + // Adding optimizations of only receiving broadcasts when wifi is enabled + // can result in race conditions when apps toggle wifi in the background + // without active user involvement. Always receive broadcasts. + registerForBroadcasts(); } private WifiController mWifiController; @@ -421,17 +424,6 @@ public final class WifiService extends IWifiManager.Stub { } mWifiController.sendMessage(CMD_WIFI_TOGGLED); - - if (enable) { - if (!mIsReceiverRegistered) { - registerForBroadcasts(); - mIsReceiverRegistered = true; - } - } else if (mIsReceiverRegistered) { - mContext.unregisterReceiver(mReceiver); - mIsReceiverRegistered = false; - } - return true; } |