WebVTTParser copies character buffer more often than necessary
authoreric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 21 Feb 2013 05:36:42 +0000 (05:36 +0000)
committereric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 21 Feb 2013 05:36:42 +0000 (05:36 +0000)
commitc72f331331d446172c9dd459492198fe543a3dcd
tree3c4964da7055108851abc5cf23b086330a1a5086
parentabf93898d06070bc9ba0ad79f07e6ff49e48474e
WebVTTParser copies character buffer more often than necessary
https://bugs.webkit.org/show_bug.cgi?id=103319

Reviewed by Eric Carlson.

Previously this codepath was creating a String just to parse the timestamp
and then if the timestamp was valid, was creating a second string.
I've fixed it to only create one string and use it in both places.
I also fixed this codepath to use 8bit strings when possible, per the FIXME.

* html/track/WebVTTParser.cpp:
(WebCore::WebVTTParser::constructTreeFromToken):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@143563 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Source/WebCore/ChangeLog
Source/WebCore/html/track/WebVTTParser.cpp