JS include requested twice if it results in an error page

NEW
Unassigned

Status

()

7 years ago
2 months ago

People

(Reporter: kdevel, Unassigned)

Tracking

Trunk
Other
Other
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

7 years ago
User Agent:  

Steps to reproduce:

0. New Profile. Set HTTP-Proxy (e.g. squid)
1. Open Web Console (shift-ctrl-k).
2. Open Testcase.


Actual results:

2. URL 404.js is requested twice.


Expected results:

2. URL 404.js is requested once.
Component: Untriaged → HTML: Parser
Product: Firefox → Core
QA Contact: untriaged → parser
PrepareLoadedRequest returns an error for failed loads.  At that point we remove the load from mPreloads.  So if we get the 404 response before default.js finishes executing, we'll hit the 404 twice.

Perhaps we should move failed preloads (at least if they fail due to a non-2xx HTTP code) to a list that we check when the real load comes in?
Status: UNCONFIRMED → NEW
Component: HTML: Parser → DOM
Ever confirmed: true
QA Contact: parser → general
Summary: JS include requested twice → JS include requested twice if it results in an error page
Sounds like the right solution yes.

Henri, you've been doing a lot of preload work lately. Could you take this?
(In reply to Jonas Sicking (:sicking) On leave waiting for visa from comment #3)
> Henri, you've been doing a lot of preload work lately. Could you take this?

OK.
Assignee: nobody → hsivonen
Status: NEW → ASSIGNED
Assignee: hsivonen → nobody
Status: ASSIGNED → NEW
You need to log in before you can comment on or make changes to this bug.