Some style changes cause tatechuyoko to be drawn off center
Created attachment 264983 [details]
Comment on attachment 264983 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=264983&action=review
> + // 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.
> + 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.
Committed r192169: <http://trac.webkit.org/changeset/192169>