940334 - [B2G][Helix][Contacts][WangChao]When import a outlook contact, it stuck on the progress bar.

Status: RESOLVED WORKSFORME

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

Description

[lecky]

User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.1; Trident/4.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0; .NET4.0C; .NET4.0E; Tablet PC 2.0)

Steps to reproduce:

1.install software and boot for the first time.
2.enter FTU, enable internet, go to "Import contacts" UI in FTU
3.select "outlook" Icon.
4.login and select a contact to import.
[Rate]:100%
Mention: This is only happen for A special outlook contact. I will upload the screenshot.


Actual results:

1.it stuck on the progress bar.


Expected results:

1.it import successfully.

Comment 1

[lecky]

Attachment: VID_0002.3gp

screenshot

Comment 2

[lecky]

Hi wayne

    this is serious bug, please help to process.

Thanks

Comment 3

[lecky]

When importing outlook contact stucks on the progress bar, we can not exit it unless we reboot the device.

Comment 4

[Wayne Chang [:wchang]]

Please provide details of this specific contact entry.
Please provide mozilla commits that you can reproduce this bug on.


Jose,
Can you check this here?

Comment 5

[lecky]

My teammate has sent the special login account to Luke.

Thanks!

Comment 6

[Luke Chang [:lchang] (inactive)]

I can reproduce this bug on the following build.

Gaia:     85f9a33cc95e04f5fd962ec37fa39eb622136f45
Gecko:    http://hg.mozilla.org/releases/mozilla-b2g18_v1_1_0_hd/rev/fa03d0230e50
BuildID   20131119042200
Version   18.0

But it can't be reproduced on v1.2 and master branch.

Comment 7

[Jose Manuel Cantera]

My colleague Marina will look into this issue with an Helix device, if she can get one

Comment 8

[Marina Rodríguez [:mai]]

Hi, 
i'm trying to reproduce the error but i was not succesfull. I followed the STR but the import works for me.
Would you mind to send me the contact details which produces the error?

Regards

Comment 9

[lecky]

Hi Luke:

  Can you help reproduce this for Marina or just send the login account to her and tell her the steps. You know the login account is restrict to public.

  I can not type the account info on this due to some reason. Sorry for this.

  Thanks for your understanding.
--------------
Hi Marina:

   Can you ask Luke for the detail, he know how to reproduce this issue. 

Thanks!

Comment 10

[lecky]

Hi Marina:

   Have you got the login account and reproduce this?

   If you need infomation about this, please let me know.

Thanks!

Comment 11

[lecky]

Hi Marina:
   
   I dump some log to track the point.

   //in contacts_importer.js
   function contactSaved(e) {
      dump("contactSaved typeof self.oncontactimported: "+typeof self.oncontactimported);  <--this been typed
      var cfdata = this;
      if (typeof self.oncontactimported === 'function') {
        window.setTimeout(function() {
          dump("setTimeout callback before self.oncontactimported");  <--this in not typed
          self.oncontactimported(cfdata);
        }, 0);
      }
      continueCb();
    }

Due to the log, function "window.setTimeout()" seems does not work. So "self.oncontactimported(cfdata);" is not called, the progress bar is stuck.

But for "window.setTimeout()", i do not know its implementation.

Thanks!

Comment 12

[Maria]

Hi,
Account information provided to Marina. 
Br

Comment 13

[Marina Rodríguez [:mai]]

Sorry for the delay. I was waiting for a coworker with a deeper knowledge than me for this application. 
I agree with Lecky that the application doesn't execute the window.setTimeout statement. In my opinion, the problem has to do with the number of contacts that you have in your account (more than 2000). 

When i made tests with another terminal, the import application replicated the behaviour with all contacts of the account, but when I used another account with less contacts, the application worked fine (with duplicate contacts, Special characters  (e.g. ç, ñ),...)

I hope to give you an update soon.

Best regards

Comment 14

[lecky]

Hi Marina:
   
   Thanks for you help. You said that it may due to the number of contacts.

   I will make a another account with more than 2000 contacts and do the same test. 

   If there is any result, i will reply to you.

Best regards

Comment 15

[lecky]

Hi Marina:

   How does this going? I think maybe the ticker is not work, but i do no know how to track or how to identify this?

   Can you help?

Thanks!

Comment 16

[Cristian Rodriguez (:crdlc)]

Hi all,

   I've just played with that hotmail account and the problem is related to memory consumption. I rendered only the first letter ('A') and it worked fine. I am not pretty sure what changes were introduced in v1.2 and master because it is not reproducible there. Maybe Gaia or in Gecko. Some help from Ben would be great for Marina if he has more context than me (I really think so ;))

Thanks a lot Ben

Comment 17

[Ben Kelly [:bkelly, not reviewing]]

Not sure when I will have time to look at this.  I'll need access to the problem hotmail account, though.  I've never actually run the outlook import code before.

Comment 18

[Cristian Rodriguez (:crdlc)]

Marina can send you this info, thanks Ben. Although IMHO it should happen with others services as well

Comment 19

[Ben Kelly [:bkelly, not reviewing]]

(In reply to Cristian Rodriguez (:crdlc) from comment #18)
> Marina can send you this info, thanks Ben. Although IMHO it should happen
> with others services as well

Well, I believe we did rewrite the importer in 1.2 or 1.3 to be more memory efficient.  Its probably not possible to uplift such a large change to 1.1hd, though.  We may not be able to fix this without helix moving to a more recent release.

Comment 20

[Ben Kelly [:bkelly, not reviewing]]

I received the login information.  Thank you Marina.  I will try to test this when I can, but due to some other high priority bugs I have it may not happen till next week.

Comment 21

[Ben Kelly [:bkelly, not reviewing]]

One possible fix which greatly helped in v1.3 was bug 904973.  That bug rewrote the importer to be much more memory efficient.

From comment 16, however, it appears v1.2 also works.  I'll see what possible changes might have landed there to help.

Comment 22

[Ben Kelly [:bkelly, not reviewing]]

All, I'm sorry I never got back to this.  Basically this should be resolved in later releases.  If this is critical for v1.1hd at this point someone needs to spend time to figure out what needs to be backported.  I am just continually busy on v1.3/v1.4 work and won't realistically have time to do this.