location object needs toString() method

VERIFIED FIXED

Status

()

Core
DOM
P2
normal
VERIFIED FIXED
19 years ago
11 years ago

People

(Reporter: Michael Lowe, Assigned: vidur (gone))

Tracking

Trunk
x86
Windows NT
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

19 years ago
The bottom frame in this page does not load.  Seems to be a DOM problem.
(Assignee)

Updated

19 years ago
Status: NEW → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → WORKSFORME
(Assignee)

Comment 1

19 years ago
Works for me. There is script on the page and it's doing the right thing (except
for the fact that Image constructors still don't preload images).
(Reporter)

Updated

19 years ago
Status: RESOLVED → REOPENED
(Reporter)

Comment 2

19 years ago
This page still causes problems for me.  Here is a simple test case which
illustrates the bug:

<html>
<head>
</head>
<body>
	<script>
	document.write(parent.location);
	</script>
</body>
</html>

With my build of nglayout, this page displays "[object location]" instead of the
page's URL.    This affects the following script in the www.village.com.au page:

                <script>
                        // Written by Phil and Symonty. Copyright Village
Roadshow, 1998
                        // 18/2/1998
                        //
                        // JavaScript to generate ad frames. Allows for nested
frames too :)
                        var middleFound = 0;
                        var i;
                        // Is there a frame called middle?
                        for (i = 0; i < top.frames.length; i++)

                                if (top.frames[i].name == "middle")
                                {
                                        middleFound = 1;
                                        break;
                                }
                        // Load the pages into the frameset
                        if (!middleFound)
                        {

top.location="http://www.village.com.au/cgi-bin/fp_frame.cgi?"+parent.location;
                        }
                        else
                        {

top.window.vil_banners.location="http://www.village.com.au/bannerdocs/fp_village
ad.htm";
                        }
                </script>
(Assignee)

Updated

19 years ago
Status: REOPENED → ASSIGNED
(Assignee)

Comment 3

19 years ago
Yup, you're right. I probably pasted the generated URL from Navigator while I
was debugging this.
(Assignee)

Updated

19 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago19 years ago
Resolution: WORKSFORME → FIXED
(Assignee)

Comment 4

19 years ago
Fixed on 1/6/1999. Added a toString method to the location object.
(Reporter)

Updated

19 years ago
Status: RESOLVED → VERIFIED

Comment 5

11 years ago
RCS file: /cvsroot/mozilla/testing/mochitest/tests/test_bug1823.html,v
done
Checking in test_bug1823.html;
/cvsroot/mozilla/testing/mochitest/tests/test_bug1823.html,v  <--  test_bug1823.html
initial revision: 1.1
done
Flags: in-testsuite+
Summary: Frame does not load → location object needs toString() method
You need to log in before you can comment on or make changes to this bug.