908124 - An error message is shown after successfully open a received photo and click 'home' key.

Status: RESOLVED FIXED

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

Comment 1

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

I'm not able to reproduce the issue on mozilla-central-inari-eng. Test environment is as following:

Build: Gaia::master / Gecko::mozilla-central, id: 20130825040201
Device: inari

I will do it again on Gaia::v1-train / Gecko::b2g18.

Comment 2

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

I'm able to reproduce the issue on Gaia::v1-train / Gecko::b2g18 . And no need to share a photo from Facebook app.

The root cause is received 'ActivityCanceled' event which is fired via Gecko and handled in activity.onerror scope. Then, Bluetooth file transfer don't identify the error case. And pop out the error dialog with unknown media message. 

Since Bug 838898 is fixed in Gaia:master, so that we're able to reproduce it on Gaia::v1-train only. But the fixing patch is handling the 'NO_PROVIDER' only. I would like to give a patch to handle it and 'ActivityCanceled' clearly. The 'ActivityCanceled' event will be triggered while requesting an activity and a user click 'home' key.

Nominating leo+..

Comment 3

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

Attachment: Pointer to Github pull request 11752.html

Alive,

Could you please help to review my pr? Thanks.

Comment 4

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 795327 [details]
Pointer to Github pull request 11752.html

r=me

Comment 5

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

Thanks for Alive's reviewing effort.

Gaia/master:: c229460290fb34f6e7d050b354e5ffb09a4f9cdb

Wait the nominating process for landing on v1-train. Then, we can close the issue.

Comment 7

[bhavana bajaj [:bajaj]]

We discussed in triage that it may be a recent regression, patch is very straight forward, hence leo+

Comment 8

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

Gaia/v1-train:: 577ab5105af418efc8ded00798fd3644b3dac674

Shell we uplift to v1.1.0hd? Nominating hd+, since the patched landed on v1-train.

Comment 9

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

v1.1.0hd: 577ab5105af418efc8ded00798fd3644b3dac674

Comment 10

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

What is needed to close this bug?

Comment 11

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

I think we don't need other patch for this bug. Since the patch is landed on v1.1.0hd, we can close the issue now. Thanks for Tim's reminder and uplifting process.:)

Comment 12

[Wayne Chang [:wchang]]

This was leo+, leo+ already covers hd+.
Setting back to leo+.