m_propertyVector should be a private member of MutableStyleProperties

https://bugs.webkit.org/show_bug.cgi?id=271293

Reviewed by Simon Fraser.

Made m_propertyVector private to MutableStyleProperties.

* Source/WebCore/css/MutableStyleProperties.h:
* Source/WebCore/editing/EditingStyle.cpp:
(WebCore::EditingStyle::removeEquivalentProperties):

Canonical link: https://commits.webkit.org/276386@main

Source/WebCore/css/MutableStyleProperties.h

@@ -85,8 +85,6 @@ class MutableStyleProperties final : public StyleProperties {
     int findPropertyIndex(CSSPropertyID) const;
     int findCustomPropertyIndex(StringView propertyName) const;
 
-    Vector<CSSProperty, 4> m_propertyVector;
-
     // Methods for querying and altering CSS custom properties.
     bool setCustomProperty(const String& propertyName, const String& value, bool important, CSSParserContext);
     bool removeCustomProperty(const String& propertyName, String* returnText = nullptr);
@@ -105,6 +103,8 @@ class MutableStyleProperties final : public StyleProperties {
     bool canUpdateInPlace(const CSSProperty&, CSSProperty* toReplace) const;
 
     friend class StyleProperties;
+
+    Vector<CSSProperty, 4> m_propertyVector;
 };
 
 inline void MutableStyleProperties::deref() const

Source/WebCore/editing/EditingStyle.cpp

@@ -1506,13 +1506,11 @@ template<typename T>
 void EditingStyle::removeEquivalentProperties(T& style)
 {
     Vector<CSSPropertyID> propertiesToRemove;
-    for (auto& property : m_mutableStyle->m_propertyVector) {
-        if (style.propertyMatches(property.id(), property.protectedValue().get()))
+    for (auto property : *m_mutableStyle) {
+        if (style.propertyMatches(property.id(), property.value()))
             propertiesToRemove.append(property.id());
     }
-    // FIXME: This should use mass removal.
-    for (auto& property : propertiesToRemove)
-        m_mutableStyle->removeProperty(property);
+    m_mutableStyle->removeProperties(propertiesToRemove.span());
 }
 
 void EditingStyle::forceInline()