Build animated font properties through FontBuilder.
Font mutations during style resolution should happen through the FontBuilder
of the StyleResolverState. Otherwise, the elaborate zooming and min-max
logic in FontSize via FontBuilder is not applied.
It is also good etiquette by AnimatedStyleBuilder to *not* secretly update
the Font without notice, because it makes the 'fontDirty' flag wrong, in
some cases. (Although the worst-case scenario is just an extra Font update).
FontBuilder::setSize used to accept a CSSValue*, and I would assume that's
why RenderStyle::setFontSize was created in the first place. It's easy to
cleanly set a new size on the builder now, though.
R=alancutter@chromium.org
BUG=227545
Review URL: https://codereview.chromium.org/721773002
git-svn-id: svn://svn.chromium.org/blink/trunk@185449 bbb929c8-8fbe-4397-9dbb-9b2b20218538
5 files changed