bugzilla.mozilla.org has resumed normal operation. Attachments prior to 2014 will be unavailable for a few days. This is tracked in Bug 1475801.
Please report any other irregularities here.

Newline and white space before doc. element is not getting reported to the DTD.

VERIFIED FIXED in M11

Status

()

Core
XML
P3
normal
VERIFIED FIXED
19 years ago
18 years ago

People

(Reporter: harishd, Assigned: Nisheeth Ranjan)

Tracking

({verifyme})

Trunk
x86
Windows NT
verifyme
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 years ago
1 <?xml version="1.0"?>
2 <?xml-stylesheet type="text/css"?>
3
4<!--
5  XXXXXXXXXXX
6  YYYYYYYYYYY
7  ZZZZZZZZZZZ
8-->
9<html xmlns="http://www.w3.org/TR/REC-html40" lang="en-US">
....

In the above sample, new line in line# 3, line# 8 is not getting reported to
CWellFormedDTD::HandleToken().
(Reporter)

Updated

19 years ago
Blocks: 10593
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
Target Milestone: M12
(Assignee)

Comment 1

19 years ago
Accepting bug and setting target milestone to M12 for now...
(Assignee)

Updated

19 years ago
Target Milestone: M12 → M14
(Assignee)

Comment 2

19 years ago
This is not a beta stopper.  Moving out to M14...
I think it would be good to fix this for beta, if possible, because of bug
10593.  Incorrect line numbers for JavaScript errors in XHTML are a rather
serious problem.  It makes things harder for developers, who are, IMO, one of
the major targets of a beta.  (I admit that the problems with JS errors on
linked scripts are even more serious - see bug 8045.)
(Assignee)

Updated

19 years ago
Target Milestone: M14 → M11
(Assignee)

Comment 4

19 years ago
David brings up a valid point.  Bringing in the bug to M11...
(Assignee)

Updated

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

Comment 5

19 years ago
I've checked in a fix for this bug.  I added the default handler to the XML
tokenizer which looks for newlines and creates newline tokens.  Earlier, the
no default handler was registered with expat, so newlines in the document prolog
were not getting reported.

Updated

19 years ago
Blocks: 17907

Comment 6

19 years ago
I'm trying to verify this fix. Line #3 and #8 were not getting reported - is
this something I can visually verify? Please advise
I *think* it's possible to verify it using the XML test case for bug 10593,
which it seems you already did.

Updated

19 years ago
Keywords: verifyme

Updated

18 years ago
No longer blocks: 17907

Comment 8

18 years ago
Fixed in the July 6th build.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.