Open Bug 718298 Opened 12 years ago Updated 2 years ago

JS include requested twice if it results in an error page

Categories

(Core :: DOM: Core & HTML, defect)

Product:
Core
Component:
DOM: Core & HTML
Platform:
Other
Other
Type:
defect

Tracking

()

People

(Reporter: kdevel, Unassigned)

References

(
URL
)

Details

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
Component: DOM → DOM: Core & HTML
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.