Open Bug 1773932 Opened 3 years ago Updated 1 day ago

Remove `nsIScriptableUnicodeConverter` from services/common/utils.js

Categories

(Firefox :: Sync, task, P2)

task

Tracking

()

ASSIGNED

People

(Reporter: mathew.hodson, Assigned: oss)

References

(Blocks 1 open bug)

Details

(Keywords: good-first-bug, Whiteboard: [fxsync-])

Attachments

(1 file)

utils.js uses nsIScriptableUnicodeConverter in some function like encodeUTF8. Callers should be moved to TextEncoder or IOUtils.

Blocks: 1347877
Component: Internationalization → General
Product: Core → Toolkit

The Bugbug bot thinks this bug should belong to the 'Toolkit::OS.File' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Component: General → OS.File
Component: OS.File → Sync
Product: Toolkit → Firefox

Hi Barret,
Please set this as "New", as I see you forgot to do this, or am I wrong? Waiting for the developers opinion about it.
Thank you!

Flags: needinfo?(brennie)
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(brennie)
Priority: -- → P2
Whiteboard: [fxsync-]
Keywords: good-first-bug
Assignee: nobody → oss
Status: NEW → ASSIGNED

Patch submitted: https://phabricator.services.mozilla.com/D275341. Ready for review

(In reply to Knot False from comment #4)

Patch submitted: https://phabricator.services.mozilla.com/D275341. Ready for review

wrong tab; patch is https://phabricator.services.mozilla.com/D275352

(In reply to Knot False from comment #4)

Patch submitted: https://phabricator.services.mozilla.com/D275341. Ready for review

wrong tab; patch is https://phabricator.services.mozilla.com/D275352

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: