Changeset 252255 in webkit

Timestamp:

Nov 8, 2019 1:10:11 PM (5 years ago)

Author:

Antti Koivisto

Message:

StyleResolver state should store user agent appearance style as RenderStyle
​https://bugs.webkit.org/show_bug.cgi?id=204008

Reviewed by Zalan Bujtas.

Generate and pass around user agent style as RenderStyle for apperance computation instead
of awkwardly passing around separate property values.

• css/StyleResolver.cpp:

(WebCore::StyleResolver::styleForElement):
(WebCore::StyleResolver::styleForKeyframe):
(WebCore::StyleResolver::pseudoStyleForElement):
(WebCore::StyleResolver::adjustRenderStyle):
(WebCore::StyleResolver::applyMatchedProperties):
(WebCore::StyleResolver::State::cacheBorderAndBackground): Deleted.

• css/StyleResolver.h:

(WebCore::StyleResolver::element const):
(WebCore::StyleResolver::State::userAgentAppearanceStyle const):
(WebCore::StyleResolver::State::setUserAgentAppearanceStyle):
(WebCore::StyleResolver::element): Deleted.
(WebCore::StyleResolver::State::hasUAAppearance const): Deleted.
(WebCore::StyleResolver::State::borderData const): Deleted.
(WebCore::StyleResolver::State::backgroundData const): Deleted.
(WebCore::StyleResolver::State::backgroundColor const): Deleted.

• rendering/RenderTheme.cpp:

(WebCore::RenderTheme::adjustStyle):
(WebCore::RenderTheme::isControlStyled const):

• rendering/RenderTheme.h:
• rendering/RenderThemeIOS.h:
• rendering/RenderThemeIOS.mm:

(WebCore::RenderThemeIOS::isControlStyled const):

• rendering/RenderThemeMac.h:
• rendering/RenderThemeMac.mm:

(WebCore::RenderThemeMac::isControlStyled const):

• style/StyleBuilder.cpp:

(WebCore::Style::Builder::Builder):

Allow providing style separately from StyleResolver state.

• style/StyleBuilder.h:

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• css/StyleResolver.cpp (modified) (8 diffs)
• css/StyleResolver.h (modified) (4 diffs)
• rendering/RenderTheme.cpp (modified) (4 diffs)
• rendering/RenderTheme.h (modified) (2 diffs)
• rendering/RenderThemeIOS.h (modified) (1 diff)
• rendering/RenderThemeIOS.mm (modified) (2 diffs)
• rendering/RenderThemeMac.h (modified) (1 diff)
• rendering/RenderThemeMac.mm (modified) (2 diffs)
• style/StyleBuilder.cpp (modified) (1 diff)
• style/StyleBuilder.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2019-11-08  Antti Koivisto  <antti@apple.com>
+
+        StyleResolver state should store user agent appearance style as RenderStyle
+        https://bugs.webkit.org/show_bug.cgi?id=204008
+
+        Reviewed by Zalan Bujtas.
+
+        Generate and pass around user agent style as RenderStyle for apperance computation instead
+        of awkwardly passing around separate property values.
+
+        * css/StyleResolver.cpp:
+        (WebCore::StyleResolver::styleForElement):
+        (WebCore::StyleResolver::styleForKeyframe):
+        (WebCore::StyleResolver::pseudoStyleForElement):
+        (WebCore::StyleResolver::adjustRenderStyle):
+        (WebCore::StyleResolver::applyMatchedProperties):
+        (WebCore::StyleResolver::State::cacheBorderAndBackground): Deleted.
+        * css/StyleResolver.h:
+        (WebCore::StyleResolver::element const):
+        (WebCore::StyleResolver::State::userAgentAppearanceStyle const):
+        (WebCore::StyleResolver::State::setUserAgentAppearanceStyle):
+        (WebCore::StyleResolver::element): Deleted.
+        (WebCore::StyleResolver::State::hasUAAppearance const): Deleted.
+        (WebCore::StyleResolver::State::borderData const): Deleted.
+        (WebCore::StyleResolver::State::backgroundData const): Deleted.
+        (WebCore::StyleResolver::State::backgroundColor const): Deleted.
+        * rendering/RenderTheme.cpp:
+        (WebCore::RenderTheme::adjustStyle):
+        (WebCore::RenderTheme::isControlStyled const):
+        * rendering/RenderTheme.h:
+        * rendering/RenderThemeIOS.h:
+        * rendering/RenderThemeIOS.mm:
+        (WebCore::RenderThemeIOS::isControlStyled const):
+        * rendering/RenderThemeMac.h:
+        * rendering/RenderThemeMac.mm:
+        (WebCore::RenderThemeMac::isControlStyled const):
+        * style/StyleBuilder.cpp:
+        (WebCore::Style::Builder::Builder):
+
+        Allow providing style separately from StyleResolver state.
+
+        * style/StyleBuilder.h:
+
 2019-11-08  Daniel Bates  <dabates@apple.com>
 

trunk/Source/WebCore/css/StyleResolver.cpp

@@ -98 +98 @@
 using namespace HTMLNames;
 
-inline void StyleResolver::State::cacheBorderAndBackground()
-{
-    m_hasUAAppearance = m_style->hasAppearance();
-    if (m_hasUAAppearance) {
-        m_borderData = m_style->border();
-        m_backgroundData = m_style->backgroundLayers();
-        m_backgroundColor = m_style->backgroundColor();
-    }
-}
-
 inline void StyleResolver::State::clear()
 {
@@ -113 +103 @@
     m_parentStyle = nullptr;
     m_ownedParentStyle = nullptr;
+    m_userAgentAppearanceStyle = nullptr;
 }
 
@@ -286 +277 @@
 
     // Clean up our style object's display and text decorations (among other fixups).
-    adjustRenderStyle(*state.style(), *state.parentStyle(), parentBoxStyle, &element);
+    adjustRenderStyle(*state.style(), *state.parentStyle(), parentBoxStyle, &element, state.userAgentAppearanceStyle());
 
     if (state.style()->hasViewportUnits())
@@ -314 +305 @@
     builder.applyAllProperties();
 
-    adjustRenderStyle(*state.style(), *state.parentStyle(), nullptr, nullptr);
+    adjustRenderStyle(*state.style(), *state.parentStyle(), nullptr, nullptr, state.userAgentAppearanceStyle());
 
     // Add all the animating properties to the keyframe.
@@ -470 +461 @@
 
     // Clean up our style object's display and text decorations (among other fixups).
-    adjustRenderStyle(*state.style(), *m_state.parentStyle(), parentBoxStyle, nullptr);
+    adjustRenderStyle(*state.style(), *m_state.parentStyle(), parentBoxStyle, nullptr, state.userAgentAppearanceStyle());
 
     if (state.style()->hasViewportUnits())
@@ -780 +771 @@
 #endif
 
-void StyleResolver::adjustRenderStyle(RenderStyle& style, const RenderStyle& parentStyle, const RenderStyle* parentBoxStyle, const Element* element)
+void StyleResolver::adjustRenderStyle(RenderStyle& style, const RenderStyle& parentStyle, const RenderStyle* parentBoxStyle, const Element* element, const RenderStyle* userAgentAppearanceStyle)
 {
     // If the composed tree parent has display:contents, the parent box style will be different from the parent style.
@@ -1000 +991 @@
     // Let the theme also have a crack at adjusting the style.
     if (style.hasAppearance())
-        RenderTheme::singleton().adjustStyle(*this, style, element, m_state.hasUAAppearance(), m_state.borderData(), m_state.backgroundData(), m_state.backgroundColor());
+        RenderTheme::singleton().adjustStyle(*this, style, element, userAgentAppearanceStyle);
 
     // If we have first-letter pseudo style, do not share this style.
@@ -1139 +1130 @@
 
     if (elementTypeHasAppearanceFromUAStyle(element)) {
-        // FIXME: This is such a hack.
         // Find out if there's a -webkit-appearance property in effect from the UA sheet.
         // If so, we cache the border and background styles so that RenderTheme::adjustStyle()
         // can look at them later to figure out if this is a styled form control or not.
-        Style::Builder builder(*this, matchResult, { Style::CascadeLevel::UserAgent }, includedProperties);
+        auto userAgentStyle = RenderStyle::clonePtr(style);
+        Style::Builder builder(*userAgentStyle, *this, matchResult, { Style::CascadeLevel::UserAgent });
         builder.applyAllProperties();
 
-        state.cacheBorderAndBackground();
+        state.setUserAgentAppearanceStyle(WTFMove(userAgentStyle));
     }
 

trunk/Source/WebCore/css/StyleResolver.h

@@ -118 +118 @@
     const RenderStyle* parentStyle() const { return m_state.parentStyle(); }
     const RenderStyle* rootElementStyle() const { return m_state.rootElementStyle(); }
-    const Element* element() { return m_state.element(); }
+    const Element* element() const { return m_state.element(); }
     Document& document() { return m_document; }
     const Document& document() const { return m_document; }
@@ -184 +184 @@
 
 private:
-    void adjustRenderStyle(RenderStyle&, const RenderStyle& parentStyle, const RenderStyle* parentBoxStyle, const Element*);
+    void adjustRenderStyle(RenderStyle&, const RenderStyle& parentStyle, const RenderStyle* parentBoxStyle, const Element*, const RenderStyle* userAgentStyle);
     void adjustRenderStyleForSiteSpecificQuirks(RenderStyle&, const Element&);
 
@@ -216 +216 @@
         const RenderStyle* rootElementStyle() const { return m_rootElementStyle; }
 
-        void cacheBorderAndBackground();
-        bool hasUAAppearance() const { return m_hasUAAppearance; }
-        BorderData borderData() const { return m_borderData; }
-        FillLayer backgroundData() const { return m_backgroundData; }
-        const Color& backgroundColor() const { return m_backgroundColor; }
+        const RenderStyle* userAgentAppearanceStyle() const { return m_userAgentAppearanceStyle.get(); }
+        void setUserAgentAppearanceStyle(std::unique_ptr<RenderStyle> style) { m_userAgentAppearanceStyle = WTFMove(style); }
 
         const SelectorFilter* selectorFilter() const { return m_selectorFilter; }
@@ -233 +230 @@
         const SelectorFilter* m_selectorFilter { nullptr };
 
-        BorderData m_borderData;
-        FillLayer m_backgroundData { FillLayerType::Background };
-        Color m_backgroundColor;
-
-        bool m_hasUAAppearance { false };
+        std::unique_ptr<RenderStyle> m_userAgentAppearanceStyle;
     };
 

trunk/Source/WebCore/rendering/RenderTheme.cpp

@@ -79 +79 @@
 }
 
-void RenderTheme::adjustStyle(StyleResolver& styleResolver, RenderStyle& style, const Element* element, bool UAHasAppearance, const BorderData& border, const FillLayer& background, const Color& backgroundColor)
+void RenderTheme::adjustStyle(StyleResolver& styleResolver, RenderStyle& style, const Element* element, const RenderStyle* userAgentAppearanceStyle)
 {
     // Force inline and table display styles to be inline-block (except for table- which is block)
@@ -91 +91 @@
         style.setDisplay(DisplayType::Block);
 
-    if (UAHasAppearance && isControlStyled(style, border, background, backgroundColor)) {
+    if (userAgentAppearanceStyle && isControlStyled(style, *userAgentAppearanceStyle)) {
         switch (part) {
         case MenulistPart:
@@ -731 +731 @@
 }
 
-bool RenderTheme::isControlStyled(const RenderStyle& style, const BorderData& border, const FillLayer& background, const Color& backgroundColor) const
+bool RenderTheme::isControlStyled(const RenderStyle& style, const RenderStyle& userAgentStyle) const
 {
     switch (style.appearance()) {
@@ -753 +753 @@
     case TextAreaPart:
         // Test the style to see if the UA border and background match.
-        return style.border() != border
-            || style.backgroundLayers() != background
-            || !style.backgroundColorEqualsToColorIgnoringVisited(backgroundColor);
+        return style.border() != userAgentStyle.border()
+            || style.backgroundLayers() != userAgentStyle.backgroundLayers()
+            || !style.backgroundColorEqualsToColorIgnoringVisited(userAgentStyle.backgroundColor());
     default:
         return false;

trunk/Source/WebCore/rendering/RenderTheme.h

@@ -66 +66 @@
     // selection of control size based off the font, the disabling of appearance when certain other properties like
     // "border" are set, or if the appearance is not supported by the theme.
-    void adjustStyle(StyleResolver&, RenderStyle&, const Element*,  bool UAHasAppearance, const BorderData&, const FillLayer&, const Color& backgroundColor);
+    void adjustStyle(StyleResolver&, RenderStyle&, const Element*, const RenderStyle* userAgentAppearanceStyle);
 
     // This method is called to paint the widget as a background of the RenderObject.  A widget's foreground, e.g., the
@@ -117 +117 @@
 
     // Whether or not the control has been styled enough by the author to disable the native appearance.
-    virtual bool isControlStyled(const RenderStyle&, const BorderData&, const FillLayer&, const Color& backgroundColor) const;
+    virtual bool isControlStyled(const RenderStyle&, const RenderStyle& userAgentStyle) const;
 
     // A general method asking if any control tinting is supported at all.

trunk/Source/WebCore/rendering/RenderThemeIOS.h

@@ -67 +67 @@
     int baselinePosition(const RenderBox&) const override;
 
-    bool isControlStyled(const RenderStyle&, const BorderData&, const FillLayer& background, const Color& backgroundColor) const override;
+    bool isControlStyled(const RenderStyle&, const RenderStyle& userAgentStyle) const override;
 
     // Methods for each appearance value.

trunk/Source/WebCore/rendering/RenderThemeIOS.mm

@@ -448 +448 @@
 }
 
-bool RenderThemeIOS::isControlStyled(const RenderStyle& style, const BorderData& border, const FillLayer& background, const Color& backgroundColor) const
+bool RenderThemeIOS::isControlStyled(const RenderStyle& style, const RenderStyle& userAgentStyle) const
 {
     // Buttons and MenulistButtons are styled if they contain a background image.
@@ -455 +455 @@
 
     if (style.appearance() == TextFieldPart || style.appearance() == TextAreaPart)
-        return style.backgroundLayers() != background;
-
-    return RenderTheme::isControlStyled(style, border, background, backgroundColor);
+        return style.backgroundLayers() != userAgentStyle.backgroundLayers();
+
+    return RenderTheme::isControlStyled(style, userAgentStyle);
 }
 

trunk/Source/WebCore/rendering/RenderThemeMac.h

@@ -50 +50 @@
     void adjustRepaintRect(const RenderObject&, FloatRect&) final;
 
-    bool isControlStyled(const RenderStyle&, const BorderData&, const FillLayer&, const Color& backgroundColor) const final;
+    bool isControlStyled(const RenderStyle&, const RenderStyle& userAgentStyle) const final;
 
     bool supportsSelectionForegroundColors(OptionSet<StyleColor::Options>) const final;

trunk/Source/WebCore/rendering/RenderThemeMac.mm

@@ -883 +883 @@
 }
 
-bool RenderThemeMac::isControlStyled(const RenderStyle& style, const BorderData& border,
-                                     const FillLayer& background, const Color& backgroundColor) const
+bool RenderThemeMac::isControlStyled(const RenderStyle& style, const RenderStyle& userAgentStyle) const
 {
     if (style.appearance() == TextFieldPart || style.appearance() == TextAreaPart || style.appearance() == ListboxPart)
-        return style.border() != border;
+        return style.border() != userAgentStyle.border();
 
     // FIXME: This is horrible, but there is not much else that can be done.  Menu lists cannot draw properly when
@@ -896 +895 @@
         return true;
 
-    return RenderTheme::isControlStyled(style, border, background, backgroundColor);
+    return RenderTheme::isControlStyled(style, userAgentStyle);
 }
 

trunk/Source/WebCore/style/StyleBuilder.cpp

@@ -50 +50 @@
 }
 
+Builder::Builder(RenderStyle& style, const StyleResolver& resolver, const MatchResult& matchResult, OptionSet<CascadeLevel> cascadeLevels, PropertyCascade::IncludedProperties includedProperties)
+    : m_cascade(matchResult, cascadeLevels, includedProperties, directionFromStyle(style))
+    , m_state(*this, style, *resolver.parentStyle(), resolver.rootElementStyle(), resolver.document(), resolver.element())
+{
+    ASSERT(resolver.parentStyle());
+}
+
 Builder::Builder(StyleResolver& resolver, const MatchResult& matchResult, OptionSet<CascadeLevel> cascadeLevels, PropertyCascade::IncludedProperties includedProperties)
-    : m_cascade(matchResult, cascadeLevels, includedProperties, directionFromStyle(*resolver.style()))
-    , m_state(*this, *resolver.style(), *resolver.parentStyle(), resolver.rootElementStyle(), resolver.document(), resolver.element())
+    : Builder(*resolver.style(), resolver, matchResult, cascadeLevels, includedProperties)
 {
     ASSERT(resolver.style());
-    ASSERT(resolver.parentStyle());
 }
 

trunk/Source/WebCore/style/StyleBuilder.h

@@ -36 +36 @@
     WTF_MAKE_FAST_ALLOCATED;
 public:
+    Builder(RenderStyle&, const StyleResolver&, const MatchResult&, OptionSet<CascadeLevel>, PropertyCascade::IncludedProperties = PropertyCascade::IncludedProperties::All);
     Builder(StyleResolver&, const MatchResult&, OptionSet<CascadeLevel>, PropertyCascade::IncludedProperties = PropertyCascade::IncludedProperties::All);
     ~Builder();