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

Fixed a bug where collapserunnables where dropped

Browse Source 
When a runnable was added while iterating over it,
it was cleared right after.

Bug: 22260165
Change-Id: I2faf9af61c08853eb8364d49ccd36d686a4b3bf8
This commit is contained in:
Selim Cinek
2015-07-07 18:43:59 -07:00
parent 2020178f97
commit ae77f8e658
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
packages/SystemUI/src/com/android/systemui/statusbar/phone/PhoneStatusBar.java
View File

@@ -81,7 +81,6 @@ import android.view.MotionEvent;
import android.view.ThreadedRenderer;
import android.view.VelocityTracker;
import android.view.View;
import android.view.ViewConfiguration;
import android.view.ViewGroup.LayoutParams;
import android.view.ViewStub;
import android.view.WindowManager;
@@ -106,7 +105,6 @@ import com.android.systemui.EventLogConstants;
import com.android.systemui.EventLogTags;
import com.android.systemui.Prefs;
import com.android.systemui.R;
import com.android.systemui.SwipeHelper;
import com.android.systemui.assist.AssistManager;
import com.android.systemui.doze.DozeHost;
import com.android.systemui.doze.DozeLog;
@@ -2145,11 +2143,13 @@ public class PhoneStatusBar extends BaseStatusBar implements DemoMode,
}
private void runPostCollapseRunnables() {
int size = mPostCollapseRunnables.size();
for (int i = 0; i < size; i++) {
mPostCollapseRunnables.get(i).run();
}
ArrayList<Runnable> clonedList = new ArrayList<>(mPostCollapseRunnables);
mPostCollapseRunnables.clear();
int size = clonedList.size();
for (int i = 0; i < size; i++) {
clonedList.get(i).run();
}
}
Animator mScrollViewAnim, mClearButtonAnim;