B2G SMS: Refactor SmsDatabaseService.createMessageList to make all requests within one transaction

RESOLVED WORKSFORME

Status

()

Core
DOM: Device Interfaces
RESOLVED WORKSFORME
6 years ago
6 years ago

People

(Reporter: ferjm, Assigned: ferjm)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

6 years ago
We should do the whole search in one transaction and not split it over multiple
ones. There´s the possibility of an incoming sms between transactions.
(Assignee)

Updated

6 years ago
Blocks: 712809
Component: General → DOM: Device Interfaces
Product: Boot2Gecko → Core
QA Contact: general → device-interfaces
Summary: [B2G] Refactor createMessageList from SmsDatabaseService to make all requests within a transaction → B2G SMS: Refactor SmsDatabaseService.createMessageList to make all requests within one transaction
(Assignee)

Updated

6 years ago
Assignee: nobody → ferjmoreno
(Assignee)

Comment 1

6 years ago
If I am not wrong, the actual implementation of SmsDatabaseService.createMessageList already does all requests within the same transaction, so this could probably be closed.
This is already addressed in the latest revision of the patch in bug 712809, so I don't think there'll be anything to do here once that lands.
Ok, we agree then :)
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.