[WebSMS] Android backend: Add 'read' attribute to nsIDOMMozSmsMessage

RESOLVED FIXED in Firefox 44

Status

()

Core
DOM: Device Interfaces
RESOLVED FIXED
6 years ago
3 years ago

People

(Reporter: ferjm, Assigned: reuben)

Tracking

Trunk
mozilla44
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox44 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

6 years ago
As a follow-up for bug 733320, we need to add the `read` attribute also on Android backend.
(Reporter)

Updated

6 years ago
Blocks: 733320
(Assignee)

Comment 1

3 years ago
Created attachment 8650754 [details] [diff] [review]
Implement MarkMessageRead, ignore default app restrictions

Android 4.4+ only lets apps set as the default Messaging app write to the SMS Provider. This is a UX problem, for now this patch ignores the case of not having permission. The Messages app will need some sort of read-only UI, or maybe we can only show it on the homescreen if we're the default app.
Assignee: nobody → reuben.bmo
(Assignee)

Comment 2

3 years ago
(The same problem applies to delete)
(Assignee)

Updated

3 years ago
Blocks: 1212679
(Assignee)

Comment 3

3 years ago
Created attachment 8671682 [details] [diff] [review]
Implement MarkMessageRead, ignore default app restrictions
Attachment #8650754 - Attachment is obsolete: true
Attachment #8671682 - Flags: review?(snorp)
Attachment #8671682 - Flags: review?(snorp) → review+
https://hg.mozilla.org/mozilla-central/rev/0519f888e933
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox44: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in before you can comment on or make changes to this bug.