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

Clean up back handler from prototracer

Browse Source 
Bug: 160161606
Test: Switch darkmode serveral times and verify there aren't
      old refs in the heap dump
Change-Id: I1678d10e725f6bdcbc807064413b4bc549ef8ef2
Merged-In: I1678d10e725f6bdcbc807064413b4bc549ef8ef2
(cherry picked from commit 469a3084b7)
This commit is contained in:
Winson Chung
2020-07-17 22:33:08 +00:00
parent c121ae683d
commit d88d71e67a
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
packages/SystemUI/src/com/android/systemui/statusbar/phone/EdgeBackGestureHandler.java
View File

@@ -231,7 +231,6 @@ public class EdgeBackGestureHandler extends CurrentUserTracker implements Displa
} }
} }
Dependency.get(ProtoTracer.class).add(this);
mLongPressTimeout = Math.min(MAX_LONG_PRESS_TIMEOUT, mLongPressTimeout = Math.min(MAX_LONG_PRESS_TIMEOUT,
ViewConfiguration.getLongPressTimeout()); ViewConfiguration.getLongPressTimeout());
@@ -286,6 +285,7 @@ public class EdgeBackGestureHandler extends CurrentUserTracker implements Displa
*/ */
public void onNavBarAttached() { public void onNavBarAttached() {
mIsAttached = true; mIsAttached = true;
Dependency.get(ProtoTracer.class).add(this);
mOverviewProxyService.addCallback(mQuickSwitchListener); mOverviewProxyService.addCallback(mQuickSwitchListener);
updateIsEnabled(); updateIsEnabled();
startTracking(); startTracking();
@@ -296,6 +296,7 @@ public class EdgeBackGestureHandler extends CurrentUserTracker implements Displa
*/ */
public void onNavBarDetached() { public void onNavBarDetached() {
mIsAttached = false; mIsAttached = false;
Dependency.get(ProtoTracer.class).remove(this);
mOverviewProxyService.removeCallback(mQuickSwitchListener); mOverviewProxyService.removeCallback(mQuickSwitchListener);
updateIsEnabled(); updateIsEnabled();
stopTracking(); stopTracking();