1000703 - [tarako] a white screen appears after launch sms and send sms to background for several times

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::SMS, defect)

Description

[Peipei Cheng (needinfo if you need my action)]

Attachment: adb log

Steps to reproduce
------------------------------------------------------------
1. Launch SMS
2. Before SMS UI is generated, tap on Home button to send SMS to background
3. Repeat 1~2 for 2/3 times
4. Launch SMS again
   --> A white screen shows up. Tap on Home button can recover from it.

According to yang.zhao, this white screen is the background color of SMS iframe. And window_manager didn't removeFrame after invoke closeWindow.

Attached is adb log. test starts: 04-24 05:26:59.562

Test build
------------------------------------------------------------------------
Gaia      3771067de006633df690a590a97b4d28c44ef8b2
Gecko     https://hg.mozilla.org/releases/mozilla-b2g28_v1_3t/rev/51a4e4d35e30
BuildID   20140423164005
Version   28.1
ro.build.version.incremental=eng.cltbld.20140423.203951
ro.build.date=Wed Apr 23 20:39:59 EDT 2014

Comment 1

[Julien Wajsberg [:julienw]]

Isn't this bug 991577 already?

Comment 2

[Peipei Cheng (needinfo if you need my action)]

Yes. It's the same with bug 991577

Comment 3

[yang.zhao]

(In reply to Julien Wajsberg [:julienw] from comment #1)
> Isn't this bug 991577 already?
Yes.The white screen is the background of the iframe in system.
I think there is something wrong in the control of window_manager.
I found when the sms was killed in background,then relaunch again,the killed process hasn't done compeletly.After the function "closeWindow",there should be "removeFrame" but didn't .
You could find the white screen in other apps,such as settings.

Comment 4

[Julien Wajsberg [:julienw]]

Yes, but that's been fixed in master, it's happenng in v1.3 only, and as far as I know the peers don't want to do the work for v1.3 now.

Comment 5

[yang.zhao]

(In reply to Julien Wajsberg [:julienw] (away May 8th) from comment #4)
> Yes, but that's been fixed in master, it's happenng in v1.3 only, and as far
> as I know the peers don't want to do the work for v1.3 now.

Hi,Julien
     bug 991577 has duplicated to bug935750, but this bug is not the same with bug 935750.When the white screen happens,you could see two Preallocated ps.Could you find someone to see this issue?Why there is two Preallocated ps here?Thank you.


APPLICATION      USER     PID   PPID  VSIZE  RSS     WCHAN    PC         NAME
b2g              root      86    1     144476 46288 ffffffff 400cc518 S /system/b2g/b2g
(Nuwa)           root      347   86    50560  6272  ffffffff 400c6518 S /system/b2g/plugin-container
Homescreen       app_391   391   347   68108  18220 ffffffff 400c6518 S /system/b2g/plugin-container
(Preallocated a  app_787   787   347   57724  10472 ffffffff 400c6518 S /system/b2g/plugin-container
(Preallocated a  root      1108  347   58744  15088 ffffffff 400c6518 S /system/b2g/plugin-container

Comment 6

[yang.zhao]

Jesse said:
  When the issue occurs,there were two Preallocated ps,that means,the sms app already was killed and restarted ,but maybe because the low memory ,the new sms ps couldn't initialize sucessfully.Need Fabrice 
to see which step blocks the Preallocated.

Comment 7

[Julien Wajsberg [:julienw]]

I can reopen but I won't help much here; Fabrice knows better than me.

Comment 8

[James Zhang (Spreadtrum)]

(In reply to Julien Wajsberg [:julienw] (away May 8th) from comment #4)
> Yes, but that's been fixed in master, it's happenng in v1.3 only, and as far
> as I know the peers don't want to do the work for v1.3 now.

Fabrice, help.

Comment 9

[yang.zhao]

the log before the issue occurs:

E/GeckoConsole( 2805): Content JS LOG at app://system.gaiamobile.org/js/window_manager.js:336 in windowClosed: windowClosed dispatch appclose event
I/Gonk    ( 2805): Setting nice for pid 3042 to 18
I/Gonk    ( 2805): Changed nice for pid 3042 from 1 to 18.
I/Gonk    ( 2805): Setting nice for pid 3042 to 1
I/Gonk    ( 2805): Changed nice for pid 3042 from 18 to 1.
I/Gecko   ( 2805): [Parent 2805] WARNING: pipe error (113): Connection reset by peer: file /home/yangzhao/B2G/B2G6821/gecko/ipc/chromium/src/chrome/common/ipc_channel_posix.cc, line 446
I/Gecko   ( 2805): 
I/Gecko   ( 2805): ###!!! [Parent][MessageChannel] Error: Channel error: cannot send/recv
I/Gecko   ( 2805): 
I/Gecko   ( 2805): -*- RadioInterfaceLayer: Received 'child-process-shutdown' message from content process
E/OomLogger( 2805): [Kill]: lowmem_shrink select 3042 (Messages), adj 10, size 7774, to kill
E/OomLogger( 2805): [Kill]: lowmem_shrink send sigkill to 3042 (Messages), adj 10, size 7774
I/Gecko   ( 2805): [Parent 2805] WARNING: waitpid failed pid:3042 errno:10: file /home/yangzhao/B2G/B2G6821/gecko/ipc/chromium/src/base/process_util_posix.cc, line 254

Comment 10

[James Zhang (Spreadtrum)]

Is bug 1005787 can fix this issue?

Comment 11

[[:fabrice] Fabrice Desré]

(In reply to yang.zhao from comment #5)
> 
> APPLICATION      USER     PID   PPID  VSIZE  RSS     WCHAN    PC         NAME
> b2g              root      86    1     144476 46288 ffffffff 400cc518 S
> /system/b2g/b2g
> (Nuwa)           root      347   86    50560  6272  ffffffff 400c6518 S
> /system/b2g/plugin-container
> Homescreen       app_391   391   347   68108  18220 ffffffff 400c6518 S
> /system/b2g/plugin-container
> (Preallocated a  app_787   787   347   57724  10472 ffffffff 400c6518 S
> /system/b2g/plugin-container
> (Preallocated a  root      1108  347   58744  15088 ffffffff 400c6518 S
> /system/b2g/plugin-container

Actually it seems that only one pre-allocated app is running (PID 1108). The one with PID 787 has a new user ID, but we failed to change the process name.

I'd like QA to verify now that bug 1005787 has landed. If we still hit this bug we'll investigate on the gecko side.

Comment 12

[yang.zhao]

(In reply to Fabrice Desré [:fabrice] from comment #11)
> (In reply to yang.zhao from comment #5)
> > 
> > APPLICATION      USER     PID   PPID  VSIZE  RSS     WCHAN    PC         NAME
> > b2g              root      86    1     144476 46288 ffffffff 400cc518 S
> > /system/b2g/b2g
> > (Nuwa)           root      347   86    50560  6272  ffffffff 400c6518 S
> > /system/b2g/plugin-container
> > Homescreen       app_391   391   347   68108  18220 ffffffff 400c6518 S
> > /system/b2g/plugin-container
> > (Preallocated a  app_787   787   347   57724  10472 ffffffff 400c6518 S
> > /system/b2g/plugin-container
> > (Preallocated a  root      1108  347   58744  15088 ffffffff 400c6518 S
> > /system/b2g/plugin-container
> 
> Actually it seems that only one pre-allocated app is running (PID 1108). The
> one with PID 787 has a new user ID, but we failed to change the process name.
> 
> I'd like QA to verify now that bug 1005787 has landed. If we still hit this
> bug we'll investigate on the gecko side.

It still happens after bug 1005787 landed.In fact ,the log in comment 9 and the |adb shell b2g-ps| are all get with bug 1005787 patch.From the log in comment 9 ,it seems that the appclose event hadn't sended out because the sms was killed by LMK,so 'this._transitionState' in window.js got a wrong state.Could you help to check this again ?Thank you!

Comment 13

[[:fabrice] Fabrice Desré]

I can't really help since I'm unable to reproduce.

Comment 14

[Julien Wajsberg [:julienw]]

Mass closing of Gaia::SMS bugs. End of an era :(

Comment 15

[Julien Wajsberg [:julienw]]

Mass closing of Gaia::SMS bugs. End of an era :(