Small cleanup of m_fontDirty tracking in CSSStyleSelector
Created attachment 101845 [details] Patch
Comment on attachment 101845 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=101845&action=review Please consider doing this without the name change. > Source/WebCore/ChangeLog:8 > + 1) Use exising setFontDescription helper where appropriate. Typo: existing > Source/WebCore/css/CSSStyleSelector.cpp:1997 > -void CSSStyleSelector::updateFont() > +void CSSStyleSelector::updateFontIfDirty() I don’t think we need the name change. It’s great to move the check into the function, even without changing the function name to make that explicit.
Created attachment 101848 [details] Patch for landing
Comment on attachment 101848 [details] Patch for landing Clearing flags on attachment: 101848 Committed r91657: <http://trac.webkit.org/changeset/91657>
All reviewed patches have been landed. Closing bug.