StringImpl isolatedCopy unnecessarily copies text-segment character data
authoroliver@apple.com <oliver@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 19 Dec 2012 21:59:25 +0000 (21:59 +0000)
committeroliver@apple.com <oliver@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 19 Dec 2012 21:59:25 +0000 (21:59 +0000)
commit29a15b6ddb62f25af734b9b6a99df35f7f5ad245
treef19f8203b7e54e9bb3ae88760eb67515bf4c052a
parent5fdb36574823c4bdf59e6deb5eb5042b01f964c8
StringImpl isolatedCopy unnecessarily copies text-segment character data
https://bugs.webkit.org/show_bug.cgi?id=105376

Reviewed by Anders Carlsson.

This patch adds a new (private) helper to StringImpl that tests whether the StringImpl
is backed by an ASCII literal.  This allows isolatedCopy() to safely use the createFromLiteral
constructor rather than making an unnecessary copy.

* wtf/text/StringImpl.h:
(StringImpl):
(WTF::StringImpl::isASCIILiteral):
(WTF::StringImpl::isolatedCopy):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@138194 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Source/WTF/ChangeLog
Source/WTF/wtf/text/StringImpl.h