1105897 - [FFOS7715 v2.1][lockscreen] after monkey test phone can't unlock

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::System::Lockscreen, defect)

Description

[ben.song]

Reproduce step:

1.phone start with 12 hours monkey test.

Ideal result:

Phone could unlock successfully.

Actual result:

Phone can't unlock until it be restart.

Reproduce rate: 4/10

Comment 1

[ben.song]

Base on the logs, we could find event of touchmove havn't be triggered in lockscreen.js of system module.

To debug the touch operation,we could find the diference bettween normal touch operation and unlock touch operation.

That is nsWindow::DispatchInputEvent in gecko/widget/gonk/nsWindow.cpp. While normal touch operation occurs, TabParent::GetEventCapturer() would be true, and TabParent::TryCapture in gecko/dom/ipc/TabParent.cpp would be invoken; but during unlock touch operation, TabParent::GetEventCapturer() would be false. In this bug, we could see TabParent::GetEventCapturer() be true during unlock.

Moreover, I have a question, during normal touch operation, in TabParent::TryCapture of gecko/dom/ipc/TabParent.cpp, If the variable of mEventCaptureDepth could less than 0 ?

Comment 2

[Vance Chen [:vchen][skype:vance.lucida][[e-mail]]]

Hi Song Ben -

Is it possible you can share your monkey script to us such that we can run it in our office as well?

Thanks

Vance

Comment 3

[ben.song]

Attachment: monkey.test.zip

Dear Vance,

This attachment contains our monkey test script.

If you would run monkey.test/test-config/run-7715_v2.1-hudson.sh, you should flash a version into the phone first.

Thanks.

Comment 4

[Vance Chen [:vchen][skype:vance.lucida][[e-mail]]]

Thanks Song Ben, but i keep getting this error while running the script:

[ERROR]No external sdcard, test is stopped.
[ERROR]./_monkey.sh(61) dev check


I am sure i already insert a SD card in the device...Any suggestions?

Comment 5

[ben.song]

(In reply to Vance Chen [:vchen][[e-mail]] from comment #4)
> Thanks Song Ben, but i keep getting this error while running the script:
> 
> [ERROR]No external sdcard, test is stopped.
> [ERROR]./_monkey.sh(61) dev check
> 
> 
> I am sure i already insert a SD card in the device...Any suggestions?

Dear Vance,

We use '$($ADB shell mount | grep -P "^/dev/block/vold/" | wc -l) -lt 2' to check sdcard is or not exist in /monkey.test/_dev_check.sh. Does your sdcard be saved at the directory of "^/dev/block/vold/" ?

Thanks.

Comment 6

[Vance Chen [:vchen][skype:vance.lucida][[e-mail]]]

(In reply to ben.song from comment #5)
> (In reply to Vance Chen [:vchen][[e-mail]] from comment #4)
> > Thanks Song Ben, but i keep getting this error while running the script:
> > 
> > [ERROR]No external sdcard, test is stopped.
> > [ERROR]./_monkey.sh(61) dev check
> > 
> > 
> > I am sure i already insert a SD card in the device...Any suggestions?
> 
> Dear Vance,
> 
> We use '$($ADB shell mount | grep -P "^/dev/block/vold/" | wc -l) -lt 2' to
> check sdcard is or not exist in /monkey.test/_dev_check.sh. Does your sdcard
> be saved at the directory of "^/dev/block/vold/" ?
> 
> Thanks.

Hum not sure how to check if the sdcard is saved at the directory of "^/dev/block/vold/", however, indeed there is a folder "/dev/block/vold/" in the device, but inside that only get the following 2 files:

179:0
179:1

Not sure what to do from here...

Comment 7

[ben.song]

(In reply to Vance Chen [:vchen][[e-mail]] from comment #6)
> (In reply to ben.song from comment #5)
> > (In reply to Vance Chen [:vchen][[e-mail]] from comment #4)
> > > Thanks Song Ben, but i keep getting this error while running the script:
> > > 
> > > [ERROR]No external sdcard, test is stopped.
> > > [ERROR]./_monkey.sh(61) dev check
> > > 
> > > 
> > > I am sure i already insert a SD card in the device...Any suggestions?
> > 
> > Dear Vance,
> > 
> > We use '$($ADB shell mount | grep -P "^/dev/block/vold/" | wc -l) -lt 2' to
> > check sdcard is or not exist in /monkey.test/_dev_check.sh. Does your sdcard
> > be saved at the directory of "^/dev/block/vold/" ?
> > 
> > Thanks.
> 
> Hum not sure how to check if the sdcard is saved at the directory of
> "^/dev/block/vold/", however, indeed there is a folder "/dev/block/vold/" in
> the device, but inside that only get the following 2 files:
> 
> 179:0
> 179:1
> 
> Not sure what to do from here...

Dear Vance,

Can you start settings app and see in Media Storage whether or not discriminate this sdcard. If not there is something wrong in your phone or sdcard. 

At the same time，I would communicate with my colleague who in charge of monkey script.

Thanks.

Comment 8

[ben.song]

Dear Vance,

You could annotate the following lines in monkey.test/_dev_check.sh. It would be OK.

#check sdcard, if no sdcard, then refuse to test
if [ $($ADB shell mount | grep -P "^/dev/block/vold/" | wc -l) -lt 2 ]
then
    if [ "$MTCFG_TEST_NEED_SDCARD" == "YES" ]
    then
        log "[ERROR]No external sdcard, test is stopped."
        exit 1
    else
        log "[WARNING]No external sdcard..."
    fi
fi

Comment 9

[ben.song]

Dear Vance,

Have you reproduce this bug after monkey test?

Thanks.

Comment 10

[Vance Chen [:vchen][skype:vance.lucida][[e-mail]]]

(In reply to ben.song from comment #9)
> Dear Vance,
> 
> Have you reproduce this bug after monkey test?
> 
> Thanks.

Hi Song Ben -

Sorry we are all in the workweek this week so I don't have time to check this yet. Will get back to you next week regarding if i can reproduce this in our end

Thanks

Comment 11

[ben.song]

(In reply to Vance Chen [:vchen][[e-mail]] from comment #10)
> (In reply to ben.song from comment #9)
> > Dear Vance,
> > 
> > Have you reproduce this bug after monkey test?
> > 
> > Thanks.
> 
> Hi Song Ben -
> 
> Sorry we are all in the workweek this week so I don't have time to check
> this yet. Will get back to you next week regarding if i can reproduce this
> in our end
> 
> Thanks

Dear Vance,

OK,thanks.

Comment 12

[Steven Yang [:styang]]

Danny, please help to check logs.

Comment 13

[ben.song]

(In reply to Steven Yang [:styang] from comment #12)
> Danny, please help to check logs.

Dear Danny,Steven,

I have got a patch to fix this problem as below for I found sending a touchstart during a real touchend could results into a busted mEventCaptureDepth as -1000 and let sEventCapturer could not be nullptr in TabParent::InjectTouchEvent.

diff --git a/dom/ipc/TabParent.cpp b/dom/ipc/TabParent.cpp
index a311f9a..c8d812e 100644
--- a/dom/ipc/TabParent.cpp
+++ b/dom/ipc/TabParent.cpp
@@ -1032,6 +1032,19 @@ TabParent::TryCapture(const WidgetGUIEvent& aEvent)
   if (event.message == NS_TOUCH_START || isTouchPointUp) {
     // Let the DOM see touch start/end events so that its touch-point
     // state stays consistent.
+    // sprdbug381835 during touchstart touchcancel occurs, we would directly
+    // return false @{
+    if (event.message == NS_TOUCH_CANCEL && !sEventCapturer) {
+      return false;
+    }
+    // during drop-down statusbar,to invoke InjectTouchEvent function may
+    // results into a busted mEventCaptureDepth, so we need fix value of
+    // mEventCaptureDepth
+    if (isTouchPointUp && mEventCaptureDepth == 0) {
+      sEventCapturer = nullptr;
+      return false;
+    }
+    //@}
     if (isTouchPointUp && 0 == --mEventCaptureDepth) {
       // All event series are un-captured, don't try to catch any
       // more.

Comment 14

[Danny Liang [:dliang]]

Hi Ben,
The log you attached is invalid, it looks like script of monkey test. Please share available log for analysis. Moreover, is above patch in your patch folder? May I know how many patches you applied on your branch? Thanks,

Comment 15

[ben.song]

(In reply to Danny Liang [:dliang] from comment #14)
> Hi Ben,
> The log you attached is invalid, it looks like script of monkey test. Please
> share available log for analysis. Moreover, is above patch in your patch
> folder? May I know how many patches you applied on your branch? Thanks,

Dear Danny,

The attachment is script of monkey test, I would find the log of unlock problem for it has been resolved a month.

In sprd v2.1 branch there are eighty patches now.

Thanks.

Comment 16

[ben.song]

Dear Danny,

I have uploaded logs to :ftp://ftp.spreadtrum.com/7715/1105897
username:mouzhi
password:mouZHI$$61

Thanks.

Comment 17

[BMO Automation]

Firefox OS is not being worked on