Closed Bug 729509 Opened 12 years ago Closed 12 years ago

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

Categories

(Core :: DOM: Device Interfaces, defect)

Product:
Core
Component:
DOM: Device Interfaces
Platform:
ARM
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WORKSFORME

People

(Reporter: ferjm, Assigned: ferjm)

References

Details

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.
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: nobody → ferjmoreno
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
Closed: 12 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.