860097 - DOMApplicationRegistry: get list of apps lazily

Status: RESOLVED WONTFIX

(Core Graveyard :: DOM: Apps, defect, P4)

Description

[Gregor Wagner [:gwagner]]

We spend a little bit more than 0.5 sec getting the list of applications during a high workload scenario. This shows up during profiling incoming calls:

http://people.mozilla.com/~bgirard/cleopatra/#report=128e2cf57e726e038e7d4a05c0e72754db02a44f

http://mxr.mozilla.org/mozilla-central/source/dom/apps/src/AppsServiceChild.jsm#35

Comment 1

[Daniel Coloma:dcoloma]

Gregor, should we block on this? It seems it blocks a blocker...

Comment 2

[Justin Lebar (not reading bugmail)]

The fact is, we don't know yet what we need to do to fix bug 847592.  I'm a bit skeptical that this is needed or will help a lot.  But I was really wrong last time I made a similar guess.

Rather than mark a bunch of bugs as tef+ on suspicion they may be needed to fix a blocker, I'd rather that you give someone permission to tef+ whatever is actually needed to fix bug 847592.  I already have permission in bugzilla to do so.  Does that work for you?

Comment 3

[Daniel Coloma:dcoloma]

(In reply to Justin Lebar [:jlebar] from comment #2)
> The fact is, we don't know yet what we need to do to fix bug 847592.  I'm a
> bit skeptical that this is needed or will help a lot.  But I was really
> wrong last time I made a similar guess.
> 
> Rather than mark a bunch of bugs as tef+ on suspicion they may be needed to
> fix a blocker, I'd rather that you give someone permission to tef+ whatever
> is actually needed to fix bug 847592.  I already have permission in bugzilla
> to do so.  Does that work for you?

Absolutely, just copy me in those bugs

Comment 4

[[:fabrice] Fabrice Desré]

I don't know either if this fix will help. We will not spend the 0.5s that early, but at some point if we call any of the getXXX() function from AppsService we will need to do this ipc round trip. Only writing the patch will tell us!

Comment 5

[[:fabrice] Fabrice Desré]

Attachment: patch v1

Gregor, let me know if this helps.

Comment 6

[Dietrich Ayala (:dietrich)]

Was it concluded that this change is not necessary to help fix bug 847592?

Comment 7

[Justin Lebar (not reading bugmail)]

(In reply to Dietrich Ayala (:dietrich) from comment #6)
> Was it concluded that this change is not necessary to help fix bug 847592?

I don't think we'll be able to reach any conclusions until we establish that bug 847592 is fixed.  This is not high on my list of things to try, but I also think it can't hurt.

Comment 8

[Sylvestre Ledru [:Sylvestre]]

Core Graveyard / DOM: Apps is inactive. Closing all bugs in this component.