180 bytes, text/html
Master build : 2013-04-03-03-09-39 project name="mozilla-central" revision="97cfc16ba5dc" project name="integration/gaia-central" revision="87f388cd8a2b" "gecko.git" revision="c9029081bedc93372d0e15e66d89c81bf74c1fd0" "gaia.git" revision="30f22a3f669779b30efdc09c6ad3f9e6476b988f" Gecko http://hg.mozilla.org/releases/mozilla-b2g18/rev/d467369d1b0c Gaia 06e0e5ce42bdfb62bdbe38271de6b5b2d9e40e75 BuildID 20130403070204 Version 18.0 ## Repro : 1. launch email app w/ an email account setup 2. select an email in the list 3. tap on the email address that's not in the contacts app 4. select Save contact 5. hit home button 6. launch contact 7. look for new contact ## Expected : contact should be found ## Actual : contact is not found. ## Note : 1. happens both on gmail and hotmail 2. doesn't matter if contact is or is not launched.
I can't find an existing contact either... it almost seems like it's using two different databases?
This only occurs when the contact database does not exist, users try to access the contact database through web activity such as 'new'. After that occurs, all created contact database cannot be found by the contact app itself. Therefore users cannot find the added contacts in the next time they launch the contact app. I'm still trying to find the root cause.
I would go for avoiding adding the contact to the list if we are currently in an activity. That's a not needed operation, as the flow is going to finish after saving the contact... What do you think?
Created attachment 735574 [details] Link to https://github.com/mozilla-b2g/gaia/pull/9075 Alberto, your suggestion makes a lot sense. However, I couldn't find a good timing to avoid the initialization of the contact list. We finish the web activity at the time of successfully adding a contact. The event of contact change fires after that, which means we cannot differentiate this case from normal ones in the "performOnContactChange" function because the "ActivityHandler.currentlyHandling" is already false. Thus I made a simpler workaround as the pull request. Please let me know if any concerns, thanks!
What about not registering the oncontactschange event if we start an activity?
Your suggestion is great. Pull request updated.
Uplifted commit 56c53e090412cf8aeb7b80a812b90229ec4d6cd6 as: v1-train: 41e5fb95b0673b8c1b457ea8ab3b1d52319a6142
verified fixed on Leo with: Gecko http://hg.mozilla.org/releases/mozilla-b2g18/rev/0c71cbc5fe0c Gaia a7b0810580afc734f3d5e441914fe895f9c1923e BuildID 20130508230207 Version 18.0