Closed Bug 1607083 Opened 2 years ago Closed 2 years ago

Remove JSString usage from TextEncoder.encode.

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla74
Tracking Status
firefox74 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

Details

Attachments

(1 file)

No description provided.
Assignee: nobody → emilio
Depends on: 1449861

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.

Blocks: 1607406
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/279c75cdf40b
Remove JSString usage from TextEncoder.encode. r=bzbarsky
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla74
You need to log in before you can comment on or make changes to this bug.