After StringImpl was made immutable, there are still some remaining callers that were making copies of strings for no reason. The only valid reason nowadays is thread safety.
Created attachment 18423 [details] patch
Comment on attachment 18423 [details] patch r=me
Committed revision 29452.