1091494 - [Flame][v2.1][Message]"Delivery reports" function does not work.

Status: RESOLVED WORKSFORME

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

Description

[Alissa（leave from Mozilla）]

[1.Description]:
There is no delivery reports after you check this function in message settings.
Occurence time:15:00
see attachments:logcat2.txt,VID_0007_Compress.MP4
[2.Testing Steps]: 
1.Launch message.
2.Tap menu at right top.
3.Tap settings.
4.Check Delivery Reports.
5.Back to main view and send a MMS or SMS.


[3.Expected Result]: 
5. Sender will not receive any delivery report.

[4.Actual Result]: 
5.The sender should receive a delivery report.

[5.Reproduction build]: 
Gaia-Rev        eb0aab0f13c78c7ac378ad860e865c4b6eaf669f
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/318019f80a8e
Build-ID        20141029001202
Version         34.0

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

Comment 1

[Alissa（leave from Mozilla）]

Attachment: VID_0007_Compress.MP4

Comment 2

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

Hi,

Set NI to reporter to clarify the following questions:
1. Does this only happened when sending MMS?
(According to the video, the delivery report was not received in the sent MMS message)
2. Did you check if the remote party correctly download the entire MMS message sent from the test device?
   The delivery report will only be replied by remote party after the MMS was downloaded by the remote party.
3. If the issue is still happened after the message was downloaded in the remote side,
   Can you help to collect the tcpdump/adb logcat for further analysis?
   https://github.com/bevis-tseng/Debug_Tools

Thanks!

Comment 3

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

per comment 2.

Comment 4

[Julien Wajsberg [:julienw] (PTO -> Aug 14)]

And also: does it work with the same SIM on a non-Firefox OS device?

Comment 5

[Alissa（leave from Mozilla）]

We can receive "Delivery reports" with the same SIM in android device.

Comment 6

[Alissa（leave from Mozilla）]

Attachment: 1091494capture.pcap

Comment 7

[Alissa（leave from Mozilla）]

Attachment: 1091494logcat.txt

Comment 8

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

From the log, 
1. the delivery report with positive response was received and update into the MobileMessageDB.
2. The corresponding 'sms-delivery-success' was broadcast to application layer.

Don't know why the delivery report was not displayed in the UI.

Set NI to myself to have more investigation later.

Comment 9

[Julien Wajsberg [:julienw] (PTO -> Aug 14)]

QA, can you please try on v2.1 and v2.2 on your side?

Comment 10

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

I was not able to reproduce this in local carrier(CHT).

I'd like to reproduce this symptom locally by feeding tester's raw data into emulator,
NI reporter to:
(With the enable_debug_flags.sh from https://github.com/bevis-tseng/Debug_Tools applied to the test device)
1. collect both main log and radio log at the same time by opening 2 consoles and executing:
   - [main log]: adb logcat -v threadtime > logcat.txt
   - [radio log]: adb logcat -v threadtime -b radio > logcat_radio.txt
2. capture the tcpdump again.

Thanks!

Comment 11

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

set NI for the help on comment 10.

Comment 12

[Alissa（leave from Mozilla）]

Attachment: capture.pcap

There is no "Delivery reports" in notification,but there is a green "√" icon after message,please comfirm if the green "√" icon is the delivery reports you need.
update the log.

Comment 13

[Alissa（leave from Mozilla）]

Attachment: logcat.txt

Comment 14

[Alissa（leave from Mozilla）]

Attachment: logcat_radio.txt

Comment 15

[Julien Wajsberg [:julienw] (PTO -> Aug 14)]

(In reply to Alissa from comment #12)
> Created attachment 8516448 [details]
> capture.pcap
> 
> There is no "Delivery reports" in notification,but there is a green "√" icon
> after message,please comfirm if the green "√" icon is the delivery reports
> you need.
> update the log.

Yep, that's it; we don't send a notification for this, we only update the message live.

Comment 16

[Julien Wajsberg [:julienw] (PTO -> Aug 14)]

So is it a misunderstanding of the feature?

Maybe the moztrap is wrong because a notification was due at some time (see bug 933133) but over time we felt it was not really urgent (or even needed) so we never implemented it.

NI Jenny/Eric to comment here.

Comment 17

[Eric Chang [:ericcc] [:echang]]

https://moztrap.mozilla.org/manage/cases/?pagenumber=1&pagesize=100&sortfield=name&sortdirection=asc&filter-id=11014
The test case looks okay, but I think we can change some wording..

Comment 18

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

Not a RIL-related issue anymore according comment 12.

Comment 19

[Jenny Lee]

Hi Alissa, 
Please see Bug 933133 comment 20 for my thought on this, as mentioned in the comment, I don't think it's necessary to show delivery notification in notification tray. Let me know if you think otherwise, thanks!

Hello Julien,
With that being said, I think we should modify the string in Message setting to reflect actual state, but that would be a different bug =)
Original: "Request notification of delivery for each message sent."
Modified: "Request delivery report for each message sent."   
Thanks!

Comment 20

[Julien Wajsberg [:julienw] (PTO -> Aug 14)]

From comment 12, let's close this bug. It's actually working :)

Comment 21

[Julien Wajsberg [:julienw] (PTO -> Aug 14)]

Filed bug 1094074 about comment 19

Comment 22

[Joshua Mitchell (Inactive)]

issue resolved - removing keywords