Chromium Code Reviews Chromium Code Reviews
Issue
20002008:
Don't use Vector<UChar> to build strings (Closed)
DescriptionDon't use Vector<UChar> to build strings
This CL removes a bunch of code sites that were using Vector<UChar> to
construct strings. That used to be a good way to construct strings, but in the
modern world, we have much better alternatives in the form of StringBuffer and
StringBuilder. (You should use StringBuffer if you know exactly how much memory
you need and you should use StringBuilder if you require a variable amount of
memory.)
This CL removes more clients of the StringImpl pointer. There are still a few
left, which require more study.
R=eseidel
BUG=262320
NOTRY=true
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=154843
Patch Set 1 #Patch Set 2 : Fix unit test #
Messages
Total messages: 12 (0 generated)
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||
