Consider not setting gEscapeUTF8 to true

RESOLVED WONTFIX

Status

()

Core
Networking
P3
normal
RESOLVED WONTFIX
9 years ago
6 months ago

People

(Reporter: bz, Unassigned)

Tracking

(Blocks: 2 bugs)

Trunk
x86
Mac OS X
Points:
---
Dependency tree / graph
Bug Flags:
blocking1.9.2 -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [necko-backlog])

Right now, nsStandardURL escapes all non-ASCII bytes in its internal representation, as far as I can tell.  It certainly does so for refs, which causes issues like bug 135309.

Is there a reason we're not storing in IRI form internally, escaping to URI as needed when someone actually needs an ASCII spec?  We have GetAsciiSpec anyway...
That said, is flipping that pref basically all we need to get IRI behavior?
No longer blocks: 135309

Updated

9 years ago
Flags: blocking1.9.2? → blocking1.9.2-
I think we're really not pursuing iri at this point. reopen if I've got that wrong
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
This blocks observable bugs in our HTML support, so if we're not going to change necko here we need to figure out some other approach...  I still think fixing this in necko (via aligning with the URL spec, not IRI) is the right thing.
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
Whiteboard: [necko-backlog]

Comment 6

6 months ago
How exactly are we not aligned with the URL Standard here? The URL Standard requires UTF-8 percent encoding of non-ASCII input (except for the query, where we use the document encoding to do so). I believe Gecko/Necko is aligned with that.
Flags: needinfo?(bzbarsky)
> How exactly are we not aligned with the URL Standard here?

No idea, sorry.  I have been tracking neither the URL standard nor Necko on this issue.  I expect Valentin knows where things stand.
Flags: needinfo?(bzbarsky) → needinfo?(valentin.gosu)
This pref was removed in Bug 1321705.
What Anne said in comment 6 is what we do.
Status: REOPENED → RESOLVED
Last Resolved: 2 years ago6 months ago
Flags: needinfo?(valentin.gosu)
Resolution: --- → FIXED
See Also: → bug 1321705
Resolution: FIXED → WONTFIX
You need to log in before you can comment on or make changes to this bug.