1220548 - Call state changes from DIALING to ALERTING automatically introduces some potential racing problem

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Emulator, defect)

Description

[Edgar Chen [:edgar]]

Call state changes from DIALING to ALERTING automatically [1] introduces some potential racing problem which is similar to https://bugzilla.mozilla.org/show_bug.cgi?id=823084#c4.

The intermittent failure (ScriptTimeout) in test_outgoing_from_stk.js [2], test_incoming_already_connected.js [3] and test_outgoing_radio_off.js [4] are caused by this.

[1] https://github.com/mozilla-b2g/platform_external_qemu/blob/5a50f96a1d7c788817abb7c57acbb75172c1f48d/telephony/android_modem.c#L3755-L3759
[2] https://treeherder.mozilla.org/logviewer.html#?job_id=12965156&repo=try
[3] https://treeherder.mozilla.org/logviewer.html#?job_id=12981645&repo=try
[4] https://treeherder.mozilla.org/logviewer.html#?job_id=13073521&repo=try

Comment 1

[Edgar Chen [:edgar]]

I would like to make the test script can tolerate such racing first. But still leave this bug open for further discussion about how to prevent such issue.

Comment 2

[Edgar Chen [:edgar]]

Attachment: MozReview Request: Bug 1220548 - Make telephony test script can tolerate the missing dailing state. r=bevis

Bug 1220548 - Make telephony test script can tolerate the missing dailing state

Comment 3

[Edgar Chen [:edgar]]

Comment on attachment 8681827 [details]
MozReview Request: Bug 1220548 - Make telephony test script can tolerate the missing dailing state. r=bevis

Bug 1220548 - Make telephony test script can tolerate the missing dailing state

Comment 4

[Bevis Tseng[:bevis][:btseng](Exited)]

Comment on attachment 8681827 [details]
MozReview Request: Bug 1220548 - Make telephony test script can tolerate the missing dailing state. r=bevis

https://reviewboard.mozilla.org/r/23913/#review21467

LGTM, thanks!

Comment 5

[Edgar Chen [:edgar]]

Comment on attachment 8681827 [details]
MozReview Request: Bug 1220548 - Make telephony test script can tolerate the missing dailing state. r=bevis

Bug 1220548 - Make telephony test script can tolerate the missing dailing state. r=bevis

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/b2g-inbound/rev/a26993f402b3

Comment 7

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/a26993f402b3

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/a26993f402b3

Comment 9

[Carsten Book [:Tomcat]]

removing the b2g 2.5 flag since this commit has been reverted due to an incorrect merge, sorry for the confusion

Comment 10

[BMO Automation]

Firefox OS is not being worked on