1683826 - Replace SanitizeOriginString by pure functions

Status: RESOLVED FIXED

(Core :: Storage: Quota Manager, task)

Description

[Simon Giesecke [:sg] [he/him]]

SanitizeOriginString is a function with an in-out parameter. In all uses, this is not an optimization, since the source string is copied before. It can therefore better be replaced by pure functions with an input parameter and a return value, which simplifies its uses.

Comment 1

[Simon Giesecke [:sg] [he/him]]

Attachment: Bug 1683826 - Replace SanitizeOriginString by pure functions. r=#dom-workers-and-storage

Depends on D99790

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4813f52b37db
Replace SanitizeOriginString by pure functions. r=dom-workers-and-storage-reviewers,janv

Comment 3

[Narcis Beleuzu [:NarcisB]]

Backed out for causing xpcshell failures on test_archive.js

Backout link: https://hg.mozilla.org/integration/autoland/rev/dea356fe0c57a8498696f48a5bcab247cc1ab5f2
Log link: https://treeherder.mozilla.org/logviewer?job_id=326715119&repo=autoland&lineNumber=6770

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e1efcb9dbb33
Replace SanitizeOriginString by pure functions. r=dom-workers-and-storage-reviewers,janv

Comment 5

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/e1efcb9dbb33