Description: Unable to access contacts imported from Facebook after device reboot. Before rebooting works OK. Repro Steps: 1) Updated Buri to BuildID: 20131113040200 2) Open Contacts app 3) Tap gear icon and select Sync Facebook friends 4) Open contact details of any of imported contacts 5) Reboot device 6) Open Contacts Actual: Unable to access imported from Facebook contact Expected: Able to access imported from Facebook contact Environmental Variables: Device: Buri 1.3 BuildID: 20131113040200 Gaia: 7243b75041c93bc7deb240131748d0b164f3f0b0 Gecko: 7b014f0f3b03 Version: 28.0a1 Firmware Version: 20131104 Repro frequency: 100%

This issue started to occur on the Buri 1.3 Build ID: 20131030040204 Gaia 162cc9cd7153dcb284e5f54d5598fb27f1ab18aa SourceStamp 829d7bef8b0a BuildID 20131030040204 Version 28.0a1 Last working Buri 1.3 Build ID: 20131029040205 Gaia 92c097fd7ac9886f937d9decb0e03ab673deaa1b SourceStamp 518f5bff0ae4 BuildID 20131029040205 Version 28.0a1

If it's a regression on the Gaia side, then it's likely caused by bug 923637. Sarah - I need gecko commits on that regression window. Can you include that?

(In reply to Jason Smith [:jsmith] from comment #2) > If it's a regression on the Gaia side, then it's likely caused by bug 923637. > > Sarah - I need gecko commits on that regression window. Can you include that? Although what seems more likely here is this could just be a bug in the Data Store API.

Here it is for the 1.3 10/30 build. Mercurial-Information: <project name="mozilla-central" path="gecko" remote="hgmozillaorg" revision="829d7bef8b0a"/> --> <project name="gecko.git" path="gecko" remote="mozillaorg" revision="fdbfcf7d24795f482e6969b0f664c53c2b3c2ed9"/> (In reply to Jason Smith [:jsmith] from comment #2) > If it's a regression on the Gaia side, then it's likely caused by bug 923637. > > Sarah - I need gecko commits on that regression window. Can you include that?

There are lots of [JavaScript Error: "too much recursion"] errors in the log. This can be a gaia bug. Sarah, can you please bisect the gaia range? Thanks!

(In reply to :Ehsan Akhgari (needinfo? me!) from comment #5) > There are lots of [JavaScript Error: "too much recursion"] errors in the > log. This can be a gaia bug. > > Sarah, can you please bisect the gaia range? Thanks! If it's Gaia related, then it's likely bug 923637 causing this. It's the only contacts patch that's data store related with facebook involved.

(In reply to Jason Smith [:jsmith] from comment #6) > (In reply to :Ehsan Akhgari (needinfo? me!) from comment #5) > > There are lots of [JavaScript Error: "too much recursion"] errors in the > > log. This can be a gaia bug. > > > > Sarah, can you please bisect the gaia range? Thanks! > > If it's Gaia related, then it's likely bug 923637 causing this. It's the > only contacts patch that's data store related with facebook involved. within the target regression range*

https://github.com/mozilla-b2g/gaia/commit/33156ceb8a7a63f07f264a2903fe5d703314bdfe is the commit you'll want to look at.

Here is the gecko commmit for 1.3 10/29 revision="518f5bff0ae4"

Checked the gecko push log - there really isn't anything here that looks like a potential cause of this bug. Adding bug 923637 since it looks like the likely cause right now of this bug.

triage: 1.3+ regression

Hi guys, I cannot repro this. Please could you share by private email (jmcf@tid.es), the details of the account you are using to test? thanks!

Now I've been able to reproduce. This is clearly a platform issue. navigator.getDataStores(DATASTORE_NAME).then(function success(ds) { }, function error(err) {} ); It is not calling the success nor the error CB. Ehsan, please could you have a look at it? thanks!

Andrea, can you please take a look? Thanks!

> I cannot repro this. Please could you share by private email (jmcf@tid.es), > the details of the account you are using to test? Can you share this with me too? I don't have a facebook account and I cannot reproduce this issue with mochitests.

I've seen this error once, using Firefox Desktop (with DEBUG=1 option), was generated by another error in the app, somehow the error log is not able to display the real reason. If I find again will try to go deeper on what's going on.

For what its worth I ran into this today trying to test the patches for bug 924139.

Andrea, can you describe how this part of getDataStoreCreate() is supposed to work? http://mxr.mozilla.org/mozilla-central/source/dom/datastore/DataStoreService.js#306 Are we guaranteed to have receiveMessage() called after this?

Actually, it can happen that receiveMessage is not called! Thanks

ah... I don't know if this patch fixes this bug. But for sure this patch is needed.

Comment on attachment 8343228 [details] [diff] [review] patch Review of attachment 8343228 [details] [diff] [review]: ----------------------------------------------------------------- Can you add a test for this please?

I don't think I can simulate this behaviour with a test. It's a timing issue. It means that IDB is slower than the request. Hard to say that that code actually runs.

The problem can still be reproduced on unagi Gecko 0871ff3

Jose, can you help me to reproduce this issue?

Sent a private email with detailed instructions and test account data

The first patch was not actually able to sent the event properly.

This patch removes an additional space between // and dump(... I know that is not related to this patch but I hate that space any time I see that code :) BTW, this patch has been tested on a real device.

https://hg.mozilla.org/integration/mozilla-inbound/rev/0275b5c15174

Hi John, Could be so kind of uplifting this patch to 1.3 branch? Bug 949214 is breacking the smoketest and it seems that will be fixed as soon as this patch lands in that branch. Thanks a lot!!

Looks like this is a gecko patch. Ryan, could you please take a look at this?