Chromium Code Reviews| Index: Source/core/css/resolver/FontBuilder.h |
| diff --git a/Source/core/css/resolver/FontBuilder.h b/Source/core/css/resolver/FontBuilder.h |
| index 3a24f72120f95d9f5e412f47c94745a22d8de02f..8367a1df36ede226769c0c86f01264e6fffc8744 100644 |
| --- a/Source/core/css/resolver/FontBuilder.h |
| +++ b/Source/core/css/resolver/FontBuilder.h |
| @@ -42,8 +42,13 @@ class FontBuilder { |
| public: |
| FontBuilder(); |
| + enum FontBuilderOperation { |
| + OnlyBuildFontOnChange, |
| + AlwaysBuildFont, |
| + }; |
| + |
| // FIXME: The name is probably wrong, but matches StyleResolverState callsite for consistency. |
| - void initForStyleResolve(const Document&, RenderStyle*, bool useSVGZoomRules); |
| + void initForStyleResolve(const Document&, RenderStyle*, FontBuilderOperation = OnlyBuildFontOnChange); |
|
pdr.
2014/06/03 16:40:46
Could you add a comment here explaining why one wo
|
| void setInitial(float effectiveZoom); |
| @@ -112,7 +117,6 @@ private: |
| float getComputedSizeFromSpecifiedSize(FontDescription&, float effectiveZoom, float specifiedSize); |
| const Document* m_document; |
| - bool m_useSVGZoomRules; |
| bool m_fontSizehasViewportUnits; |
| // FIXME: This member is here on a short-term lease. The plan is to remove |
| // any notion of RenderStyle from here, allowing FontBuilder to build Font objects |