Replace String::replace() functions with makeStringByReplacingAll() free functions. "All" is added to the name since every instance is replaced, not just the first one. Making it a free function that returns a new String is a step towards making String immutable. In the future, we may want to add a makeStringByReplacingAll() that takes in a StringView instead of a `const String`. Some call sites would benefit from that.
Pull request: https://github.com/WebKit/WebKit/pull/368
Committed r293326 (249949@main): <https://commits.webkit.org/249949@main> Reviewed commits have been landed. Closing PR #368 and removing active labels.
<rdar://problem/92270440>