[TIMOB-6700] Android: TiActivity - v8/rhino - Console reports activity has leaked window
| GitHub Issue | n/a |
|---|---|
| Type | Bug |
| Priority | Critical |
| Status | Closed |
| Resolution | Fixed |
| Resolution Date | 2012-02-14T23:29:05.000+0000 |
| Affected Version/s | Release 1.8.0.1 |
| Fix Version/s | Sprint 2012-03, Release 2.0.0, Release 1.8.2 |
| Components | Android |
| Labels | qe-and121411, qe-testadded, regression |
| Reporter | Eric Merriman |
| Assignee | Hieu Pham |
| Created | 2011-12-15T12:26:22.000+0000 |
| Updated | 2012-02-14T23:29:05.000+0000 |
Description
Description:
During a KS pass on v8 on the LG slate and Xoom, pressing "back" on the Phone>Play Movie>Local test outputs the following in the console:
E/WindowManager( 812): Activity org.appcelerator.titanium.TiActivity has leaked window com.android.internal.policy.impl.PhoneWindow$DecorView@40fbb588 that was originally added here
E/WindowManager( 812): android.view.WindowLeaked: Activity org.appcelerator.titanium.TiActivity has leaked window com.android.internal.policy.impl.PhoneWindow$DecorView@40fbb588 that was originally added here
E/WindowManager( 812): at android.view.ViewRoot.(ViewRoot.java:288)
E/WindowManager( 812): at android.view.WindowManagerImpl.addView(WindowManagerImpl.java:249)
E/WindowManager( 812): at android.view.WindowManagerImpl.addView(WindowManagerImpl.java:193)
E/WindowManager( 812): at android.view.WindowManagerImpl$CompatModeWrapper.addView(WindowManagerImpl.java:118)
E/WindowManager( 812): at android.view.Window$LocalWindowManager.addView(Window.java:532)
E/WindowManager( 812): at android.app.Dialog.show(Dialog.java:269)
E/WindowManager( 812): at ti.modules.titanium.ui.widget.TiUIDialog.show(TiUIDialog.java:248)
E/WindowManager( 812): at ti.modules.titanium.ui.AlertDialogProxy$1.run(AlertDialogProxy.java:72)
E/WindowManager( 812): at android.os.Handler.handleCallback(Handler.java:587)
E/WindowManager( 812): at android.os.Handler.dispatchMessage(Handler.java:92)
E/WindowManager( 812): at android.os.Looper.loop(Looper.java:132)
E/WindowManager( 812): at android.app.ActivityThread.main(ActivityThread.java:4123)
E/WindowManager( 812): at java.lang.reflect.Method.invokeNative(Native Method)
E/WindowManager( 812): at java.lang.reflect.Method.invoke(Method.java:491)
E/WindowManager( 812): at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:841)
E/WindowManager( 812): at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:599)
E/WindowManager( 812): at dalvik.system.NativeStart.main(Native Method)
D/dalvikvm( 222): GC_EXPLICIT freed 7K, 5% free 20303K/21191K, paused 7ms+2ms
Steps to reproduce:
1) Install KS with SDK 1.8.0.1
2) Navigate to Phone>Play Movie>Local
3) Press the back button while examining the console
Result:
Console logs error
Expected:
No error in console
Note:
Does not occur on 175 SDK with KS 175
Also occurs on the nook color with SDK: 1.8.0.1.v20111216124633 returning from the Platform Data page. Output:
Cannot reproduce with latest master
I was also unable to reproduce with KS built with SDK 1.9.0.v20120118151633 on a Nexus S. Attempted 10 times.
Tested with 1.8.1.v20120124200134 on Droid 3 (2.3.4) run the following code (test came from timob-4947)
_win = Titanium.UI.createWindow(); var button1 = Ti.UI.createButton({ width : 150, height : 150, title : 'press me!' }); button1.addEventListener('click', function() { var win = Ti.UI.createWindow({ modal : true, backgroundColor : '#000' }); win.addEventListener('close', function() { alert('Close event fired'); Ti.API.info('Close event fired'); }); var button2 = Ti.UI.createButton({ width : 150, height : 150, title : 'Press me!' }); button2.addEventListener('click', function() { win.close(); }); win.open(); win.add(button2); }); _win.add(button1); _win.open();Also seen on Rhino KS pass but not by following steps. This was second click to Platform>Platform data so it seems that the area the bug was originally reported in may not be relevant.
Note to QE: This test was added as part of TIMOB-4947.
Verified fixed with SDK 1.9.0.v20120207000134 and Titanium Studio, build: 1.0.8.201201262211 with both V8 and Rhino using attached sample code from Natalie.