Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Add some synchronization to hotspot callbacks" into nyc-dev

Browse Source
This commit is contained in:
Jason Monk
2016-03-17 13:31:29 +00:00
committed by Android (Google) Code Review
parent 6123bf5d66 ffc63f45a0
commit c049cb92a1
1 changed files with 14 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
packages/SystemUI/src/com/android/systemui/statusbar/policy/HotspotControllerImpl.java
View File

@@ -69,18 +69,22 @@ public class HotspotControllerImpl implements HotspotController {
@Override
public void addCallback(Callback callback) {
if (callback == null || mCallbacks.contains(callback)) return;
if (DEBUG) Log.d(TAG, "addCallback " + callback);
mCallbacks.add(callback);
mReceiver.setListening(!mCallbacks.isEmpty());
synchronized (mCallbacks) {
if (callback == null || mCallbacks.contains(callback)) return;
if (DEBUG) Log.d(TAG, "addCallback " + callback);
mCallbacks.add(callback);
mReceiver.setListening(!mCallbacks.isEmpty());
}
}
@Override
public void removeCallback(Callback callback) {
if (callback == null) return;
if (DEBUG) Log.d(TAG, "removeCallback " + callback);
mCallbacks.remove(callback);
mReceiver.setListening(!mCallbacks.isEmpty());
synchronized (mCallbacks) {
mCallbacks.remove(callback);
mReceiver.setListening(!mCallbacks.isEmpty());
}
}
@Override
@@ -110,8 +114,10 @@ public class HotspotControllerImpl implements HotspotController {
}
private void fireCallback(boolean isEnabled) {
for (Callback callback : mCallbacks) {
callback.onHotspotChanged(isEnabled);
synchronized (mCallbacks) {
for (Callback callback : mCallbacks) {
callback.onHotspotChanged(isEnabled);
}
}
}