Closed
Bug 123803
Opened 23 years ago
Closed 23 years ago
mozilla crashes when scrolling or filling in a form after downloading page with javascript error
Categories
(Core :: Layout, defect, P1)
Tracking
()
VERIFIED
WORKSFORME
mozilla1.0
People
(Reporter: rein, Assigned: attinasi)
Details
(Keywords: testcase)
Attachments
(6 files)
When clicking on a link resulting in downloading a page with sources as giving
in the attachment, both scrolling the page, or trying to fill in the form
results in a crash of Mozilla.
The javascript console gives the following error messages:
Error: document.getElementById("helptext").currentStyle has no properties
Source File: http://abba.eccoo.rug.nl/bambix/addcourse.php?edit=INFL.2001-2002.1
Line: 150
Error: document.getElementById("helptext").currentStyle has no properties
Source File: http://abba.eccoo.rug.nl/bambix/addcourse.php?edit=INGA
Line: 150
Error: redeclaration of const hide
Source File: chrome://wallet/content/walletOverlay.js
Line: 1
etc.
The messages above appear after downloading the page.
What happens during the crash has been captures by the talkback agent, see
the incidents with ID's TB2572354Q and TB2571850K
Reproducable: always
URL: unfortunately, the urls are behind a password and used for local applications.
Reporter | ||
Comment 1•23 years ago
|
||
Reporter | ||
Comment 2•23 years ago
|
||
Additional comment: I am using Build 2002020406.
Updated•23 years ago
|
Attachment #68120 -
Attachment mime type: text/html → text/plain
Comment 3•23 years ago
|
||
Comment 4•23 years ago
|
||
Comment 5•23 years ago
|
||
Confirming crash on Attachment #1 [details] [diff] above. All the stack traces look
similar; based on them, reassigning to Layout component -
Assignee: rogerl → attinasi
Status: UNCONFIRMED → NEW
Component: JavaScript Engine → Layout
Ever confirmed: true
QA Contact: pschwartau → petersen
Comment 6•23 years ago
|
||
Taking bug. Marking nsbeta1+
Keywords: nsbeta1+
Target Milestone: --- → mozilla1.0
Updated•23 years ago
|
Priority: -- → P1
Updated•23 years ago
|
Whiteboard: creating testcase
Comment 7•23 years ago
|
||
Comment 8•23 years ago
|
||
Phil,
I reduced the test to a mininium number of functions in the script element (same
thing done with document's HTML elements). Could you take a look at the file ?
Comment 9•23 years ago
|
||
Chris: I was able to reduce the testcase even further and still get
the same stack trace as in Comment #4 above. Will attach this below -
Comment 10•23 years ago
|
||
Comment 11•23 years ago
|
||
To expand on my setup: using WinNT Mozilla trunk binary 2002021413
and a WinNT debug build 2002-02-10.
I can't add any more from the JS Debugger, because the reduced test
crashes on load, before I can set a breakpoint. At any rate, the
WinNT c++ stack trace in Comment #4 alwayss come up in any debug build
as soon as you load the test -
Comment 12•23 years ago
|
||
Here is the HTML source of the reduced test in Comment #10:
<html>
<head>
<title>Bug 123803</title>
<script>
function onLoadActions()
{
document.getElementById('cursusgegevens').style.display = 'none';
document.getElementById('periodeaanduiding').style.display = 'none';
}
</script>
</head>
<body onload="onLoadActions()">
<div id='cursusgegevens'>
<input id='periodeaanduiding'>
</div>
<input type='button'>
</body>
</html>
Comment 13•23 years ago
|
||
Note I get almost the same exact stack trace as in Comment #4 above
at http://software.xfx.net/uboards/indexp.php?forum=uboard_main.htm
when I mouseover the "DHTML Menu Builder" menu item at the top left.
Exact steps in http://bugzilla.mozilla.org/show_bug.cgi?id=125318#c9
One of the differences in the stack traces is the presence here of
.
.
DeviceContextImpl::GetMetricsFor() line 303
nsTextFrame::TextStyle::TextStyle() line 553
nsTextFrame::Paint() line 1429
whereas at that site I get:
.
.
DeviceContextImpl::GetMetricsFor() line 303
ComputeLineHeight() line 2179 + 63 bytes
nsHTMLReflowState::CalcLineHeight() line 2219
Will attach full stack trace below -
Comment 14•23 years ago
|
||
Comment 15•23 years ago
|
||
I'm not getting the crash with today's build on the test cases or url. If this
is still crashing, please be specific on how to recreate it.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → WORKSFORME
Reporter | ||
Comment 16•23 years ago
|
||
With 2002031104 no crashes any more so that problem seems to be fixed.
However, also no additonal texts and such appear after selecting the appropriate
radiobuttons and javascript console says:
Error: document.getElementById("helptext").currentStyle has no properties
Source File: http://abba.eccoo.rug.nl/bambix/forms/addcourse.php?edit=INPA99
Line: 62
I imagine this is because non-standard solutions are used in the script??
(It worked on previous versions of Mozilla)
Comment 17•23 years ago
|
||
Marking verfied wfm in the April 23rd OS X (2002-04-23-08) and Windows ME
(2002-04-23-06).
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•