Closed
Bug 1607083
Opened 5 years ago
Closed 5 years ago
Remove JSString usage from TextEncoder.encode.
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla74
| Tracking | Status | |
|---|---|---|
| firefox74 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
Details
Attachments
(1 file)
No description provided.
|
Assignee | |
Comment 1•5 years ago
|
||
UTF8String has small-size optimization already, and does what we were doing
taking less memory when the string is latin1, so it should be a clear
improvement unless I'm missing something.
I didn't remove it from encodeInto because it'd do more work in the case where
the UInt8Array doesn't have enough capacity... Not sure if that case is worth
optimizing or not, honestly.
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/279c75cdf40b
Remove JSString usage from TextEncoder.encode. r=bzbarsky
|
||
Comment 3•5 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox74:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla74
You need to log in
before you can comment on or make changes to this bug.
Description
•