Closed
Bug 286074
Opened 20 years ago
Closed 9 years ago
Add charset to XML declaration
Categories
(Core Graveyard :: Profile: Roaming, defect)
Core Graveyard
Profile: Roaming
Tracking
(Not tracked)
RESOLVED
INCOMPLETE
People
(Reporter: annevk, Unassigned)
References
Details
Attachments
(1 file)
(deleted),
patch
|
BenB
:
review-
|
Details | Diff | Splinter Review |
Per bug 247024 comment 12 filing a separate bug so the issue can be tracked
separately.
See RFC 3023 section 8.4 for the reason why we should avoid text/xml.
I'm going to attach a patch that replaces text/xml with application/xml. Can
someone tell me whether that is a good thing or not and from who I should get
review? Thanks.
Reporter | ||
Comment 1•20 years ago
|
||
Reporter | ||
Updated•20 years ago
|
Attachment #177368 -
Flags: review?(ben.bucksch)
Comment 2•18 years ago
|
||
> RFC 3023 section 8.4
? http://www.apps.ietf.org/rfc/rfc3023.html#sec-8.4
Reporter | ||
Comment 3•18 years ago
|
||
That link doesn't work, but I should've said 8.5. Sorry about that. Anyway, I'm no longer driving this patch.
Comment 4•15 years ago
|
||
Comment on attachment 177368 [details] [diff] [review]
patch #1
I don't think RFC 3023 section 8.5 is a good reason to avoid text/xml.
It just says that if the documents *omits* the charset in the XML declaration, then it needs to be treated as ASCII.
That is a good reason to add a charset to the declaration, not to avoid text/xml.
WONTFIX.
Attachment #177368 -
Flags: review?(ben.bucksch) → review-
Comment 5•15 years ago
|
||
filesList.js, function createListingFile()
Summary: avoid usage of text/xml → Add charset to XML declaration
Blocks: 1243899
Comment 6•9 years ago
|
||
This bug is filed in a bugzilla component related to pre-Firefox code which no longer exists. I believe it is no longer relevant and I am therefore closing it INCOMPLETE.
If you believe that this bug is still valid and needs to be fixed, please reopen it and move it to the Toolkit:Startup and Profile System product/component.
Assignee | ||
Updated•8 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•