Closed
Bug 816811
Opened 12 years ago
Closed 12 years ago
If a wifi network is configured, FTU experience gets laggy and horrible
Categories
(Firefox OS Graveyard :: Gaia::First Time Experience, defect, P2)
Firefox OS Graveyard
Gaia::First Time Experience
ARM
Gonk (Firefox OS)
Tracking
(blocking-basecamp:+)
People
(Reporter: cjones, Unassigned)
References
Details
Attachments
(1 file)
(deleted),
text/html
|
Details |
STR
(1) Run FTU
(2) Configure wifi network
The experience goes to crap. The UI can stop responding for seconds at a time.
Filing this separately from bug 813765 because I suspect this is going to start being a common complaint. The underlying cause is that configuring a wifi network takes the device online, which initiates an update poll that FTU's force-update-check queues.
We only get one chance to make a first impression, and if a user configures a network ours will be "barely usable junk". Putting it mildly.
Comment 1•12 years ago
|
||
Triage: Reluctant to block on this until we can see the experience following the fix to bug 813765.
blocking-basecamp: ? → -
Updated•12 years ago
|
Priority: -- → P4
Reporter | ||
Comment 2•12 years ago
|
||
(In reply to Dylan Oliver [:doliver] from comment #1)
> Triage: Reluctant to block on this until we can see the experience following
> the fix to bug 813765.
I disagree with this. We may not be able to fix bug 813765. In that unfortunate eventuality, I think this bug should be bb+ on its own for FTU-specific workarounds.
I really can't emphasize enough how ridiculously bad it is right now :(.
blocking-basecamp: - → ?
Comment 4•12 years ago
|
||
So, if this is really dependant on update polls as said before, and we are not sure we will be able to fix it on time, my workaround proposal on this specific bug is to disable the update process during the ftu, if possible. We could enable it after finishing it (or skipping it)
Reporter | ||
Comment 5•12 years ago
|
||
I agree, and probably even then only off a user-idle timer.
Comment 6•12 years ago
|
||
During the Taipei triage today, Thinker will provide some comments in bug 813765.
Comment 7•12 years ago
|
||
Mass Modify: All un-milestoned, unresolved blocking-basecamp+ bugs are being moved into the C3 milestone. Note that the target milestone does not mean that these bugs can't be resolved prior to 12/10, rather C2 bugs should be prioritized ahead of C3 bugs.
Target Milestone: --- → B2G C3 (12dec-1jan)
Comment 8•12 years ago
|
||
Is there any updates on this? cause I've been trying today before applying my patch (just turning off gaia.system.checkForUpdates flag on settings when entering ftu), and didn't noticed any change on performance on my tests (no SIM inserted, just connecting to wifi).
Flags: needinfo?(jones.chris.g)
Reporter | ||
Updated•12 years ago
|
Flags: needinfo?(philipp)
Comment 9•12 years ago
|
||
Bug 809947 made us do a lot less disk I/O for update checks. This definitely improves performance a lot. Actual updates still involve writing lots of cache metadata to disk (which happens synchronously on the main thread, unfortunately). There is a WIP by Thinker (bug 771420) to avoid those as much as possible, but it's bitrotted and not a patch Honza would like to take. It's unlikely we can do anything right now at the appcache level code (see discussion in bug 813765).
Flags: needinfo?(philipp)
Updated•12 years ago
|
Flags: needinfo?(jones.chris.g)
Comment 10•12 years ago
|
||
Fernando, if your patch is ready, I'd say this is still something nice to have; it makes sense to me to not check updates before FTU is finished anyway.
Comment 11•12 years ago
|
||
assigning to fcampo as he's already begun some work on this.
Assignee: nobody → fernando.campo
Comment 12•12 years ago
|
||
Yup, sorry for delay, I'll upload the patch asap.
Comment 13•12 years ago
|
||
Patch is simple, only set the update flag to false, and it gets enabled again after FTU with code on bootstrap.
But I still feel no improve on performance with this, so I wouldn't say it's fixed entirely with this, I think is more a gecko thing.
Attachment #693467 -
Flags: review?(fbsc)
Comment 14•12 years ago
|
||
I'm backing out the patch, as the settings app already default the value to false, so as Etienne points out in https://github.com/mozilla-b2g/gaia/pull/7069#issuecomment-11496834, the workaround only fixes not updated dogfooding phones (which I guess should be updated eventually). Still think this has to be fixed on lower level
Updated•12 years ago
|
Attachment #693467 -
Flags: review?(fbsc)
Updated•12 years ago
|
Assignee: fernando.campo → nobody
Comment 15•12 years ago
|
||
Since the update flag is configured as false already, and it looks like we cannot reproduce this issue anymore, should we close this issue?
Comment 17•12 years ago
|
||
(In reply to Julien Wajsberg [:julienw] from comment #16)
> can't reproduce it either.
Tried this yesterday and couldn't reproduce either with a wifi network requiring a password. Resolving as a worksforme.
Status: NEW → RESOLVED
Closed: 12 years ago
Flags: needinfo?(jsmith)
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•