Closed Bug 1256 Opened 27 years ago Closed 27 years ago

tags aren't flushed before processing scripts

Categories

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

Product:
Core
Component:
DOM: Core & HTML
Platform:
x86
Windows NT
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: buster, Assigned: vidur)

References

(
URL
)

Details

1. the page should display "yes" in the text field if the browser is javascript-enabled. 2. when you click on the button for "Javascript demo" and destroy the resulting window that is popped up, you crash with this stack: GlobalWindowImpl::~GlobalWindowImpl() line 102 + 23 bytes GlobalWindowImpl::`scalar deleting destructor'() + 20 bytes GlobalWindowImpl::Release() line 116 + 99 bytes nsWebShell::~nsWebShell() line 401 + 18 bytes nsWebShell::`scalar deleting destructor'() + 20 bytes nsWebShell::Release() line 434 + 110 bytes nsBrowserWindow::Close() line 1273 + 18 bytes HandleBrowserEvent() line 255 nsWindow::DispatchEvent() line 295 + 10 bytes nsWindow::DispatchWindowEvent() line 311 nsWindow::DispatchStandardEvent() line 326 + 15 bytes nsWindow::OnDestroy() line 1621 nsWindow::ProcessMessage() line 1284 nsWindow::WindowProc() line 354 + 27 bytes USER32! 77e71ab7() USER32! 77e71a77() NTDLL! 77f7624f() USER32! 77e7288d() USER32! 77e72918() nsWindow::WindowProc() line 364 + 31 bytes USER32! 77e71ab7() USER32! 77e71a77() NTDLL! 77f7624f() USER32! 77e7288d() USER32! 77e72918() nsWindow::WindowProc() line 364 + 31 bytes USER32! 77e71ab7() USER32! 77e71a77() NTDLL! 77f7624f() USER32! 77e7288d() USER32! 77e72918() nsWindow::WindowProc() line 364 + 31 bytes USER32! 77e71250()
Status: NEW → ASSIGNED
Severity: critical → major
Summary: 2 problems, 1 is a crash → tags aren't flushed before processing scripts
Committed a partial fix to get rid of the crash. Now a script context is created whenever a new script global object is created for a webshell. Updated the summary to reflect remaining problem.
Status: ASSIGNED → RESOLVED
Closed: 27 years ago
Resolution: --- → FIXED
Well, I fixed the bug, but the page source changed and is now incorrect (it doesn't work on Navigator 4.5 or IE either). It can be fixed by making a local version of the file and changing the form: <form name="yesno"> to <form name="YesNo">
Status: RESOLVED → VERIFIED
verified on build 98120413 on winnt 4.0
can't find the test case. denying in-testsuite.
Flags: in-testsuite-
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.