1046068 - [B2G][MMS] The ReadReport Request for Sending MMS is enabled after 1st OOBE or factory reset but in Message Settings UI, it is disabled visually

Status: RESOLVED FIXED

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

Description

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

STR:
1. Flash build with "make reset-gaia"
2. Go to Message Settings, then you will found that |Read reports| is disabled.
3. Send a MMS to other reference device.
4. Reference device will try to send the read report back to the sender.

How to recover:
1. Toggle Read report twice in Message Settings.

Root cause:
1. When implementing Bug 921919, the default value of the pref "dom.mms.requestReadReport" in gecko was set to true.
2. the settings of 'ril.mms.requestReadReport.enabled' won't be set when 1st OOBE.

Short-term solution:
change default value of "dom.mms.requestReadReport" in gecko to false.

Long-term solution:
1. Need Gaia to provide the default value into settings database.
2. always get this setting from Settings database.

Comment 1

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

[Blocking Requested - why for this release]:
The behavior of Read Report for MMS is wrong if user never change it from Message Settings.

Comment 2

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

After further analysis, I found that 
1. In gecko side (b2g/chrome/content/settings.js), we will try to read the 'ril.mms.requestReadReport.enabled' when OOBE.
2. However, the value of 'ril.mms.requestReadReport.enabled' is undefined in this case, then the value will be set to the default value(true) defined in gecko.
3. Comparing with "ril.sms.requestStatusReport.enabled", gaia provides the default value in gaia/build/config/common-settings.json. Hence there is no such problem for Delivery Report.

Hi Arthur,
Per discussion off-line, I'd like to transfer this bug to you to have default value of 'ril.mms.requestReadReport.enabled' in gaia.

Thanks!

Comment 3

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

Attachment: Link to https://github.com/mozilla-b2g/gaia/pull/22382

EJ, could you help review this simple patch? Thanks.

Comment 4

[EJ Chen [:eragonj][:小龍哥][Contributor mode on](inactive)]

Comment on attachment 8465930 [details]
Link to https://github.com/mozilla-b2g/gaia/pull/22382

Thanks Arthur ! r+

Comment 5

[Marvin Khoo [:Marvin_Khoo]]

triage result, 1.4+

Comment 6

[Evelyn Hung]

Merged into master: 
https://github.com/mozilla-b2g/gaia/commit/0b626971d4d5334bf3ed615a76d33e735a386dda

Comment 7

[Evelyn Hung]

v1.4
https://github.com/mozilla-b2g/gaia/commit/9377274b17200a60cebcd2427d489a7756c4cc72

2.0
https://github.com/mozilla-b2g/gaia/commit/b9d7489abe1aaf81a1b6493043d1160e1abe14c2