From ef0bdb7d10a0f0b6c0d8e022e6b557030dcfda3b Mon Sep 17 00:00:00 2001 From: juwei Date: Fri, 30 Dec 2016 18:15:33 +0800 Subject: SnapdragonCamera: Fix crash during UI stress test CaptureModule and CaptureUI have register listeners in SettingsManager, but forget unregiseter when UI destroy. This makes the too many RenderNode objects exist and never GC. At last, the weak reference of RenderNode at native layer exceed the 51200 limitation and crash. Unregister the listener of SettingsManager will fix it. Change-Id: I881b47f8f062c366e028db59a8cf6983c1d99f5a CRs-Fixed: 1103907 --- src/com/android/camera/CaptureModule.java | 2 ++ 1 file changed, 2 insertions(+) (limited to 'src') diff --git a/src/com/android/camera/CaptureModule.java b/src/com/android/camera/CaptureModule.java index 26613786f..4b71f9b09 100644 --- a/src/com/android/camera/CaptureModule.java +++ b/src/com/android/camera/CaptureModule.java @@ -2358,6 +2358,8 @@ public class CaptureModule implements CameraModule, PhotoController, if(mFrameProcessor != null){ mFrameProcessor.onDestory(); } + mSettingsManager.unregisterListener(this); + mSettingsManager.unregisterListener(mUI); } @Override -- cgit v1.2.3