Closed Bug 1048543 Opened 10 years ago Closed 10 years ago

[Vertical Homescreen] Defer saving of grid until after initial render

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(b2g-v2.1 wontfix, b2g-v2.2 fixed)

RESOLVED FIXED
2.1 S5 (26sep)
Tracking Status
b2g-v2.1 --- wontfix
b2g-v2.2 --- fixed

People

(Reporter: kgrandon, Assigned: kgrandon)

Details

(Whiteboard: [systemsfe])

Attachments

(1 file)

Currently we try to save to indexedDB after each source synchronizes. We should be able to shave some time if we save only after the initial render.
Attached file Github pull request
[Blocking Requested - why for this release]: This should save ~100ms on startup.
blocking-b2g: --- → 2.0?
BLocking as this will help the CAF issue in 1048543
blocking-b2g: 2.0? → 2.0+
Whiteboard: [systemsfe]
Comment on attachment 8467325 [details] [review] Github pull request Flagging a few people for review. Chris, I think Cristian/Carmen might be on PTO - if you have a few minutes to help with a review I would appreciate it.
Attachment #8467325 - Flags: review?(crdlc)
Attachment #8467325 - Flags: review?(chrislord.net)
Attachment #8467325 - Flags: review?(carmen.jimenezcabezas)
Comment on attachment 8467325 [details] [review] Github pull request Comments on the pull request.
Attachment #8467325 - Flags: review?(chrislord.net) → review+
Comment on attachment 8467325 [details] [review] Github pull request At first look the pr looks good. It works well when SV isn't used and when the system is installed from scratch (that is, it's not upgraded via fota). But I tested it on a fota scenario and it didn't work (vertical home started up without any apps). For that reason I cannot give the r+ here, sorry. I'm searching for the reason of failure now.
Attachment #8467325 - Flags: review?(carmen.jimenezcabezas)
Carmen - Thanks for testing. Can you tell me what version you upgraded fota over? Was it 1.3 -> 2.0 or 2.0 -> 2.0 for example? Any chance you saw anything in logcat? I will also try to emulate this manually soon.
Flags: needinfo?(carmen.jimenezcabezas)
I've tested it from v1.4 -> master and v1.3 -> master with the same result (I'm afraid). To discard fota errors, I had tested a update from v1.4 -> master without the current patch too. The logcat says nothing
Flags: needinfo?(carmen.jimenezcabezas)
After looking at this again, the main gains here are only for the very first run, and not subsequent runs. I think we may be able to get some additional time out of this, but I don't think we should shoot for 2.0 as it possibly increases risk to fota updates. Unblocking for now, but will try to re-measure to get some statistics here.
No longer blocks: 1037706
blocking-b2g: 2.0+ → ---
Comment on attachment 8467325 [details] [review] Github pull request Removing my review because I've seen that Chris and Carmen are doing it. Thanks
Attachment #8467325 - Flags: review?(crdlc)
Target Milestone: 2.1 S2 (15aug) → 2.1 S3 (29aug)
Kevin, do we still need this?
Flags: needinfo?(kgrandon)
(In reply to Gregor Wagner [:gwagner] from comment #11) > Kevin, do we still need this? Yes, it's something I definitely want. I think the OTA issue should be resolved now as it was something we were seeing before but went away. I would like to verify this before landing though.
Flags: needinfo?(kgrandon)
Target Milestone: 2.1 S3 (29aug) → 2.1 S5 (26sep)
Unfortunately I am having trouble generating an FOTA update locally, but I am fairly confident in landing this for a few reasons: 1 - I distinctly remember a blank screen after upgrade bug that was resolved in 2.0, but I can't find the bug number. 2 - 1.x -> 2.2 upgrades are not supported, so it should be fine to land this in master and let it ride the 2.2 train. I'm going to go ahead and land and watch upgrades/keep this bug in mind to make sure it doesn't cause any problems.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: