895734 - WebSMS: accept array for SmsFilter::threadId and make delete() accept a SmsFilter

Status: RESOLVED INCOMPLETE

(Core :: DOM: Device Interfaces, defect)

Description

[Vicamo Yang [:vicamo][:vyang]]

From bug 771458, we accept an array of message IDs in delete() call, and it accelerates message deletion of the same thread a lot.  But when it comes to delete messages of many threads, Gaia has to call |getMessages()| for each of the target threads, extract message IDs and call |delete()|.  Bug 890921 proposes a new |deleteThreads()| call to address this problem, but maybe we can reuse the existing API |delete()|.

Comment 1

[Vicamo Yang [:vicamo][:vyang]]

Suddenly find this might be a bad idea for long term because W3C[1] has |deleteConversation(conversationID)| instead.

[1]: http://messaging.sysapps.org/#widl-MessagingManager-deleteConversation-Future-DOMString-conversationID

Comment 2

[Julien Wajsberg [:julienw] (PTO -> Mar 17)]

but if we find that "deleteConversation(conversationID)" is not good enough we should maybe contribute this ?

Comment 3

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

FxOS/Gonk has been removed from the codebase. Mass-invalidating FxOS related Device Interface bugs to clean up the component. 

If I incorrectly invalidated something, please let me know.

Comment 4

[Firefox Bug Husbandry Bot]

Bulk correction of resolution of B2G bugs to INCOMPLETE.