704162 - Switching tabs yields a org.mozilla.gecko.TabsTray WIN DEATH

Status: RESOLVED WORKSFORME

(Firefox for Android Graveyard :: General, defect)

Description

[Aaron Train [:aaronmt]]

I/GeckoApp( 3404): Got message: Tab:Selected
I/GeckoApp( 3404): Switched to tab: 1
I/GeckoDoorHangerPopup( 3404): Showing all doorhangers for tab: 1
I/GeckoApp( 3404): resume
D/GeckoInputConnection( 3404): IME: handleCreateInputConnection called
D/GeckoInputConnection( 3404): IME: run()
D/GeckoInputConnection( 3404): IME: v=org.mozilla.gecko.gfx.LayerView@4058ca90
D/Zygote  (   75): Process 3404 exited cleanly (1)
I/ActivityManager(  110): Process org.mozilla.fennec (pid 3404) has died.
I/WindowManager(  110): WIN DEATH: Window{40b67058 SurfaceView paused=false}
I/WindowManager(  110): WIN DEATH: Window{407b65a0 org.mozilla.fennec/org.mozilla.fennec.App paused=false}
I/WindowManager(  110): WIN DEATH: Window{40ab05a8 org.mozilla.fennec/org.mozilla.gecko.TabsTray paused=false}

STR:

1. http://cnn.com (first tab), http://msnbc.com (second tab)
2. Switch back and forth between tabs

--
Samsung Nexus S (Android 2.3.6)
20111121040255
http://hg.mozilla.org/projects/birch/rev/41b63b2b6420

Comment 1

[Sriram Ramasubramanian [:sriram]]

I tried reproducing the crash. I couldn't get any stack trace for this. The WIN DEATH happens at SurfaceView now. Basically SurfaceView dies, causing the Application "stack" to crash. This could be "TabsTray" or "GeckoApp".

I/GeckoAppShell(20362): remove view:com.adobe.flashplayer.FlashPaintSurface@40777338

I feel this is the issue. CNN's website is flash rich. It causes the browser to load/unload flash during tabs-switching, causing the crash.

I/GeckoDoorHangerPopup(20362): Showing all doorhangers for tab: 2
I/GeckoAppShell(20362): run GeckoRunnableCallback
I/GeckoJNI(20362): void Java_org_mozilla_gecko_GeckoAppShell_executeNextRunnable(JNIEnv*, _jclass*)
I/GeckoJNI(20362): leaving void Java_org_mozilla_gecko_GeckoAppShell_executeNextRunnable(JNIEnv*, _jclass*)
D/GeckoInputConnection(20362): IME: run()
D/GeckoInputConnection(20362): IME: v=org.mozilla.gecko.gfx.LayerView@405a8210
D/GeckoInputConnection(20362): IME: handleCreateInputConnection called
D/GeckoInputConnection(20362): IME: finishComposingText
D/GeckoInputConnection(20362): IME: finishComposingText
I/ActivityManager(  110): Displayed org.mozilla.fennec_sriram/org.mozilla.gecko.TabsTray: +1s28ms
I/ActivityManager(  110): Start proc com.facebook.orca for service com.facebook.orca/.server.OrcaService: pid=20481 uid=10070 gids={3003, 1015, 1006}
I/GeckoAppShell(20362): addPluginView:com.adobe.flashplayer.FlashPaintSurface@40777338 @ x:22.0 y:294.0 w:156.0 h:52.0
I/GeckoAppShell(20362): addPluginView:com.adobe.flashplayer.FlashPaintSurface@405c1628 @ x:702.0 y:370.0 w:300.0 h:250.0
I/GeckoPlugins(20362): void anp_event_postEvent(NPP_t*, const ANPEvent*)
I/GeckoAppShell(20362): post to main java thread
I/GeckoPlugins(20362): returning from void anp_event_postEvent(NPP_t*, const ANPEvent*)
I/GeckoAppShell(20362): run GeckoRunnableCallback
I/GeckoJNI(20362): void Java_org_mozilla_gecko_GeckoAppShell_executeNextRunnable(JNIEnv*, _jclass*)
I/GeckoJNI(20362): leaving void Java_org_mozilla_gecko_GeckoAppShell_executeNextRunnable(JNIEnv*, _jclass*)
I/GeckoApp(20362): resume
D/GeckoInputConnection(20362): IME: handleCreateInputConnection called
D/GeckoInputConnection(20362): notifyIME
D/GeckoInputConnection(20362): notifyIME v!= null
D/GeckoInputConnection(20362): notifyIME = reset
I/GeckoApp(20362): Got message: Tab:Selected
I/GeckoApp(20362): Switched to tab: 1
I/GeckoDoorHangerPopup(20362): Showing all doorhangers for tab: 1
V/orca:OrcaApplication(20481): Forcing initialization of AsyncTask
D/GeckoInputConnection(20362): IME: run()
D/GeckoInputConnection(20362): IME: v=org.mozilla.gecko.gfx.LayerView@405a8210
D/GeckoInputConnection(20362): IME: handleCreateInputConnection called
I/GeckoApp(20362): Got message: Content:StateChange
I/GeckoApp(20362): State - 983041
I/GeckoApp(20362): Got a document start
I/GeckoAppShell(20362): remove view:com.adobe.flashplayer.FlashPaintSurface@40777338
I/GeckoPlugins(20362): void anp_event_postEvent(NPP_t*, const ANPEvent*)
I/GeckoAppShell(20362): post to main java thread
I/GeckoPlugins(20362): returning from void anp_event_postEvent(NPP_t*, const ANPEvent*)
I/GeckoAppShell(20362): run GeckoRunnableCallback
I/GeckoJNI(20362): void Java_org_mozilla_gecko_GeckoAppShell_executeNextRunnable(JNIEnv*, _jclass*)
I/GeckoJNI(20362): leaving void Java_org_mozilla_gecko_GeckoAppShell_executeNextRunnable(JNIEnv*, _jclass*)
I/DEBUG   (20365): debuggerd committing suicide to free the zombie!
I/DEBUG   (20488): debuggerd: Sep 20 2011 11:11:25
I/ActivityManager(  110): Process org.mozilla.fennec_sriram (pid 20362) has died.
I/WindowManager(  110): WIN DEATH: Window{407d7d60 org.mozilla.fennec_sriram/org.mozilla.fennec_sriram.App paused=false}
I/WindowManager(  110): WIN DEATH: Window{40916d40 SurfaceView paused=false}

Comment 2

[Sriram Ramasubramanian [:sriram]]

I tried reproducing the crash quite few times. Every time a page with flash gets switched, the flash "view" is removed and added-back-and-painted again. This during fast switch causes the crash.

Adding an "id" to the LayerView, made android "save" the view, which prolonged the death of the process. However, I was able to see the flash view persistent across tabs in that case ["Breaking news" from CNN was still seen in MSNBC]. Hence, this cannot be a better approach.

There should be a way to unload and load Flash / or cache them, which can solve this issue.

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

Aaron - Can you retest? We have been landing fixes in this area

Comment 4

[Aaron Train [:aaronmt]]

Old code - WFM.