Closed
Bug 1887719
Opened 11 months ago
Closed 10 months ago
More consistently use UTF8String/nsCString for URLs
Categories
(Core :: DOM: Networking, defect, P2)
Core
DOM: Networking
Tracking
()
RESOLVED
FIXED
126 Branch
Tracking | Status | |
---|---|---|
firefox126 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
(Blocks 1 open bug)
Details
(Whiteboard: [necko-triaged])
Attachments
(1 file)
No description provided.
Assignee | ||
Comment 1•11 months ago
|
||
Sorry for the massive patch but I found it hard to split without
introducing a bunch of copies around...
This mostly makes necko and DOM agree on which strings to use, which
should result on less copies and conversions.
Depends on D205560
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2bb244f58cca
More consistently use UTF8String/nsCString for URLs. r=necko-reviewers,webidl,anti-tracking-reviewers,places-reviewers,jari,kershaw,janv,smaug,hsivonen
Pushed by emilio@crisal.io:
https://hg.mozilla.org/integration/autoland/rev/f12fc395b95a
Fix fuzzing build.
Comment 4•10 months ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/2bb244f58cca
https://hg.mozilla.org/mozilla-central/rev/f12fc395b95a
Status: NEW → RESOLVED
Closed: 10 months ago
status-firefox126:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 126 Branch
Comment 5•10 months ago
|
||
Looks like this may have improved some Debugger tests: https://treeherder.mozilla.org/perfherder/alerts?id=42171&hideDwnToInv=0
You need to log in
before you can comment on or make changes to this bug.
Description
•