1214557 - [B2GDroid] Can't send a message via B2GDroid's default message app.

Status: RESOLVED FIXED

(B2GDroid Graveyard :: General, defect)

Description

[Verson Xiong (Leave from Mozilla)]

Attachment: logcat_0056.txt

[1.Description]:
When user taps message's send icon to send a message, they will find device has no response, message can't be sent.
Found time:00:56
See attcchment:AriesKK_0056.3gp & logcat_0056.txt

[2.Testing Steps]: 
Prerequisite:Insert a valid SIM card.
1. Open Message app.
2. Tap on the Compose icon.
3. Enter the recipient and content.
4. Tap send icon.
 
[3.Expected Result]: 
4.Message can be sended successfully.

[4.Actual Result]: 
4.Device hasn't any response, and message can't be sended.

[5.Reproduction build]: 
Android version: 4.4.4(Build Number 23.0.A.2.93)
B2GDroid version:0.11

[6.Reproduction Frequency]: 
Always Recurrence,5/5

[7.TCID]: 
10742

[8.Note]:
This function works normally on latest build of flame v2.5 & aries v2.5.

Comment 1

[Verson Xiong (Leave from Mozilla)]

Attachment: AriesKK_0056.3gp

Comment 2

[Reuben Morais [:reuben]]

The problem seems to be somewhere in Gaia. navigator.mozMobileMessage.send never gets called.

Comment 3

[Reuben Morais [:reuben]]

Julien (please redirect if you're not the right person), I've been looking at this and I can't make sense of what the app is doing. The event handler for the send button never actually sends a message:

https://github.com/mozilla-b2g/gaia/blob/master/apps/sms/views/conversation/js/conversation.js#L2236

The MultiSimActionButton click callback does get called, but bails out because navigator.mozIccManager and its "phoneNumberGetter" are undefined.

How exactly do we end up calling navigator.mozMobileMessage.send? Or is mozIccManager being undefined likely to be the problem?

Comment 4

[Julien Wajsberg [:julienw]]

(In reply to Reuben Morais [:reuben] from comment #3)
> Julien (please redirect if you're not the right person), I've been looking
> at this and I can't make sense of what the app is doing. The event handler
> for the send button never actually sends a message:
> 
> https://github.com/mozilla-b2g/gaia/blob/master/apps/sms/views/conversation/
> js/conversation.js#L2236
> 
> The MultiSimActionButton click callback does get called, but bails out
> because navigator.mozIccManager and its "phoneNumberGetter" are undefined.
> 
> How exactly do we end up calling navigator.mozMobileMessage.send? Or is
> mozIccManager being undefined likely to be the problem?

Yes :)

When MultiSimActionButton succeeds, we end up calling [1].

[1] https://github.com/mozilla-b2g/gaia/blob/ef8d5801f34219d7162415f56e6811a0e7cae305/apps/sms/views/conversation/js/conversation.js#L2264

On desktop we use [2] to simulate mozIccManager, maybe you'd like to reuse this ? Unless you plan to implement this API in B2GDroid too.

[2] https://github.com/mozilla-b2g/gaia/blob/ef8d5801f34219d7162415f56e6811a0e7cae305/apps/sms/desktop-mock/navigator_moz_icc_manager.js


Please ask if you have any more question :)

Comment 5

[Reuben Morais [:reuben]]

Attachment: Enable ICC Manager on B2GDroid

I looked into using the shim, but got lost in the Gaia build system. This patch, believe it or not, Just Works™.

The message is sent, but for some reason doesn't get saved. I'll look into that next.

Hmm, I should've thought of this earlier, but I bet I can just move the ICC manager shim out of the desktop-mock folder so it gets included all the time, and that should work.

Comment 6

[Julien Wajsberg [:julienw]]

Yes, for such a small script, I don't mind if it's always included ;)

Comment 7

[GitHub Autolander Bot]

Attachment: [gaia] reuben:bug1214557 > mozilla-b2g:master

Comment 8

[Reuben Morais [:reuben]]

Comment on attachment 8682486 [details] [review]
[gaia] reuben:bug1214557 > mozilla-b2g:master

Not sure if services/js/ is the best place to put this, let me know what you think.

Comment 9

[Julien Wajsberg [:julienw]]

Comment on attachment 8682486 [details] [review]
[gaia] reuben:bug1214557 > mozilla-b2g:master

Let's put it directly in /sms/mocks (instead of /sms/services/js).

r=me once it's there :)

Comment 10

[Julien Wajsberg [:julienw]]

Hey Reuben, just in case you forgot about this ;)

Comment 11

[Reuben Morais [:reuben]]

Hi Julien, I did not forget. I've been sick since the weekend and couldn't sit in front of the computer without feeling sick, but I'm getting better now and will I'll get to this today. Thanks for the reminder.

Comment 12

[Julien Wajsberg [:julienw]]

No problem, have a good rest and come back strong :)

Comment 13

[Reuben Morais [:reuben]]

Merged! https://github.com/mozilla-b2g/gaia/commit/bc1d39caef86be61b0d532becd37ea3bfea54a68

PS: Don't sign up for a bike race in the Brazilian summer heat unless you're very well trained.