Closed
Bug 63137
Opened 24 years ago
Closed 21 years ago
[FIX]View Source converts tags to lower case
Categories
(Core :: DOM: HTML Parser, defect, P3)
Core
DOM: HTML Parser
Tracking
()
VERIFIED
FIXED
mozilla1.6beta
People
(Reporter: stevehalasz, Assigned: bzbarsky)
References
()
Details
(Keywords: regression, testcase)
Attachments
(2 files)
(deleted),
text/html
|
Details | |
(deleted),
patch
|
choess
:
review+
hjtoi-bugzilla
:
superreview+
|
Details | Diff | Splinter Review |
1. Go to http://www.hotelschool.cornell.edu/
2. View->Page Source
What happens: The page source is displayed with the opening and closing text of
the HTML tags converted to lower-case, like:
<a HREF="http://www.hotelschool.cornell.edu/publications/news/">...</a>
What should happen: The tags should remain upper-case, like:
<A HREF="http://www.hotelschool.cornell.edu/publications/news/">...</A>
Assignee | ||
Comment 1•24 years ago
|
||
I do not see this with a linux 2000-12-05-08 nightly. I do see it with a linux
2000-12-15 CVS build. adding regression keyword, OS -> All. Could be a parser
problem.
Comment 3•24 years ago
|
||
worksforme - ziz@ziz.org
Comment 4•24 years ago
|
||
whups, somehow the qa contact never got filled. chris, d'you see this? if not,
pls mark wfm. thx!
QA Contact: petersen
Comment 5•24 years ago
|
||
hrm, actually this is still a problem. will attach simple testcase.
Comment 6•24 years ago
|
||
try parser.
Assignee: ben → harishd
Component: Layout → Parser
QA Contact: petersen → janc
This bug has been marked "future" because the original netscape engineer working
on this is over-burdened. If you feel this is an error, that you or another
known resource will be working on this bug,or if it blocks your work in some way
-- please attach your concern to the bug for reconsideration --.
Status: NEW → ASSIGNED
Target Milestone: --- → Future
Comment 10•24 years ago
|
||
*** Bug 78621 has been marked as a duplicate of this bug. ***
Comment 11•23 years ago
|
||
It appears that Mozilla (linux, build id 2001062823 & earlier) presents the
source as it appears after it has been parsed, often incorporating "corrections"
made by the parser. Another example is if you leave the closing quote of an
attribute value (e.g. <p align="left> ) Mozilla will fill it in when
you view the source.
I think this is bad because I normally view the source so I can debug the
source! In the worst case you can create an unrenderable page, look at the
source, see there's nothing there (because the parser has thrown it all out),
and make the not unreasonable assumption that CGI script is not returning
anything (this has happened to me, sorry no examples).
Assignee | ||
Comment 12•23 years ago
|
||
paynter@infomine.ucr.edu, we have to parse the source to usefully highlight
it... And there are bugs on all sorts of problems that arise as a result, this
being one of them. If we decide to drop syntax highlighting, all these bugs
would disappear, of course.
Comment 13•23 years ago
|
||
*** This bug has been marked as a duplicate of 57724 ***
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
Assignee | ||
Comment 15•23 years ago
|
||
*** Bug 110095 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 16•23 years ago
|
||
*** Bug 115172 has been marked as a duplicate of this bug. ***
Comment 17•23 years ago
|
||
Reopening 57724 dependencies for independent resolution.
Comment 18•23 years ago
|
||
This bug still happens with Mozilla 0.9.9 for me on Red Hat Linux.
Ideally, the bug could be fixed, but it seems like a low priority.
Failing an outright fix, the parsing/highlighting should be removed and the
original source displayed as a plain text file.
It is more important to have the correct source than incorrect colorful source.
Comment 19•23 years ago
|
||
seeing this on: 2002042608 winXP
Comment 20•22 years ago
|
||
*** Bug 169450 has been marked as a duplicate of this bug. ***
Comment 21•22 years ago
|
||
*** Bug 169906 has been marked as a duplicate of this bug. ***
Comment 22•21 years ago
|
||
*** Bug 208264 has been marked as a duplicate of this bug. ***
Comment 23•21 years ago
|
||
*** Bug 210372 has been marked as a duplicate of this bug. ***
Updated•21 years ago
|
Flags: blocking1.6b+
Comment 24•21 years ago
|
||
only drivers can set blocking+ flags. if you want to request it, set it to ?
also, reassigning to default because @netscape.com doesn't exist anymore which
means both the assignee and qacontact on this bug are invalid.
Assignee: harishd → parser
Status: REOPENED → NEW
Flags: blocking1.6b+
Comment 25•21 years ago
|
||
sorry. i viewed the help and there was no mention. figured if i could change it,
then i had permissions to do so. my mistake won't happen again.
i've changed it to requested. hope I'm not spamming.
Flags: blocking1.6b?
Assignee | ||
Comment 26•21 years ago
|
||
The simple problem here is that for known tags the token stores the tag id, not
the actual string...
Assignee | ||
Comment 27•21 years ago
|
||
Assignee | ||
Comment 28•21 years ago
|
||
Comment on attachment 135098 [details] [diff] [review]
This fixes the bug, as far as I can tell
Choess, heikki, what do you think?
Attachment #135098 -
Flags: superreview?(hjtoi-bugzilla)
Attachment #135098 -
Flags: review?(choess)
Comment 29•21 years ago
|
||
Comment on attachment 135098 [details] [diff] [review]
This fixes the bug, as far as I can tell
Wow, why didn't we do this before? Looks good to me, although I can't build
right now. Be aware that the GetIdentifier routine will consume the characters
:, -, _, and alphanumerics; upon hitting any other character, it returns what
it's consumed so far. But case-folding alone should be resolved by this.
Attachment #135098 -
Flags: review?(choess) → review+
Assignee | ||
Comment 30•21 years ago
|
||
Taking.
Assignee: parser → bz-vacation
Priority: -- → P3
Summary: View Source converts tags to lower case → [FIX]View Source converts tags to lower case
Target Milestone: Future → mozilla1.6beta
Updated•21 years ago
|
Attachment #135098 -
Flags: superreview?(hjtoi-bugzilla) → superreview+
Assignee | ||
Comment 31•21 years ago
|
||
Fixed.
Status: NEW → RESOLVED
Closed: 23 years ago → 21 years ago
Resolution: --- → FIXED
Updated•21 years ago
|
Flags: blocking1.6b?
Comment 32•21 years ago
|
||
*** Bug 226492 has been marked as a duplicate of this bug. ***
Comment 33•21 years ago
|
||
vrfy fixed 2004011809 (url, testcase, data:text/html,<A><bB></Qq></a>)
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•