1002406 - [Tarako][Bluetooth] Pairing dialog(attention screen) cannot be close after run on monkey test

Status: RESOLVED WORKSFORME

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

Description

[[e-mail], [e-mail]]

=== STR ===:
Run the build on monkey test.

=== Actual Result ===
Bluetooth pairing dialog cannot be close.

=== Expected Result ===
Even if the pairing process is unsuccessful, the pairing dialog should be closed normally. 

=== Build ===:
ro.build.version.incremental=eng.kaizhen.20140425.155902
ro.build.date=Fri Apr 25 15:59:09 CST 2014

Comment 1

[[e-mail], [e-mail]]

Attachment: 2014-09-13-11-10-36.png

Comment 2

[[e-mail], [e-mail]]

Attachment: 2014-09-13-11-10-47.png

Comment 3

[Steven Yang [:styang]]

Triage. 1.3T+

Comment 4

[[e-mail], [e-mail]]

Attachment: WIP 18739

* Check module is existed or not before call the method.

This is a WIP. And will request review after run on monkey test normally.

Comment 5

[[e-mail], [e-mail]]

Kai-Zhe, could you please help to run monkey test with the WIP?

Comment 6

[Kai-Zhen Li [:seinlin][:kli]]

OK, it will be included in 04/29 monkey test.

Comment 7

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Any update?

Comment 8

[Kai-Zhen Li [:seinlin][:kli]]

Tim, monkey test was failed due to another error in kernel. So not sure if attachment 8413671 [details] [review] fix the problem. There will be a 6 hours result this evening and a 12 hours result on tomorrow morning. I'll update the result then.

Comment 9

[Kai-Zhen Li [:seinlin][:kli]]

6 hours test is finished, 3/3 device passed the test. All devices do work properly after test.

Comment 10

[Kai-Zhen Li [:seinlin][:kli]]

14 hours test is also finished, 3/3 devices didn't hit this error.

Comment 11

[[e-mail], [e-mail]]

Kai-Zhen, thanks for your verification. Per offline discussion, I just rebase my patch. If we wonder that the patch is not relative with the issue. We might run the monkey test again without my patch. Then, we observer that issue is able to be reproduced.

Comment 12

[Kai-Zhen Li [:seinlin][:kli]]

Ian, 24 hours test is also finished. It's still not reproduced. 
If the patch is only doing good but no side effect, I think we should land it. 
How do you think?

Comment 13

[[e-mail], [e-mail]]

Comment on attachment 8413671 [details] [review]
WIP 18739

Kai-Zhen, thanks for your test with patience. I also agree with your suggestion since the patch is very low risk.

Nominate Arthur to be reviewer because he has reviewed the module before. Arthur, could you please help to review my pr? I add some check before call these function. Thanks.

Comment 14

[Arthur Chen [:arthurcc][inactive after 6/30]]

Comment on attachment 8413671 [details] [review]
WIP 18739

Is there any log supporting the patch? I was thinking the problem might only due to the racing between loading "bluetooth_helper.js" and "pair_manager.js".

Comment 15

[[e-mail], [e-mail]]

Per offline discussion with Kai-Zhen, the daily monkey test is not including of WIP here after 5/6. It means that the issue is no longer to be reproduced without any updated patch recently. I would like to change the bug status to be work for me. But, we can reopen it while the issue is able to reproduce again.