[New Tab Page] don't immediately preload the new tab page on startup

RESOLVED FIXED in Firefox 18

Status

()

defect
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: ttaubert, Assigned: ttaubert)

Tracking

(Depends on 1 bug)

Trunk
Firefox 18
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

After delayed startup, we should wait for a couple of seconds (like the download manager does before resuming) until we start preloading new tab pages in the background. This feature must not interfere with startup times.
(Assignee)

Updated

7 years ago
Blocks: 791670
(Assignee)

Updated

7 years ago
Assignee: nobody → ttaubert
Status: NEW → ASSIGNED
Posted patch patch v1Splinter Review
Attachment #662564 - Flags: review?(jaws)
Comment on attachment 662564 [details] [diff] [review]
patch v1

Review of attachment 662564 [details] [diff] [review]:
-----------------------------------------------------------------

r+ with nit addressed.

Thanks for fixing this. Hopefully this is the last remaining piece before we can enable the preload pref.

::: browser/modules/BrowserNewTabPreloader.jsm
@@ +58,5 @@
> +    }
> +  },
> +
> +  observe: function Initializer_observe(aSubject, aTopic, aData) {
> +    if (aTopic == "browser-delayed-startup-finished") {

if (aTopic == TOPIC_DELAYED_STARTUP) {
Attachment #662564 - Flags: review?(jaws) → review+
A hardcoded 5s timer is a bit gross, can we file a followup to use some better mechanism (perhaps dependent on bug 712478)?
(In reply to :Gavin Sharp (use gavin@gavinsharp.com for email) from comment #3)
> A hardcoded 5s timer is a bit gross, can we file a followup to use some
> better mechanism (perhaps dependent on bug 712478)?

Yeah I didn't like that too much either but I think we don't have a real alternative at the moment. The download panel does the same with a 10s timer to resume previous downloads. I'll file a follow-up but I'm not sure if bug 712478 can help us with that. Wouldn't we need some event/notification that tells us when user is *not* doing something? Or when the browser has finished doing the important stuff after startup?
(Assignee)

Updated

7 years ago
Depends on: 793115
https://hg.mozilla.org/mozilla-central/rev/9c75ed3092e8
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 18
You need to log in before you can comment on or make changes to this bug.