RESOLVED FIXED 97379
Update SVGFontData for 8 bit TextRun changes
https://bugs.webkit.org/show_bug.cgi?id=97379
Summary Update SVGFontData for 8 bit TextRun changes
Michael Saboff
Reported 2012-09-21 16:29:58 PDT
Make SVGFontData aware of 8 bit changes to TextRun.
Attachments
Patch (2.05 KB, patch)
2012-09-23 22:08 PDT, Michael Saboff
ggaren: review+
Michael Saboff
Comment 1 2012-09-23 22:08:14 PDT
Sam Weinig
Comment 2 2012-09-25 00:54:18 PDT
Comment on attachment 165310 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=165310&action=review > Source/WebCore/svg/SVGFontData.cpp:148 > remainingTextInRun = createStringWithMirroredCharacters(remainingTextInRun.characters(), remainingTextInRun.length()); Is this line also 8 / 16 bit aware?
Michael Saboff
Comment 3 2012-09-25 09:43:28 PDT
(In reply to comment #2) > (From update of attachment 165310 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=165310&action=review > > > Source/WebCore/svg/SVGFontData.cpp:148 > > remainingTextInRun = createStringWithMirroredCharacters(remainingTextInRun.characters(), remainingTextInRun.length()); > > Is this line also 8 / 16 bit aware? Yes, characters() up converts 8 bit data if necessary.
Geoffrey Garen
Comment 4 2012-09-25 23:32:53 PDT
Comment on attachment 165310 [details] Patch r=me
Michael Saboff
Comment 5 2012-09-26 10:43:42 PDT
Note You need to log in before you can comment on or make changes to this bug.