Consider not setting gEscapeUTF8 to true

RESOLVED WONTFIX

Status

()

P3
normal
RESOLVED WONTFIX
10 years ago
11 months ago

People

(Reporter: bzbarsky, 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])

(Reporter)

Description

10 years ago
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...
(Reporter)

Comment 1

10 years ago
That said, is flipping that pref basically all we need to get IRI behavior?
(Reporter)

Updated

10 years ago
Blocks: 135309
No longer blocks: 135309
(Reporter)

Updated

10 years ago
Flags: blocking1.9.2?
(Reporter)

Updated

10 years ago
Blocks: 436006

Updated

10 years ago
Flags: blocking1.9.2? → blocking1.9.2-
(Reporter)

Updated

8 years ago
Blocks: 582361
I think we're really not pursuing iri at this point. reopen if I've got that wrong
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → WONTFIX
(Reporter)

Comment 3

3 years ago
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

11 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)
(Reporter)

Comment 7

11 months ago
> 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: 3 years ago11 months ago
Flags: needinfo?(valentin.gosu)
Resolution: --- → FIXED
See Also: → bug 1321705

Updated

11 months ago
Resolution: FIXED → WONTFIX
You need to log in before you can comment on or make changes to this bug.