Trigger computation of font size when crossing foreignObject boundary

Source/core/css/resolver/FontBuilder.h

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 24 matching lines @@
 class FontSelector;
 class RenderStyle;
 
 class FontDescriptionChangeScope;
 
 class FontBuilder {
     WTF_MAKE_NONCOPYABLE(FontBuilder); WTF_MAKE_FAST_ALLOCATED;
 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 would need to use
AlwaysBuildFont? I think I understand in the context of this patch but it isn't
obvious to a naive caller of this.

 
     void setInitial(float effectiveZoom);
 
     void didChangeFontParameters(bool);
 
     void inheritFrom(const FontDescription&);
     void fromSystemFont(CSSValueID, float effectiveZoom);
 
     void setFontFamilyInitial();
     void setFontFamilyInherit(const FontDescription&);
@@ skipping 48 matching lines @@
     // FIXME: "size" arg should be first for consistency with other similar functions.
     void setSize(FontDescription&, float effectiveZoom, float size);
     void checkForOrientationChange(RenderStyle*);
     // This function fixes up the default font size if it detects that the current generic font family has changed. -dwh
     void checkForGenericFamilyChange(RenderStyle*, const RenderStyle* parentStyle);
     void updateComputedSize(RenderStyle*, const RenderStyle* parentStyle);
 
     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
     // directly, rather than as a byproduct of calling RenderStyle::setFontDescription.
     // FontDescriptionChangeScope should be the only consumer of this member.
     // If you're using it, U R DOIN IT WRONG.
     RenderStyle* m_style;
 
     // Fontbuilder is responsbile for creating the Font()
     // object on RenderStyle from various other font-related
     // properties on RenderStyle. Whenever one of those
     // is changed, FontBuilder tracks the need to update
     // style->font() with this bool.
     bool m_fontDirty;
 
     friend class FontBuilderTest;
 };
 
 }
 
 #endif