Closed
Bug 1839
Opened 26 years ago
Closed 26 years ago
frame reference
Categories
(Core :: Layout: Images, Video, and HTML Frames, defect, P2)
Tracking
()
VERIFIED
FIXED
M7
People
(Reporter: godfrey.lee, Assigned: joki)
References
Details
From the window containing the frameset, you can refer to window.frames["FX"]
but not window.FX.
See code below:
FR.htm
------
<HTML>
<HEAD>
<SCRIPT LANGUAGE=JavaScript>
window.name="FR";
var FXloaded=0;
var FTloaded=0;
function sync(){
if(FXloaded==0 || FTloaded==0)
return;
alert("sync");
window.frames["FT"].sync();
self.FX.sync()}
</SCRIPT>
</HEAD>
<FRAMESET onUnload="defaultStatus=''" ROWS="*,50" frameborder=0 border=0>
<FRAME NAME=FX MARGINHEIGHT=0 MARGINWIDTH=0 SCROLLING=auto SRC=FX.htm>
<FRAME NAME=FT MARGINHEIGHT=10 MARGINWIDTH=0 SCROLLING=no SRC=FT.htm>
</FRAMESET>
<BODY>
</BODY>
</HTML>
FX.htm
------
<HTML>
<HEAD>
<SCRIPT LANGUAGE="JavaScript">
<!--
function sync(){
alert("FX Loaded");
}
function init(){
if(parent.FXloaded!=null){
parent.FXloaded=1;
parent.sync();
}
}
//-->
</SCRIPT>
</HEAD>
<BODY onLoad="init();">
First Frame
</BODY>
</HTML>
FT.htm
------
<HTML>
<HEAD>
<SCRIPT LANGUAGE=JavaScript>
<!--
function sync(){
alert("FT Loaded");
}
function init(){
if(parent.name=="FR"){
parent.FTloaded=1;
parent.sync();
}
}
//-->
</SCRIPT>
</HEAD>
<BODY onLoad="init();">
Second Frame
</BODY>
</HTML>
|
||
Updated•26 years ago
|
Assignee: mccabe → vidur
|
|
||
Comment 1•26 years ago
|
||
This sounds like one of those navigator-and-ie-aren't-the-same issues.
They're different.
But perhaps this is a DOM level 1 conformance issue, if the bug is against the
new layout browser.
Vidur, care to comment?
(Also, shouldn't there be a 'Document Object Model' component in bugsplat to
assign things to, to correspond with the entry in owners.html? I'd like to
assign this bug to that component.)
|
||
Updated•26 years ago
|
Assignee: vidur → joki
|
|
||
Comment 2•26 years ago
|
||
Tom, I'm reassigning this to you. You can get window.FX working by introducing a
Window NamedItem(in DOMString name) for the Window interface.
|
|
||
Comment 4•26 years ago
|
||
Well, I actually did a trial implementation by adding NamedItem to Window. The
good news is that it works. The bad news is that it's horribly inefficient since
a lot of stuff goes through a window's getProperty() and we're doing unnecessary
lookups for each property lookup, frame-related or otherwise. We need to come up
with a better implementation of this.
|
||
Updated•26 years ago
|
QA Contact: 4590
|
|
||
Comment 7•26 years ago
|
||
per leger, assigning QA contacts to all open bugs without QA contacts according
to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser
QA contact re-assigned according to the product areas we're currently working
on.
|
Assignee | |
Updated•26 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M4 → M5
|
|
Assignee | |
Comment 10•26 years ago
|
||
Moving this to M5. I won't get to it tomorrow.
|
|
||
Comment 11•26 years ago
|
||
*** Bug 3011 has been marked as a duplicate of this bug. ***
|
|
Assignee | |
Updated•26 years ago
|
Target Milestone: M5 → M6
|
|
Assignee | |
Updated•26 years ago
|
Target Milestone: M6 → M7
|
|
Assignee | |
Comment 12•26 years ago
|
||
Moving out to M7
|
Reporter | |
Comment 13•26 years ago
|
||
Hi guys,
I would like to know if this bug is currently fixed but you are keeping it open
so you can provide a better implementation later.
I have a large DHTML application that I would like to start testing on
gecko/mozilla, and would like to know if I need to avoid this construct or not.
Thanks,
Godfrey Lee
|
|
||
Comment 14•26 years ago
|
||
*** Bug 6789 has been marked as a duplicate of this bug. ***
|
|
Assignee | |
Comment 15•26 years ago
|
||
I believe this one is indeed still broken.
|
|
||
Comment 16•26 years ago
|
||
*** Bug 7647 has been marked as a duplicate of this bug. ***
|
|
Assignee | |
Comment 17•26 years ago
|
||
*** Bug 7260 has been marked as a duplicate of this bug. ***
|
|
Assignee | |
Comment 18•26 years ago
|
||
Got a fix. Waiting for a slightly greener tree to put it in.
|
|
Assignee | |
Updated•26 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 19•26 years ago
|
||
Fix checked in.
|
||
Updated•26 years ago
|
Status: RESOLVED → VERIFIED
|
|
||
Comment 20•26 years ago
|
||
Working fine now. Marining as verified.
|
||
Updated•6 years ago
|
Product: Core → Core Graveyard
|
||
Updated•6 years ago
|
Component: Layout: HTML Frames → Layout: Images
Product: Core Graveyard → Core
You need to log in
before you can comment on or make changes to this bug.
Description
•