about:blank loaded twice when opening a blank new nav window!

RESOLVED FIXED in mozilla0.9

Status

defect
RESOLVED FIXED
18 years ago
15 years ago

People

(Reporter: hyatt, Assigned: jag+mozbugs)

Tracking

({perf})

Trunk
mozilla0.9
x86
Windows 2000

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments)

(Reporter)

Description

18 years ago
All new nav windows start off with about:blank loaded.  In the case where 
you've opened a new blank window, we load about:blank a second time!
(Reporter)

Updated

18 years ago
Status: NEW → ASSIGNED
Keywords: perf
Target Milestone: --- → mozilla0.9

Comment 1

18 years ago
Related to bug 66034?
(Reporter)

Comment 3

18 years ago
Wow.  There are two other code paths that cause this to happen.  The way box
interacts with iframe on a reflow causes a double-load, and the navigator JS
triggers a reload (which should never do anything for about:blank).

The following patch fixes all cases and ensures that about:blank is exempt from
all this madness.
(Reporter)

Comment 4

18 years ago
(Assignee)

Comment 5

18 years ago
r=jag

Comment 6

18 years ago
sr=scc (based on the notion that |SchemeIs| will be getting better, so playing
to its strengths is wise)
(Reporter)

Comment 7

18 years ago
Turns out editor needs the double load... they have to monitor loads of docs,
even about:blank.  So I'm patching navigator only.  The fix is to navigator.js
instead of to docshell.
(Reporter)

Comment 8

18 years ago
(Reporter)

Comment 9

18 years ago
Need a fresh r and sr.
(Assignee)

Comment 10

18 years ago
The change to navigator.js is also needed for bug 70682. r=jag on this.

What's the patch to BrowserInstance for? Accidentally included?
(Assignee)

Comment 11

18 years ago
(Reporter)

Comment 12

18 years ago
sr=hyatt

Comment 13

18 years ago
r=hwaara
(Reporter)

Comment 14

18 years ago
Reassigning to jag.
Assignee: hyatt → disttsc
Status: ASSIGNED → NEW
a=asa on (behalf of drivers)
(Assignee)

Comment 16

18 years ago
Checked in on both the trunk and the branch. Marking fixed.
Status: NEW → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Updated

18 years ago
Blocks: 49141
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.