Title: [224840] trunk/Source/WebCore
Revision
224840
Author
[email protected]
Date
2017-11-14 13:26:48 -0800 (Tue, 14 Nov 2017)

Log Message

Remove StyleResolver::State::document()
https://bugs.webkit.org/show_bug.cgi?id=179678

Reviewed by Antoine Quint.

We don't need both StyleResolver::document() and StyleResolver::State::document().

* css/StyleResolver.cpp:
(WebCore::StyleResolver::State::updateConversionData):
(WebCore::StyleResolver::resolvedVariableValue):
(WebCore::StyleResolver::colorFromPrimitiveValue const):
(WebCore::StyleResolver::createFilterOperations):
* css/StyleResolver.h:
(WebCore::StyleResolver::State::document const): Deleted.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (224839 => 224840)


--- trunk/Source/WebCore/ChangeLog	2017-11-14 21:22:51 UTC (rev 224839)
+++ trunk/Source/WebCore/ChangeLog	2017-11-14 21:26:48 UTC (rev 224840)
@@ -1,3 +1,20 @@
+2017-11-14  Antti Koivisto  <[email protected]>
+
+        Remove StyleResolver::State::document()
+        https://bugs.webkit.org/show_bug.cgi?id=179678
+
+        Reviewed by Antoine Quint.
+
+        We don't need both StyleResolver::document() and StyleResolver::State::document().
+
+        * css/StyleResolver.cpp:
+        (WebCore::StyleResolver::State::updateConversionData):
+        (WebCore::StyleResolver::resolvedVariableValue):
+        (WebCore::StyleResolver::colorFromPrimitiveValue const):
+        (WebCore::StyleResolver::createFilterOperations):
+        * css/StyleResolver.h:
+        (WebCore::StyleResolver::State::document const): Deleted.
+
 2017-11-14  Dean Jackson  <[email protected]>
 
         Stub implementations of OffscreenCanvas getContext and transferToImageBitmap

Modified: trunk/Source/WebCore/css/StyleResolver.cpp (224839 => 224840)


--- trunk/Source/WebCore/css/StyleResolver.cpp	2017-11-14 21:22:51 UTC (rev 224839)
+++ trunk/Source/WebCore/css/StyleResolver.cpp	2017-11-14 21:26:48 UTC (rev 224840)
@@ -319,7 +319,7 @@
 
 inline void StyleResolver::State::updateConversionData()
 {
-    m_cssToLengthConversionData = CSSToLengthConversionData(m_style.get(), m_rootElementStyle, m_element ? document().renderView() : nullptr);
+    m_cssToLengthConversionData = CSSToLengthConversionData(m_style.get(), m_rootElementStyle, m_element ? m_element->document().renderView() : nullptr);
 }
 
 inline void StyleResolver::State::setStyle(std::unique_ptr<RenderStyle> style)
@@ -1702,7 +1702,7 @@
 
 RefPtr<CSSValue> StyleResolver::resolvedVariableValue(CSSPropertyID propID, const CSSValue& value)
 {
-    CSSParser parser(m_state.document());
+    CSSParser parser(document());
     return parser.parseValueWithVariableReferences(propID, value, m_state.style()->customProperties(), m_state.style()->direction(), m_state.style()->writingMode());
 }
 
@@ -1826,11 +1826,11 @@
     case 0:
         return Color();
     case CSSValueWebkitText:
-        return state.document().textColor();
+        return document().textColor();
     case CSSValueWebkitLink:
-        return (state.element()->isLink() && forVisitedLink) ? state.document().visitedLinkColor() : state.document().linkColor();
+        return (state.element()->isLink() && forVisitedLink) ? document().visitedLinkColor() : document().linkColor();
     case CSSValueWebkitActivelink:
-        return state.document().activeLinkColor();
+        return document().activeLinkColor();
     case CSSValueWebkitFocusRingColor:
         return RenderTheme::focusRingColor();
     case CSSValueCurrentcolor:
@@ -1928,7 +1928,7 @@
                 continue;
 
             String cssUrl = primitiveValue.stringValue();
-            URL url = ""
+            URL url = ""
 
             RefPtr<ReferenceFilterOperation> operation = ReferenceFilterOperation::create(cssUrl, url.fragmentIdentifier());
             operations.operations().append(operation);

Modified: trunk/Source/WebCore/css/StyleResolver.h (224839 => 224840)


--- trunk/Source/WebCore/css/StyleResolver.h	2017-11-14 21:22:51 UTC (rev 224839)
+++ trunk/Source/WebCore/css/StyleResolver.h	2017-11-14 21:26:48 UTC (rev 224840)
@@ -348,7 +348,6 @@
     public:
         void clear();
 
-        Document& document() const { return m_element->document(); }
         const Element* element() const { return m_element; }
 
         void setStyle(std::unique_ptr<RenderStyle>);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to