RESOLVED FIXED Bug 150986
Some style changes cause tatechuyoko to be drawn off center 
https://bugs.webkit.org/show_bug.cgi?id=150986
Summary Some style changes cause tatechuyoko to be drawn off center
Myles C. Maxfield
Reported 2015-11-06 17:08:29 PST
Some style changes cause tatechuyoko to be drawn off center
Attachments
Patch (5.21 KB, patch)
2015-11-06 17:13 PST, Myles C. Maxfield 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Myles C. Maxfield
Comment 1 2015-11-06 17:13:30 PST
Created attachment 264983 [details] Patch
Myles C. Maxfield
Comment 2 2015-11-06 17:14:25 PST
<rdar://problem/20748013>
Darin Adler
Comment 3 2015-11-07 16:31:40 PST
Comment on attachment 264983 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=264983&action=review > Source/WebCore/rendering/RenderCombineText.cpp:93 > + // An ancestor element may trigger us to relayout even when we're already combined. We should use better grammar in this comment. I don’t think “trigger us to relayout” is grammatical. > Source/WebCore/rendering/RenderCombineText.cpp:149 > + RenderText::setRenderedText(objectReplacementCharacterString.get().impl()); The .impl() here is really mysterious. Why would we need to include that? The function takes a const String&, not a StringImpl*. I think that saying impl() just introduces another String temporary and thus reference count churn.
Myles C. Maxfield
Comment 4 2015-11-09 11:40:46 PST
Committed r192169: <http://trac.webkit.org/changeset/192169>
Note You need to log in before you can comment on or make changes to this bug.