1056943 - Remove contact images from call log DB

Status: RESOLVED WONTFIX

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

Description

[Doug Sherk (:drs) (inactive)]

In bug 1045499, we removed contact photos from the call log. We should remove them from the call log DB as well.

Comment 1

[Anthony Ricaud (:rik)]

I'm having some difficulties with IndexedDB migrations and getting this error when just trying to log all the records in the groups store:
"A mutation operation was attempted on a database that did not allow mutations."

Comment 2

[Anthony Ricaud (:rik)]

I think we have an issue with our upgrade4. To re-populate the store, we need to query mozContacts. And that is async. So by the time we have all the information we wanted, the transaction given by upgradeneeded is closed. The current code opens a new one to store the new groups.

We haven't done any store or indexes change since upgrade4 but if we do, this will bite us because we're not in the transaction that can do that anymore.

Ben: Do you have any idea how we could avoid closing the upgradeneeded transaction while waiting for mozContacts results? See the upgrade code. Careful, it is ugly ugly code.

https://github.com/mozilla-b2g/gaia/blob/75df85cbcf0440a59602c01a65ece6aca61f6952/apps/communications/dialer/js/call_log_db.js#L280-L478

Comment 3

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Unfortunately there's no way to keep a transaction open while other async code is running. Can you gather the data you need from mozcontacts before upgrading your database?

Comment 4

[Anthony Ricaud (:rik)]

I won't have time to work on this. I think the path forward here is to upgrade the reference-workload databases after upgrade4 and remove these bogus upgrades (since no end-user ever bought a phone before this upgrade).

Also, we can do a simpler patch that just stops recording photos in the DB. Because Call log eventually discards old entries, end-users will end up with a smaller DB without upgrades.

Needinfo-ing Doug to put this into a sprint.

Comment 5

[:gerard-majax]

Maybe this would fix bug 1085536 ?

Comment 6

[Doug Sherk (:drs) (inactive)]

Clearing needinfo since we found out that bug 1085536 is a dupe of bug 1087464.

Comment 7

[BMO Automation]

Firefox OS is not being worked on