Closed
Bug 155752
Opened 22 years ago
Closed 22 years ago
about: Use XML parser for XHTML services
Categories
(SeaMonkey :: UI Design, defect)
SeaMonkey
UI Design
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.1beta
People
(Reporter: alexeyc2003, Assigned: alexeyc2003)
References
Details
(Keywords: qawanted)
Attachments
(1 file, 1 obsolete file)
6.75 KB,
patch
|
alexeyc2003
:
review+
alexeyc2003
:
superreview+
alexeyc2003
:
approval+
|
Details | Diff | Splinter Review |
In bug 95768 I have converted most of about: services into XHTML 1.1. However they still take path through HTML parser instead of XML parser. I attaching patch for the following about: services to be treated as XML: about: about:mozilla about:cache-entry I left about:cache with HTML parser because we still do not have incremental XML content sink (see bug 18333).
Assignee | ||
Comment 1•22 years ago
|
||
Together with this patch the following files MUST be renamed: http://lxr.mozilla.org/mozilla/source/xpfe/global/resources/locale/en-US/about.html to about.xhtml http://lxr.mozilla.org/mozilla/source/xpfe/global/resources/content/mozilla.html to mozilla.xhtml The following file is an old version of mozilla.html and I don't know who uses it. http://lxr.mozilla.org/mozilla/source/l10n/us/xp/mozilla.html We might want to replace it with mozilla.xhtml from above. The about:cache-entry service stream is changed from text/html to application/xhtml+xml. The patch also HTML-escapes URLs in about:cache and about:cache-entry services to make them fully XHTML valid, and to avoid any possible security issues.
Assignee | ||
Comment 2•22 years ago
|
||
I think I've covered all the files needed for this change. Can someone please test this to see if I didn't miss anything. CCing leaf who is able to rename files while preserving their history.
Assignee | ||
Updated•22 years ago
|
Depends on: incrementalxml
Assignee | ||
Updated•22 years ago
|
Target Milestone: --- → mozilla1.1beta
Comment 3•22 years ago
|
||
Comment on attachment 90206 [details] [diff] [review] patch sr=jst
Attachment #90206 -
Flags: superreview+
Attachment #90206 -
Flags: review+
Who tests XHTML layout? They should be the QA owner. In fact, this should probably go to any XHTML related component.
Keywords: qawanted
Comment 5•22 years ago
|
||
so, about:config is still html, right? I have copied the ,v files; i'll let someone that knows more about the code do the checkin; jst? Alex, if you can't checkin, i suggest you reassign to jst in case he isn't reading his bugmail ;)
Comment 6•22 years ago
|
||
We still need approval from drivers@mozilla.org for checking this in, get that and I'll be happy to land this...
Comment 7•22 years ago
|
||
Comment on attachment 90206 [details] [diff] [review] patch a=asa (on behalf of drivers) for checkin to the 1.1 trunk.
Attachment #90206 -
Flags: approval+
Assignee | ||
Comment 8•22 years ago
|
||
Attachment #90206 -
Attachment is obsolete: true
Assignee | ||
Updated•22 years ago
|
Attachment #91595 -
Flags: superreview+
Attachment #91595 -
Flags: review+
Attachment #91595 -
Flags: approval+
Comment 9•22 years ago
|
||
The patch still doesn't apply because it appears to be hand edited and patch cries. That notwithstanding, after manually typing in filenames to patch, it appears to break my Linux opt depend build. I don't have the time to investigate it fully ATM...
Comment 10•22 years ago
|
||
Alexey, so ok I've got it to work. THe problem wasn't with the patch but with a few changes I had in my tree. Anyway, your approval had already expired by the time you asked me to check this in. Please get your approval renewed (they expire after 3 days) and I will check it in for you.
Comment 11•22 years ago
|
||
Checked in with a=asa (renewed via email).
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Comment 12•22 years ago
|
||
-> XP apps for qa, per samir.
Component: Networking → XP Apps
Summary: Use XML parser for XHTML about: services → about: Use XML parser for XHTML services
Updated•20 years ago
|
Product: Core → Mozilla Application Suite
You need to log in
before you can comment on or make changes to this bug.
Description
•