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

RESOLVED FIXED in mozilla0.9

Status

defect
RESOLVED FIXED
19 years ago
15 years ago

People

(Reporter: hyatt, Assigned: jag+mozbugs)

Tracking

({perf})

Trunk
mozilla0.9
x86
Windows 2000
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments)

Reporter

Description

19 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

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

Comment 1

19 years ago
Related to bug 66034?
Reporter

Comment 3

19 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

19 years ago
Assignee

Comment 5

19 years ago
r=jag

Comment 6

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

Comment 7

19 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

19 years ago
Reporter

Comment 9

19 years ago
Need a fresh r and sr.
Assignee

Comment 10

19 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?
Reporter

Comment 12

19 years ago
sr=hyatt

Comment 13

19 years ago
r=hwaara
Reporter

Comment 14

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

Comment 16

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

Updated

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