1113355 - [Datastore] ConstraintError when upgrading the phone

Status: RESOLVED WORKSFORME

(Core :: DOM: Core & HTML, defect)

Description

[Julien Wajsberg [:julienw]]

After upgrading my phone, I got this error:

12-18 22:59:55.249   490   490 W Smart Collections: [JavaScript Error: "ConstraintError" {file: "jar:file:///system/b2g/omni.ja!/components/DataStoreImpl.js" line: 233}]

I saved the IDB I have on the phone but I don't really know which DB you'd like to see, especially with the weird file names. Please tell me what you need and I can attach to this bug.

Thanks!

Comment 1

[Julien Wajsberg [:julienw]]

Attachment: bug-1113355.zip

Here are the DB for the datastores with issues.

Comment 2

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

You need to set PR_LOG_MODULES=IndexedDB:5 and that should tell you which db is being accessed.

Comment 3

[Julien Wajsberg [:julienw]]

The issue is with the bookmarks_store. Deleting this one only made the phone boot again.

Comment 4

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

This looks like some kind of bug in DataStore

Comment 5

[Andrea Marchesini [:baku]]

> 12-18 22:59:55.249   490   490 W Smart Collections: [JavaScript Error:
> "ConstraintError" {file:
> "jar:file:///system/b2g/omni.ja!/components/DataStoreImpl.js" line: 233}]

let request = aStore.add(aObj, aId);

This is what we do at that line. Can you tell me when this code is called?

Comment 6

[Julien Wajsberg [:julienw]]

I don't know about this at all.

Ben, Dale, you probably know when we add something in the bookmarks_store ?

Comment 7

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

triage: removing the nomination for now.
Hi Julien,
Do you think there's a more general STR which end user may encounter?
If so, please go ahead and renominate.

Comment 8

[Julien Wajsberg [:julienw]]

This is an upgrade bug, this is happening with my data, so this _will_ happen with other data as well.

The STR should be simple: flash the datastore in the attached file with a newer build.

Comment 9

[Andrea Marchesini [:baku]]

Julien, are you still able to reproduce this issue?

Comment 10

[Dale Harvey (:daleharvey)]

Sorry, been trying to clear out a long list of issues, finally on to take a look at this one now, ill check if it still reproduces

Comment 11

[Julien Wajsberg [:julienw]]

I tried to push these 2 db on a phone running central and I don't have the issue anymore.

So either it was fixed elsewhere or these files do not make a good testcase :/

We should try with a build from around when the bug was filed to see if the issue reproduces.

Comment 12

[Dale Harvey (:daleharvey)]

I tried flashing several base builds and havent been able to reproduce this, closing out unless someone else can manage