967272 - Contacts spends to much time decoding image: Should be using thumbnails

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Contacts, defect, P1)

Description

[Benoit Girard (:BenWa)]

Profile:
http://people.mozilla.org/~bgirard/cleopatra/#report=a3ab371e7a8663d3f94e61ed172b2b213684fa17&search=blob

The above profile with the filter shows the extra time decoding. We need to store thumbnail instead of decoding the full image to downscale it.

Comment 1

[Milan Sreckovic [:milan] (needinfo for best results)]

David, how are we doing those images in the contacts app?  Are we keeping the original images, and if so, are we decoding the full size or looking at the EXIF thumbnail (if there)?  Or do we scale it the first time and then keep the scaled down version around?

Comment 2

[Benoit Girard (:BenWa)]

The image source are 256x256 from the test workload.

I took a image from the camera and I see that we're using a 320x320 image. These aren't massive but they're 60x60 on the screen so we're loosing a ton of performance here.

Comment 3

[Mike Lee [:mlee]]

Benoit, what's the end-user impact and frequency of them encountering its effect?

Comment 4

[Benoit Girard (:BenWa)]

The end impact is scrolling the contacts list means the thumbnail take ~5 times longer to show up. This can make the difference between having them show up while you're scrolling your contacts or having to stop panning and wait for the contact photos to show up.

Comment 5

[Milan Sreckovic [:milan] (needinfo for best results)]

Outside of the images showing up "later", is there an impact on the text part checkerboarding because of this?

Comment 6

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to Milan Sreckovic [:milan] from comment #5)
> Outside of the images showing up "later", is there an impact on the text
> part checkerboarding because of this?

Etienne is working on this. We already discussed how to do that and to solve it for call log, sms and contacts where I assume the issue is the same.

Comment 7

[Mike Lee [:mlee]]

Thanks Benoit. I'd consider the text checkerboarding a blocker but not the image issue of needing to stop panning.

Benoit & Vivien,
Do both issues need to be handled at the same time or can we split them into separate bugs so we can block on only the checkerboarding piece?

Comment 8

[Benoit Girard (:BenWa)]

(In reply to Milan Sreckovic [:milan] from comment #5)
> Outside of the images showing up "later", is there an impact on the text
> part checkerboarding because of this?

Yes, we can delay it by an extra ~5ms here and there. But fixing this and tweaking how we decode means we could load the thumbnails as we're scrolling with minimal scrolling impact giving a much better user experience.

(In reply to Mike Lee [:mlee] from comment #7)

It's all the same cause that the thumbnails are too big. I don't see a great way to break it down further.

(In reply to Vivien Nicolas (:vingtetun) (:21) from comment #6)

Awesome!

Comment 9

[Joe Cheng [:jcheng] (please needinfo)]

triage: 1.3+ blocks a blocker

Comment 10

[Etienne Segonzac (:etienne)]

(In reply to Vivien Nicolas (:vingtetun) (:21) from comment #6)
> (In reply to Milan Sreckovic [:milan] from comment #5)
> > Outside of the images showing up "later", is there an impact on the text
> > part checkerboarding because of this?
> 
> Etienne is working on this. We already discussed how to do that and to solve
> it for call log, sms and contacts where I assume the issue is the same.

Yep, experimenting with saving a thumbnail when we save a contact picture.
Should have a solid proof of concept by the end of the day.

(https://github.com/etiennesegonzac/gaia/tree/contact-thumbnails)

Comment 11

[Francisco Jordano [:arcturus] [:francisco]]

(In reply to Etienne Segonzac (:etienne) from comment #10)
> (In reply to Vivien Nicolas (:vingtetun) (:21) from comment #6)
> > (In reply to Milan Sreckovic [:milan] from comment #5)
> > > Outside of the images showing up "later", is there an impact on the text
> > > part checkerboarding because of this?
> > 
> > Etienne is working on this. We already discussed how to do that and to solve
> > it for call log, sms and contacts where I assume the issue is the same.
> 
> Yep, experimenting with saving a thumbnail when we save a contact picture.
> Should have a solid proof of concept by the end of the day.
> 
> (https://github.com/etiennesegonzac/gaia/tree/contact-thumbnails)

Nice, it's looking awesome, a pitty that we don't have a thumbnail field on the api, but the photo array will do the trick.

Thanks a lot Etienne!

Comment 12

[Etienne Segonzac (:etienne)]

Progress report: it's working, dialer, contacts and sms are updated, all tests are green!

https://github.com/etiennesegonzac/gaia/compare/contact-thumbnails?expand=1

Still need to add tests, but this is really to be tested/profiled.

Comment 13

[Etienne Segonzac (:etienne)]

Attachment: Pointer to gaia PR

Here we go!

This patch:
* generates thumbnails for new contacts
* introduces a new ContactPhotoHelper
* updates all the comms app to use this helper

Tried to optimize for upliftability without sacrificing the test coverage :)

r? Arcturus for the Contacts part (biggest one obviously)
r? Rik for the dialer part
r? Julien for the sms part 

We should maybe open a bug to track a possible migration of all the old contact pictures but not sure the risk/reward ratio looks good.

Comment 14

[Etienne Segonzac (:etienne)]

Notes:

* would be cool to have this patch profiled to see how much it helps, but since the reference workload contacts don't have thumbnails it will require some manual contact entry...

* the current version of the patch should uplift to 1.3 pretty easily (conflicts are only in test files and pretty straightforward).

Comment 15

[Julien Wajsberg [:julienw]]

Comment on attachment 8370836 [details] [review]
Pointer to gaia PR

Ok, this is great, r=me
I added some small nits but none are mandatory.

As discussed, we need to know whether the photos from Facebook are also Blob. I think they are but Francisco or José will know better. If they're Blob, then we can remove the legacy data-url code (but maybe in another patch that will land in master only, so that we don't risk a regression).

Comment 16

[Anthony Ricaud (:rik)]

Comment on attachment 8370836 [details] [review]
Pointer to gaia PR

We should open a bug to update the reference-workload to include a thumbnail.

Two comments in the pull request. They should be addressed but no need to ask for another review.

Comment 17

[Etienne Segonzac (:etienne)]

(First rounds of comments addressed, no more sadpanda)

Comment 18

[David Flanagan [:djf]]

Clearing the needinfo. Looks like this has been completely handled without me.

Comment 19

[Milan Sreckovic [:milan] (needinfo for best results)]

Review ping

Comment 20

[Francisco Jordano [:arcturus] [:francisco]]

Comment on attachment 8370836 [details] [review]
Pointer to gaia PR

Left some comments in github, we are almost there!

Thanks a lot for the hard work Etienne!

Comment 21

[Francisco Jordano [:arcturus] [:francisco]]

Flagging Jose for feedback just in case I missed some of the Fb stuff.

Comment 22

[Jose Manuel Cantera]

Comment on attachment 8370836 [details] [review]
Pointer to gaia PR

I don't like the changes made on the contacts merger module. I wouls suggest you revert those and only change what is needed for this patch and do not change parts of the code that have nothing to do with this patch.

Comment 23

[Jose Manuel Cantera]

and as a last comment I don't like how the problem has been approached. This patch should have been divided in smaller patches, one to save the contact thumbnail and n number of patches to align that new data model with the rest of affected modules, in contacts and in the rest of apps. The regression risk of this huge patch is high and the revert of it can be painful.

Comment 24

[Etienne Segonzac (:etienne)]

(In reply to Jose M. Cantera from comment #22)
> Comment on attachment 8370836 [details] [review]
> Pointer to gaia PR
> 
> I don't like the changes made on the contacts merger module. I wouls suggest
> you revert those and only change what is needed for this patch and do not
> change parts of the code that have nothing to do with this patch.

Can you be more specific?
The thumbnail generation is async, so I have to *move* (not change) some code in its callback. What other solution are you suggesting.

And as a side note, I'd really like some feedback on how to implement this in the facebook-related code paths.

Comment 25

[Francisco Jordano [:arcturus] [:francisco]]

Right, the diff viewer is not helping there, it almost happen to me as well. But as commented before is just moving code.

Comment 26

[Ben Kelly [:bkelly, not reviewing]]

Might be easier to read with the the w=1 option to ignore whitespace changes:

  https://github.com/mozilla-b2g/gaia/pull/15990/files?w=1

Comment 27

[Jose Manuel Cantera]

(In reply to Etienne Segonzac (:etienne) from comment #24)
> (In reply to Jose M. Cantera from comment #22)
> > Comment on attachment 8370836 [details] [review]
> > Pointer to gaia PR
> > 
> > I don't like the changes made on the contacts merger module. I wouls suggest
> > you revert those and only change what is needed for this patch and do not
> > change parts of the code that have nothing to do with this patch.
> 
> Can you be more specific?
> The thumbnail generation is async, so I have to *move* (not change) some
> code in its callback. What other solution are you suggesting.
> 

ok, you are right. I retract from that comment, although the logic has been slightly changed, please see my latest comment on GH. 

> And as a side note, I'd really like some feedback on how to implement this
> in the facebook-related code paths.

Following the principle stated in comment #23 (that I do not retract), that should be done in another bug. If you open it I would be more than happy to contribute to it.

Comment 28

[Etienne Segonzac (:etienne)]

Comment on attachment 8370836 [details] [review]
Pointer to gaia PR

Pull request updated!

And tested with gmail and facebook imports :)

Comment 29

[Etienne Segonzac (:etienne)]

Hey Benoit,

The patch is pretty much done, but we'll probably need to quantify the gain it brings us in order to make an inform uplift decision.

Could you help us with profiling?

(Since the reference workloads don't have thumbnails, the best way to test this is to import your facebook contacts, at least if you're a facebook user :))

Comment 30

[Francisco Jordano [:arcturus] [:francisco]]

Comment on attachment 8370836 [details] [review]
Pointer to gaia PR

There we go, thanks a lot Etienne!

Comment 31

[Jose Manuel Cantera]

I left substantial comments on GH. I think the patch is far from ready to land and I would ask reviewers to reconsider the r+ given

Comment 32

[Julien Wajsberg [:julienw]]

(In reply to Jose M. Cantera from comment #31)
> I left substantial comments on GH. I think the patch is far from ready to
> land and I would ask reviewers to reconsider the r+ given

My r+ was only regarding the SMS part, so I'm keeping it.

Comment 33

[Francisco Jordano [:arcturus] [:francisco]]

Hi Etienne did you take a loot to Jose's comments regarding the algorithm to create the thumbnails, apparently we are loosing quality.

Comment 34

[Etienne Segonzac (:etienne)]

(In reply to Jose M. Cantera from comment #31)
> I left substantial comments on GH. I think the patch is far from ready to
> land and I would ask reviewers to reconsider the r+ given

Sorry guys, don't have the time nor the energy to play politics with José.
The patch is here, I gave it my very best shot. Feel free to take it, split it, build on it, or throw it away.

Comment 35

[Francisco Jordano [:arcturus] [:francisco]]

Wilfred, can you give us your POV about the quality of the thumbnails?

If that's ok for you, let's land this and I'll open two follow ups:

- One for FB sync.
- Other for the device pixel ratio.

Cheers,
F.

Comment 36

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to Francisco Jordano [:arcturus] from comment #35)
> Wilfred, can you give us your POV about the quality of the thumbnails?
> 
> If that's ok for you, let's land this and I'll open two follow ups:
> 
> - One for FB sync.
> - Other for the device pixel ratio.
> 
> Cheers,
> F.

Fwiw Etienne has fixed some of the quality concerns as far as I know.

Comment 37

[Wilfred Mathanaraj [:WDM]]

Given the fixes that go in I think we can live with the current quality of the thumbnail image.

Comment 38

[Etienne Segonzac (:etienne)]

Thanks for being the mediator Francisco :)
And sorry for over-reacting.

(In reply to Francisco Jordano [:arcturus] from comment #35)
> If that's ok for you, let's land this and I'll open two follow ups:

The contacts app has still 5 open blockers as of today...
I think we're past opening follow ups.

> - One for FB sync.

Can we get a exhaustive feedback on this, with the goal of actually landing the patch?

I'm afraid

> Unfortunately this is not the only place where the thumbnail has to be created as there are other code paths, mainly related to synchronization 

isn't actionable :/

> - Other for the device pixel ratio.

I don't think there's anything left to be done here.

Comment 40

[Jose Manuel Cantera]

:etienne,

thanks for fixing the DPI! and sorry if my messages led you to misinterpret them as playing politics, It was not my real intention honestly. But in the future, as I'm a Contacts App peer I would like to be informed of this kind of changes, as soon as possible. Also I strongly believe that cross-app are not a good idea in terms of code control and change management. But I can understand you could be in a hurry for the 1.3+ deadline. 

Nonetheless, before landing I think we need to fix two things. 

A/ Generating thumbnails when FB sync updates images. That's an easy fix

https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/facebook/js/fb_sync.js#L130 

B/ I still think that the generated thumbnails should be square instead of preserving the aspect ratio. We already have code that makes a picture square

https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/contacts/js/utilities/image_square.js

thank you very much!

Comment 41

[Etienne Segonzac (:etienne)]

(In reply to Jose M. Cantera from comment #40)
> Nonetheless, before landing I think we need to fix two things. 
> 
> A/ Generating thumbnails when FB sync updates images. That's an easy fix
> 
> https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/facebook/
> js/fb_sync.js#L130 

done, how can I trigger a sync to manually test this?

> 
> B/ I still think that the generated thumbnails should be square instead of
> preserving the aspect ratio. We already have code that makes a picture square
> 
> https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/contacts/
> js/utilities/image_square.js


We display only squared contact images in gaia, but the mozContacts database is meant to be shared with other third party apps. This is why we don't want to remove part of the image.

The current implementation lets us (gaia) display full squared image without stretching (only croping), while still giving the option for other apps to do differently.

Comment 42

[Jose Manuel Cantera]

(In reply to Etienne Segonzac (:etienne) from comment #41)
> (In reply to Jose M. Cantera from comment #40)
> > Nonetheless, before landing I think we need to fix two things. 
> > 
> > A/ Generating thumbnails when FB sync updates images. That's an easy fix
> > 
> > https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/facebook/
> > js/fb_sync.js#L130 
> 
> done, how can I trigger a sync to manually test this?

trigger FB import, set airplane mode, import contacts, they will be imported without photo. Then, unset airplane mode, click on update imported friends and a sync will be triggered and the contacts photos will downloaded. 

> 
> > 
> > B/ I still think that the generated thumbnails should be square instead of
> > preserving the aspect ratio. We already have code that makes a picture square
> > 
> > https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/contacts/
> > js/utilities/image_square.js
> 
> 
> We display only squared contact images in gaia, but the mozContacts database
> is meant to be shared with other third party apps. This is why we don't want
> to remove part of the image.
> 

Yes, but if we follow the same rationale, why are you creating the thumbnail with Gaia in mind i.e. with the dimensions required by the Gaia apps? 

> The current implementation lets us (gaia) display full squared image without
> stretching (only croping), while still giving the option for other apps to
> do differently.

Comment 43

[Jose Manuel Cantera]

(In reply to Jose M. Cantera from comment #42)
> (In reply to Etienne Segonzac (:etienne) from comment #41)
> > (In reply to Jose M. Cantera from comment #40)
> > > Nonetheless, before landing I think we need to fix two things. 
> > > 
> > > A/ Generating thumbnails when FB sync updates images. That's an easy fix
> > > 
> > > https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/facebook/
> > > js/fb_sync.js#L130 
> > 
> > done, how can I trigger a sync to manually test this?
> 
> trigger FB import, set airplane mode, import contacts, they will be imported
> without photo. Then, unset airplane mode, click on update imported friends
> and a sync will be triggered and the contacts photos will downloaded. 
> 
> > 
> > > 
> > > B/ I still think that the generated thumbnails should be square instead of
> > > preserving the aspect ratio. We already have code that makes a picture square
> > > 
> > > https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/contacts/
> > > js/utilities/image_square.js
> > 
> > 
> > We display only squared contact images in gaia, but the mozContacts database
> > is meant to be shared with other third party apps. This is why we don't want
> > to remove part of the image.
> > 
> 
> Yes, but if we follow the same rationale, why are you creating the thumbnail
> with Gaia in mind i.e. with the dimensions required by the Gaia apps? 
> 
> > The current implementation lets us (gaia) display full squared image without
> > stretching (only croping), while still giving the option for other apps to
> > do differently.

Well, the current implementation works with the images loaded manually because they are made square beforehand https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/contacts/js/views/form.js#L1058 but that's not true for all the photos as they could come from other sources.

Comment 44

[Etienne Segonzac (:etienne)]

(In reply to Jose M. Cantera from comment #42)
> > We display only squared contact images in gaia, but the mozContacts database
> > is meant to be shared with other third party apps. This is why we don't want
> > to remove part of the image.
> > 
> 
> Yes, but if we follow the same rationale, why are you creating the thumbnail
> with Gaia in mind i.e. with the dimensions required by the Gaia apps? 

We're creating thumbnails because that's the point of the bug...
I strongly believe that the thumbnail version of an image should be the same image but smaller, we need to pick a size, the current setting is my best guest for the current use cases, and since it does not arbitrarily removes part of the image it should work well for other use cases.

Comment 45

[Etienne Segonzac (:etienne)]

(In reply to Jose M. Cantera from comment #43)
> Well, the current implementation works with the images loaded manually
> because they are made square beforehand
> https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/contacts/
> js/views/form.js#L1058 but that's not true for all the photos as they could
> come from other sources.

Not sure I follow you.
We already make sure that the images will be displayed in squares in CSS [1], we don't need to crop every picture in the DB.

[1] https://github.com/mozilla-b2g/gaia/blob/master/shared/style/lists.css#L125-134

Comment 46

[Jose Manuel Cantera]

(In reply to Etienne Segonzac (:etienne) from comment #45)
> (In reply to Jose M. Cantera from comment #43)
> > Well, the current implementation works with the images loaded manually
> > because they are made square beforehand
> > https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/contacts/
> > js/views/form.js#L1058 but that's not true for all the photos as they could
> > come from other sources.
> 
> Not sure I follow you.
> We already make sure that the images will be displayed in squares in CSS
> [1], we don't need to crop every picture in the DB.
> 
> [1]
> https://github.com/mozilla-b2g/gaia/blob/master/shared/style/lists.css#L125-
> 134

Yes, but if a picture is not square it could be stretched in a way that the aspect ratio is not kept and as a result you can get a distorted image.

Comment 47

[Etienne Segonzac (:etienne)]

(In reply to Jose M. Cantera from comment #46)
> (In reply to Etienne Segonzac (:etienne) from comment #45)
> > (In reply to Jose M. Cantera from comment #43)
> > > Well, the current implementation works with the images loaded manually
> > > because they are made square beforehand
> > > https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/contacts/
> > > js/views/form.js#L1058 but that's not true for all the photos as they could
> > > come from other sources.
> > 
> > Not sure I follow you.
> > We already make sure that the images will be displayed in squares in CSS
> > [1], we don't need to crop every picture in the DB.
> > 
> > [1]
> > https://github.com/mozilla-b2g/gaia/blob/master/shared/style/lists.css#L125-
> > 134
> 
> Yes, but if a picture is not square it could be stretched in a way that the
> aspect ratio is not kept and as a result you can get a distorted image.

No, we use background-size: cover [1].

[1] https://developer.mozilla.org/en-US/docs/Web/CSS/background-size
> The image is rendered, preserving its intrinsic proportion

Comment 48

[Etienne Segonzac (:etienne)]

(In reply to Jose M. Cantera from comment #42)
> (In reply to Etienne Segonzac (:etienne) from comment #41)
> > (In reply to Jose M. Cantera from comment #40)
> > > Nonetheless, before landing I think we need to fix two things. 
> > > 
> > > A/ Generating thumbnails when FB sync updates images. That's an easy fix
> > > 
> > > https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/facebook/
> > > js/fb_sync.js#L130 
> > 
> > done, how can I trigger a sync to manually test this?
> 
> trigger FB import, set airplane mode, import contacts, they will be imported
> without photo. Then, unset airplane mode, click on update imported friends
> and a sync will be triggered and the contacts photos will downloaded. 
> 

Just confirmed this works properly now :)

Comment 49

[Jose Manuel Cantera]

(In reply to Etienne Segonzac (:etienne) from comment #47)
> (In reply to Jose M. Cantera from comment #46)
> > (In reply to Etienne Segonzac (:etienne) from comment #45)
> > > (In reply to Jose M. Cantera from comment #43)
> > > > Well, the current implementation works with the images loaded manually
> > > > because they are made square beforehand
> > > > https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/contacts/
> > > > js/views/form.js#L1058 but that's not true for all the photos as they could
> > > > come from other sources.
> > > 
> > > Not sure I follow you.
> > > We already make sure that the images will be displayed in squares in CSS
> > > [1], we don't need to crop every picture in the DB.
> > > 
> > > [1]
> > > https://github.com/mozilla-b2g/gaia/blob/master/shared/style/lists.css#L125-
> > > 134
> > 
> > Yes, but if a picture is not square it could be stretched in a way that the
> > aspect ratio is not kept and as a result you can get a distorted image.
> 
> No, we use background-size: cover [1].
> 
> [1] https://developer.mozilla.org/en-US/docs/Web/CSS/background-size
> > The image is rendered, preserving its intrinsic proportion

ok, in that case I trust you :)

thanks!

Comment 50

[Jose Manuel Cantera]

(In reply to Etienne Segonzac (:etienne) from comment #48)
> (In reply to Jose M. Cantera from comment #42)
> > (In reply to Etienne Segonzac (:etienne) from comment #41)
> > > (In reply to Jose M. Cantera from comment #40)
> > > > Nonetheless, before landing I think we need to fix two things. 
> > > > 
> > > > A/ Generating thumbnails when FB sync updates images. That's an easy fix
> > > > 
> > > > https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/facebook/
> > > > js/fb_sync.js#L130 
> > > 
> > > done, how can I trigger a sync to manually test this?
> > 
> > trigger FB import, set airplane mode, import contacts, they will be imported
> > without photo. Then, unset airplane mode, click on update imported friends
> > and a sync will be triggered and the contacts photos will downloaded. 
> > 
> 
> Just confirmed this works properly now :)

I reviewed the code and it looks ok!

thanks!

Comment 51

[Jose Manuel Cantera]

Comment on attachment 8370836 [details] [review]
Pointer to gaia PR

the code looks good now.

Comment 52

[Etienne Segonzac (:etienne)]

https://github.com/mozilla-b2g/gaia/commit/9fc36dde3a4a3c5ca200275b68ffb56b4173bec3

I'll prepare a 1.3 version of the patch (I think only tests are conflicting)

Comment 53

[Etienne Segonzac (:etienne)]

Uplifted on 1.3, with a green travis, after a manual smoke test covering contact creation, update, merge, import (gmail and facebook) and import from the ftu.

https://github.com/mozilla-b2g/gaia/commit/00cd1ae74dfe773a41b55275b0b468a62f35076d

Comment 54

[Tony Chung [:tchung]]

Etienne, can you comment on a list of test areas that QA can focus around this patch?   other than smoketesting, give some tips here on highest areas of regression points, things that would be good to retest, etc..   

Massimo's team at Telefonica has volunteered to assist with QA on the next 1.3 build.  Thanks Massimo!

Comment 55

[Etienne Segonzac (:etienne)]

Like I said:

> smoke test covering contact creation, update, merge, import (gmail and facebook) and import from the ftu.

Comment 56

[Loli (:lolimartinezcr)]

Related to :

- Import contact to gmail and facebook, you can see this bugs:
https://bugzilla.mozilla.org/show_bug.cgi?id=970832
https://bugzilla.mozilla.org/show_bug.cgi?id=970874
https://bugzilla.mozilla.org/show_bug.cgi?id=970899, duplicate of https://bugzilla.mozilla.org/show_bug.cgi?id=838605

Comment 57

[Loli (:lolimartinezcr)]

This bugs have been tested (02/12/2014)
1.3
Gecko 3820ecd
Gaia 01e9da4

Comment 58

[Etienne Segonzac (:etienne)]

Thanks Loli!

(In reply to Loli from comment #56)
> Related to :
> 
> - Import contact to gmail and facebook, you can see this bugs:
> https://bugzilla.mozilla.org/show_bug.cgi?id=970832

Trying to reproduce it to see if it was caused by this patch.

> https://bugzilla.mozilla.org/show_bug.cgi?id=970874
> https://bugzilla.mozilla.org/show_bug.cgi?id=970899, duplicate of
> https://bugzilla.mozilla.org/show_bug.cgi?id=838605

Those _look_ unrelated.