Closed
Bug 319312
Opened 20 years ago
Closed 19 years ago
Wrong breakpoint line numbers after <title> tag with newlines
Categories
(Core :: DOM: HTML Parser, defect)
Tracking
()
RESOLVED
DUPLICATE
of bug 331482
People
(Reporter: jed.anderson, Assigned: mrbkap)
Details
Attachments
(1 file)
|
58.97 KB,
image/png
|
Details |
Screenshot of Venkman being off by one line.
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8) Gecko/20051111 Firefox/1.5
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.12) Gecko/20050915
Run this HTML in the debugger:
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<title>
test.html
</title>
<script type="text/javascript">
<!--
function test() {
baz = 'funk';
}
-->
</script>
</head>
<body onload="test()">
This is my HTML page.
<br>
</body>
</html>
If you put a breakpoint on the line with "foo = 'bar';" it will not get hit. If you put breakpoints all throughout the method you notice that breakpoints on line 11 and 12 are installed, but the breakpoint on line 9 is not.
Reproducible: Always
Steps to Reproduce:
1. Debug the script in Venkman
2. Put breakpoints on lines 9-12
Actual Results:
Breakpoints are installed on lines 10, 11.
Expected Results:
The breakpoint should be installed on line 9.
I will attach a screenshot.
|
Reporter | |
Comment 1•20 years ago
|
||
|
||
Comment 2•20 years ago
|
||
This would be a problem with the DOM parser, which is feeding bad line numbers to the JS Engine.
Component: JavaScript Debugger → HTML: Parser
Product: Other Applications → Core
Version: unspecified → 1.0 Branch
|
|
Reporter | |
Comment 3•19 years ago
|
||
Can we move this to DOM parser land then?
|
|
||
Comment 4•19 years ago
|
||
Oops. I changed the component with my last comment, but bever changed the asignee.
Assignee: rginda → mrbkap
QA Contact: caillon → parser
|
Assignee | |
Comment 5•19 years ago
|
||
This is on the 1.8 branch, really? If so, then it's a dupe of bug 331482.
*** This bug has been marked as a duplicate of 331482 ***
Status: UNCONFIRMED → RESOLVED
Closed: 19 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•