1093943 - [Settings] When opening the 'Settings' app, the 'Theme' category takes several seconds to appear

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::Settings, defect)

Description

[Robert Mead [:RobertM]]

Attachment: Flame2.1logcat.txt

Description:
When opening the settings app, if you scroll down to the bottom of the Personalization category, the 'Theme' row will take several seconds to appear.

Repro Steps:
1) Update a Flame device to BuildID: 20141104041844
2) Tap the settings app
3) Quickly scroll down to the bottom of the Personalization category and observe
  
Actual:
After a few seconds the theme category will be there
  
Expected: 
The Theme category should already be there


Flame 2.1(319mb)(KitKat)(Shallow Flash)(Engineer Build)

Environmental Variables:
Device: Flame 2.1
BuildID: 20141104070645
Gaia: 9d80d556123553e5fd8acd982192e33807e9e1fe
Gecko: 0953b46ee9a8
Version: 34.0 (2.1)
Firmware: V188-1
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

  
Repro frequency: 100%
See attached: logcat, video - http://youtu.be/8yU2RvkzNi4

Comment 1

[Robert Mead [:RobertM]]

This issue also occurs on Flame 2.2(319mb), although it loads much faster then v2.1

After opening the settings app, the 'Theme' row takes a few seconds before it appears


Flame 2.2(319mb)(KitKat)(Shallow Flash)(Engineer Build)

Environmental Variables:
Device: Flame 2.2 Master
BuildID: 20141104041844
Gaia: 3c50520982560ccba301474d1ac43706138fc851
Gecko: cadcd47db610
Version: 36.0a1 (2.2 Master)
Firmware: V188-1
User Agent: Mozilla/5.0 (Mobile; rv:36.0) Gecko/36.0 Firefox/36.0

Comment 2

[Tony Chung [:tchung]]

when bug 1086167 is fixed and landed, please verify if this bug is also fixed.

Comment 3

[EJ Chen [:eragonj][:小龍哥][Contributor mode on](inactive)]

In these sub-panels, we have to get all installed apps and try to filter out needed ones to decide whether we want to show the panel or not.

If you check bug 1089459 (2.1+ blocker), you would notice that what we did there is to move the `mgmt.getAll()` timing later to make load timer better. To be honest, no matter how we tweak, there would always be the small gap between showing up the panel.

And for master patch (bug 1086167), what we tried to do is call `mgmt.getAll()` only once and the other callers would only use the caches apps to do anything they want (and this doesn't fix the gap either !)

The proper fix for this kind of bugs should be handled in Gecko and they have to provide a better API like 

```
mgmt.getAll({
  type: 'theme'
})
```

and fetch out needed apps from indexes instead of iterating all apps. (There would still be a small gap, but Gecko can speed up the process and return apps quicker than before)
--

For now, if this bug is a blocker, please set the ? flag on it and let release mangers decide what we want here - (A) Root panel should be shown without delays with bad load time (B) Load time performance is crucial than the others

For now, option B) is the current decision and we already did a lot of efforts on this in Settings app.

Comment 4

[EJ Chen [:eragonj][:小龍哥][Contributor mode on](inactive)]

I filed a bug 1094562 for Gecko and hope we can get faster API for these cases :)

Comment 5

[Derek Harris [:DerekH] - Dont need info. Last day 6/5/15]

Not nominating to block on this issue becuase the themes section does load, and settings does a lazy load purposefully

Comment 6

[BMO Automation]

Firefox OS is not being worked on