Created attachment 8644880 [details] logcat_1616.txt [1.Description]: [Aries v2.5][Flame v2.5][Contacts]When you add contact as attachment, then view it, device will enter import page, and it can't display contact details. Found time:16:16 See attachment:Aries_2.5.3gp and logcat_1616.txt [2.Testing Steps]: 1.Launch Message. 2.Create new message. 3.Tap attachment icon -> "Communications", then select a contact. 4.Tap the attachment -> "View". [3.Expected Result]: 4.It should display the contact details. [4.Actual Result]: 4.Device will enter import page, and only display portrait and name. [5.Reproduction build]: Device: Aries KK 2.5(Affected) Build ID 20150806003241 Gaia Revision 581de383687dc441a878d2c91a0167c6ec688fef Gaia Date 2015-08-05 01:48:40 Gecko Revision https://hg.mozilla.org/mozilla-central/rev/9801f91760d9 Gecko Version 42.0a1 Device Name aries Firmware(Release) 4.4.2 Firmware(Incremental) eng.worker.20150805.235801 Firmware Date Wed Aug 5 23:58:08 UTC 2015 Bootloader s1 Device: Flame KK 2.2(Unaffected) Build ID 20150806032503 Gaia Revision f8b119ac30e97df991c97682ac4d4f9ca22e1793 Gaia Date 2015-07-31 13:20:55 Gecko Revision https://hg.mozilla.org/releases/mozilla- b2g37_v2_2/rev/a9379f716e9f Gecko Version 37.0 Device Name flame Firmware(Release) 4.4.2 Firmware(Incremental) eng.cltbld.20150806.065240 Firmware Date Thu Aug 6 06:52:51 EDT 2015 Firmware Version v18D v4 Bootloader L1TC000118D0 Device: Flame KK 2.5(Affected) Build ID 20150806103226 Gaia Revision 497fe3f938722b0aa49c93f975fad5d9ed3b0a82 Gaia Date 2015-08-06 05:30:02 Gecko Revision https://hg.mozilla.org/mozilla-central/rev/2fad09a0540d Gecko Version 42.0a1 Device Name flame Firmware(Release) 4.4.2 Firmware(Incremental) eng.cltbld.20150806.135403 Firmware Date Thu Aug 6 13:54:17 EDT 2015 Firmware Version v18D v4 Bootloader L1TC000118D0 [6.Reproduction Frequency]: Always Recurrence,5/5 [7.TCID]: Free Test
[Blocking Requested - why for this release]:functional regression issue
Comms triage: Functional regression.
b2g-inbound Regression Window: Last Working Environmental Variables: Device: Flame KK v2.5 Build ID 20150730014841 Gaia Revision ac76af0d04df109601cc5e5893a32c8435eb12e5 Gaia Date 2015-07-30 08:08:42 Gecko Revision https://hg.mozilla.org/integration/b2g-inbound/rev/c84484ec8c70 Gecko Version 42.0a1 Device Name flame Firmware(Release) 4.4.2 Firmware(Incremental) eng.cltbld.20150728.064331 Firmware Date Tue Jul 28 06:43:41 EDT 2015 Firmware Version v18D v4 Bootloader L1TC000118D0 First Broken Environmental Variables: Device: Flame KK v2.5 Build ID 20150730032845 Gaia Revision 5e31fb6f88cd8c42e53214183defa2ffd455ea00 Gaia Date 2015-07-30 09:39:02 Gecko Revision https://hg.mozilla.org/integration/b2g-inbound/rev/c05339efba8d Gecko Version 42.0a1 Device Name flame Firmware(Release) 4.4.2 Firmware(Incremental) eng.cltbld.20150728.064331 Firmware Date Tue Jul 28 06:43:41 EDT 2015 Firmware Version v18D v4 Bootloader L1TC000118D0 First Broken Gaia & Last Working Gecko - issue DOES repro Gaia Revision 5e31fb6f88cd8c42e53214183defa2ffd455ea00 Gecko Revision https://hg.mozilla.org/integration/b2g-inbound/rev/c84484ec8c70 First Broken Gecko & Last Working Gaia – issue DOES NOT repro Gecko Revision https://hg.mozilla.org/integration/b2g-inbound/rev/c05339efba8d Gaia Revision ac76af0d04df109601cc5e5893a32c8435eb12e5 Gaia pushlog: https://github.com/mozilla-b2g/gaia/compare/ac76af0d04df109601cc5e5893a32c8435eb12e5...5e31fb6f88cd8c42e53214183defa2ffd455ea00 Possibly caused by bug 1184075
Fernando, can you have a look at this. It seems to be a regression introduced when merging #open and #import activities
I wouldn't say it is a bug, nor a regression, as it is the expected behaviour after merging the OPEN VCARD and IMPORT VCARD activities to follow the same steps. The new process is described in bug 1184075 comment 0 As an explanation, the process described above for SMS is not opening the Contact details, but the VCard attached on the message, that certainly contains information about a contact from the phone, but without the internal ID for it. In the old version, this process would end up opening not the details page, but the New contact one, giving the option to save the information. In that case, we could see the rest of the information, but we wouldn't be opening the details per se. The idea is to solve this lack of information on the NGA by making the importing list clickable, and show the details page (bug 1184080)
You are right, Fernando and we considered that as an extra feature. In that case, perhaps we should try to implement the clickable list feature (if we have enough time) after finishing the views isolation as this bug has been set as 2.5+. Francisco, what do you think?
I would focus first on stability and blockers, and if we have time will go for this. Marking it as a feature p1.
[Blocking Requested - why for this release]: Not a regression.
Obviously this needs to block :(
I am going to clear the assignee for this bug, as we are not sure about how we are going to proceed yet in case the pre-rendering is not in time for 2.5. In that case and if we had to do roll back of the views separation already landed in master it will also include the merging of #open and #import activities that were the cause of this issue (bug 1184075). As soon as the importing list is clickable and show the details page (bug 1184080) the issue should be fixed but this a not a quick feature to implement so setting ni to Francisco to confirm how to proceed and in case the rollback happens this bug would be invalid.
Removed tracking-b2g as its a feature.
Clearing ni, as we have a plan to go for 2.5