Cleanup of MathML headers
authorfred.wang@free.fr <fred.wang@free.fr@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 14 Jul 2016 17:59:17 +0000 (17:59 +0000)
committerfred.wang@free.fr <fred.wang@free.fr@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 14 Jul 2016 17:59:17 +0000 (17:59 +0000)
https://bugs.webkit.org/show_bug.cgi?id=159336

Reviewed by Alex Christensen.

We do some cleanup in MathML headers:
- Use #pragma once
- Use final for class that are not extended.
- Use final instead of override for virtual members that are not overridden by derived classes.
- Try and reduce the visibility of function members to private or protected as appropriate.
- Remove useless #include
- Remove useless class or friendship declaration
- Remove unused functions

No new tests, behavior is unchanged.

* mathml/MathMLElement.h:
* mathml/MathMLInlineContainerElement.h:
* mathml/MathMLMathElement.h:
* mathml/MathMLMencloseElement.h:
* mathml/MathMLOperatorDictionary.h:
* mathml/MathMLPaddedElement.h:
* mathml/MathMLSelectElement.h:
* mathml/MathMLSpaceElement.h:
* mathml/MathMLTextElement.h:
* rendering/mathml/MathOperator.h:
* rendering/mathml/RenderMathMLBlock.h:
* rendering/mathml/RenderMathMLFenced.h:
* rendering/mathml/RenderMathMLFraction.h:
* rendering/mathml/RenderMathMLMath.h:
* rendering/mathml/RenderMathMLMenclose.h:
* rendering/mathml/RenderMathMLOperator.h:
* rendering/mathml/RenderMathMLRoot.h:
* rendering/mathml/RenderMathMLRow.cpp:
(WebCore::RenderMathMLRow::RenderMathMLRow): Deleted. We no longer create anonymous row.
* rendering/mathml/RenderMathMLRow.h:
* rendering/mathml/RenderMathMLScripts.h:
* rendering/mathml/RenderMathMLSpace.h:
* rendering/mathml/RenderMathMLToken.h:
* rendering/mathml/RenderMathMLUnderOver.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@203228 268f45cc-cd09-0410-ab3c-d52691b4dbfc

26 files changed:
Source/WebCore/ChangeLog
Source/WebCore/mathml/MathMLElement.h
Source/WebCore/mathml/MathMLInlineContainerElement.h
Source/WebCore/mathml/MathMLMathElement.h
Source/WebCore/mathml/MathMLMencloseElement.h
Source/WebCore/mathml/MathMLOperatorDictionary.h
Source/WebCore/mathml/MathMLPaddedElement.h
Source/WebCore/mathml/MathMLSelectElement.h
Source/WebCore/mathml/MathMLSpaceElement.h
Source/WebCore/mathml/MathMLTextElement.h
Source/WebCore/rendering/mathml/MathMLStyle.h
Source/WebCore/rendering/mathml/MathOperator.h
Source/WebCore/rendering/mathml/RenderMathMLBlock.h
Source/WebCore/rendering/mathml/RenderMathMLFenced.h
Source/WebCore/rendering/mathml/RenderMathMLFraction.h
Source/WebCore/rendering/mathml/RenderMathMLMath.h
Source/WebCore/rendering/mathml/RenderMathMLMenclose.h
Source/WebCore/rendering/mathml/RenderMathMLOperator.h
Source/WebCore/rendering/mathml/RenderMathMLPadded.h
Source/WebCore/rendering/mathml/RenderMathMLRoot.h
Source/WebCore/rendering/mathml/RenderMathMLRow.cpp
Source/WebCore/rendering/mathml/RenderMathMLRow.h
Source/WebCore/rendering/mathml/RenderMathMLScripts.h
Source/WebCore/rendering/mathml/RenderMathMLSpace.h
Source/WebCore/rendering/mathml/RenderMathMLToken.h
Source/WebCore/rendering/mathml/RenderMathMLUnderOver.h

index 2a631ac..a96b721 100644 (file)
@@ -1,3 +1,46 @@
+2016-07-14  Frederic Wang  <fred.wang@free.fr>
+
+        Cleanup of MathML headers
+        https://bugs.webkit.org/show_bug.cgi?id=159336
+
+        Reviewed by Alex Christensen.
+
+        We do some cleanup in MathML headers:
+        - Use #pragma once
+        - Use final for class that are not extended.
+        - Use final instead of override for virtual members that are not overridden by derived classes.
+        - Try and reduce the visibility of function members to private or protected as appropriate.
+        - Remove useless #include
+        - Remove useless class or friendship declaration
+        - Remove unused functions
+
+        No new tests, behavior is unchanged.
+
+        * mathml/MathMLElement.h:
+        * mathml/MathMLInlineContainerElement.h:
+        * mathml/MathMLMathElement.h:
+        * mathml/MathMLMencloseElement.h:
+        * mathml/MathMLOperatorDictionary.h:
+        * mathml/MathMLPaddedElement.h:
+        * mathml/MathMLSelectElement.h:
+        * mathml/MathMLSpaceElement.h:
+        * mathml/MathMLTextElement.h:
+        * rendering/mathml/MathOperator.h:
+        * rendering/mathml/RenderMathMLBlock.h:
+        * rendering/mathml/RenderMathMLFenced.h:
+        * rendering/mathml/RenderMathMLFraction.h:
+        * rendering/mathml/RenderMathMLMath.h:
+        * rendering/mathml/RenderMathMLMenclose.h:
+        * rendering/mathml/RenderMathMLOperator.h:
+        * rendering/mathml/RenderMathMLRoot.h:
+        * rendering/mathml/RenderMathMLRow.cpp:
+        (WebCore::RenderMathMLRow::RenderMathMLRow): Deleted. We no longer create anonymous row.
+        * rendering/mathml/RenderMathMLRow.h:
+        * rendering/mathml/RenderMathMLScripts.h:
+        * rendering/mathml/RenderMathMLSpace.h:
+        * rendering/mathml/RenderMathMLToken.h:
+        * rendering/mathml/RenderMathMLUnderOver.h:
+
 2016-07-14  Alex Christensen  <achristensen@webkit.org>
 
         Pass SessionID to WebSocketHandle constructor
index 216cec6..4c9600a 100644 (file)
@@ -25,8 +25,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef MathMLElement_h
-#define MathMLElement_h
+#pragma once
 
 #if ENABLE(MATHML)
 
@@ -113,5 +112,3 @@ SPECIALIZE_TYPE_TRAITS_END()
 #include "MathMLElementTypeHelpers.h"
 
 #endif // ENABLE(MATHML)
-
-#endif // MathMLElement_h
index a5895ef..d26ab5f 100644 (file)
@@ -24,8 +24,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef MathMLInlineContainerElement_h
-#define MathMLInlineContainerElement_h
+#pragma once
 
 #if ENABLE(MATHML)
 #include "MathMLElement.h"
@@ -36,8 +35,6 @@ class MathMLInlineContainerElement : public MathMLElement {
 public:
     static Ref<MathMLInlineContainerElement> create(const QualifiedName& tagName, Document&);
 
-    bool isPresentationMathML() const override { return true; }
-
 protected:
     MathMLInlineContainerElement(const QualifiedName& tagName, Document&);
     void childrenChanged(const ChildChange&) override;
@@ -45,9 +42,9 @@ protected:
 
 private:
     RenderPtr<RenderElement> createElementRenderer(RenderStyle&&, const RenderTreePosition&) override;
+    bool isPresentationMathML() const final { return true; }
 };
 
 }
 
 #endif // ENABLE(MATHML)
-#endif // MathMLInlineContainerElement_h
index 92f1802..71e122d 100644 (file)
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef MathMLMathElement_h
-#define MathMLMathElement_h
+#pragma once
 
 #if ENABLE(MATHML)
 #include "MathMLInlineContainerElement.h"
 
 namespace WebCore {
 
-class MathMLMathElement : public MathMLInlineContainerElement {
+class MathMLMathElement final : public MathMLInlineContainerElement {
 public:
     static Ref<MathMLMathElement> create(const QualifiedName& tagName, Document&);
 
-protected:
-    void didAttachRenderers() final;
-
 private:
     MathMLMathElement(const QualifiedName& tagName, Document&);
     void parseAttribute(const QualifiedName&, const AtomicString&) final;
+    void didAttachRenderers() final;
 
-    RenderPtr<RenderElement> createElementRenderer(RenderStyle&&, const RenderTreePosition&) override;
+    RenderPtr<RenderElement> createElementRenderer(RenderStyle&&, const RenderTreePosition&) final;
 };
 
 }
 
 #endif // ENABLE(MATHML)
-#endif // MathMLMathElement_h
index c4fcf33..a0c93b6 100644 (file)
@@ -24,8 +24,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef MathMLMencloseElement_h
-#define MathMLMencloseElement_h
+#pragma once
 
 #if ENABLE(MATHML)
 #include "Element.h"
@@ -36,7 +35,6 @@ namespace WebCore {
 class MathMLMencloseElement final: public MathMLInlineContainerElement {
 public:
     static Ref<MathMLMencloseElement> create(const QualifiedName& tagName, Document&);
-    void parseAttribute(const QualifiedName&, const AtomicString&) final;
 
     enum MencloseNotationFlag {
         LongDiv = 1 << 1,
@@ -59,6 +57,7 @@ public:
 private:
     MathMLMencloseElement(const QualifiedName&, Document&);
     RenderPtr<RenderElement> createElementRenderer(RenderStyle&&, const RenderTreePosition&) final;
+    void parseAttribute(const QualifiedName&, const AtomicString&) final;
     void clearNotations() { m_notationFlags = 0; }
     void addNotation(MencloseNotationFlag notationFlag) { m_notationFlags |= notationFlag; }
     unsigned short m_notationFlags;
@@ -67,4 +66,3 @@ private:
 }
 
 #endif // ENABLE(MATHML)
-#endif // MathMLMencloseElement_h
index e9c4ef6..03387f2 100644 (file)
@@ -23,8 +23,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef MathMLOperatorDictionary_h
-#define MathMLOperatorDictionary_h
+#pragma once
 
 #if ENABLE(MATHML)
 
@@ -57,4 +56,3 @@ bool isVertical(UChar);
 
 }
 #endif // ENABLE(MATHML)
-#endif // MathMLOperatorDictionary_h
index 0dc9696..6002add 100644 (file)
@@ -23,8 +23,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef MathMLPaddedElement_h
-#define MathMLPaddedElement_h
+#pragma once
 
 #if ENABLE(MATHML)
 #include "MathMLInlineContainerElement.h"
@@ -55,4 +54,3 @@ private:
 }
 
 #endif // ENABLE(MATHML)
-#endif // MathMLPaddedElement_h
index f1a44ed..0dfc0d8 100644 (file)
@@ -23,8 +23,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef MathMLSelectElement_h
-#define MathMLSelectElement_h
+#pragma once
 
 #if ENABLE(MATHML)
 #include "MathMLInlineContainerElement.h"
@@ -40,26 +39,25 @@ public:
 
 private:
     MathMLSelectElement(const QualifiedName& tagName, Document&);
-    RenderPtr<RenderElement> createElementRenderer(RenderStyle&&, const RenderTreePosition&) override;
+    RenderPtr<RenderElement> createElementRenderer(RenderStyle&&, const RenderTreePosition&) final;
 
-    bool childShouldCreateRenderer(const Node&) const override;
+    bool childShouldCreateRenderer(const Node&) const final;
 
-    void finishParsingChildren() override;
-    void childrenChanged(const ChildChange&) override;
-    void attributeChanged(const QualifiedName&, const AtomicString& oldValue, const AtomicString& newValue, AttributeModificationReason = ModifiedDirectly) override;
-    void defaultEventHandler(Event*) override;
-    bool willRespondToMouseClickEvents() override;
+    void finishParsingChildren() final;
+    void childrenChanged(const ChildChange&) final;
+    void attributeChanged(const QualifiedName&, const AtomicString& oldValue, const AtomicString& newValue, AttributeModificationReason = ModifiedDirectly) final;
+    void defaultEventHandler(Event*) final;
+    bool willRespondToMouseClickEvents() final;
 
     void toggle();
     int getSelectedActionChildAndIndex(Element*& selectedChild);
     Element* getSelectedActionChild();
     Element* getSelectedSemanticsChild();
 
-    void updateSelectedChild() override;
+    void updateSelectedChild() final;
     RefPtr<Element> m_selectedChild;
 };
 
 }
 
 #endif // ENABLE(MATHML)
-#endif // MathMLSelectElement_h
index 0f8e168..93f67c6 100644 (file)
@@ -23,8 +23,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef MathMLSpaceElement_h
-#define MathMLSpaceElement_h
+#pragma once
 
 #if ENABLE(MATHML)
 #include "MathMLElement.h"
@@ -51,4 +50,3 @@ private:
 }
 
 #endif // ENABLE(MATHML)
-#endif // MathMLSpaceElement_h
index bb2bb07..0ed1f64 100644 (file)
@@ -24,8 +24,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef MathMLTextElement_h
-#define MathMLTextElement_h
+#pragma once
 
 #if ENABLE(MATHML)
 #include "MathMLElement.h"
@@ -35,21 +34,20 @@ namespace WebCore {
 class MathMLTextElement final : public MathMLElement {
 public:
     static Ref<MathMLTextElement> create(const QualifiedName& tagName, Document&);
-    void didAttachRenderers() override;
-
-    bool isPresentationMathML() const override { return true; }
 
 private:
     MathMLTextElement(const QualifiedName& tagName, Document&);
 
-    RenderPtr<RenderElement> createElementRenderer(RenderStyle&&, const RenderTreePosition&) override;
-    bool childShouldCreateRenderer(const Node&) const override;
+    RenderPtr<RenderElement> createElementRenderer(RenderStyle&&, const RenderTreePosition&) final;
+    bool childShouldCreateRenderer(const Node&) const final;
+
+    void childrenChanged(const ChildChange&) final;
+    void parseAttribute(const QualifiedName&, const AtomicString&) final;
+    void didAttachRenderers() final;
 
-    void childrenChanged(const ChildChange&) override;
-    void parseAttribute(const QualifiedName&, const AtomicString&) override;
+    bool isPresentationMathML() const final { return true; }
 };
 
 }
 
 #endif // ENABLE(MATHML)
-#endif // MathMLTextElement_h
index 65f7e96..27eac76 100644 (file)
@@ -23,8 +23,6 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef MathMLStyle_h
-#define MathMLStyle_h
 #pragma once
 
 #if ENABLE(MATHML)
@@ -86,4 +84,3 @@ private:
 }
 
 #endif // ENABLE(MATHML)
-#endif // MathMLStyle_h
index 60c6469..de2d03f 100644 (file)
@@ -23,8 +23,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef MathOperator_h
-#define MathOperator_h
+#pragma once
 
 #if ENABLE(MATHML)
 #include "GlyphPage.h"
@@ -104,4 +103,3 @@ private:
 }
 
 #endif // ENABLE(MATHML)
-#endif // MathOperator_h
index 9b3b3e1..dac776a 100644 (file)
@@ -24,8 +24,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef RenderMathMLBlock_h
-#define RenderMathMLBlock_h
+#pragma once
 
 #if ENABLE(MATHML)
 
@@ -104,13 +103,13 @@ public:
     {
     }
 
-    Optional<int> firstLineBaseline() const override;
 
     MathMLStyle* mathMLStyle() const { return const_cast<MathMLStyle*>(&m_mathMLStyle.get()); }
 
 private:
-    bool isRenderMathMLTable() const override { return true; }
-    const char* renderName() const override { return "RenderMathMLTable"; }
+    bool isRenderMathMLTable() const final { return true; }
+    const char* renderName() const final { return "RenderMathMLTable"; }
+    Optional<int> firstLineBaseline() const final;
 
     Ref<MathMLStyle> m_mathMLStyle;
 };
@@ -125,4 +124,3 @@ SPECIALIZE_TYPE_TRAITS_RENDER_OBJECT(RenderMathMLBlock, isRenderMathMLBlock())
 SPECIALIZE_TYPE_TRAITS_RENDER_OBJECT(RenderMathMLTable, isRenderMathMLTable())
 
 #endif // ENABLE(MATHML)
-#endif // RenderMathMLBlock_h
index bcbbce1..7a816ce 100644 (file)
@@ -23,8 +23,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef RenderMathMLFenced_h
-#define RenderMathMLFenced_h
+#pragma once
 
 #if ENABLE(MATHML)
 
@@ -40,10 +39,10 @@ public:
     MathMLInlineContainerElement& element() { return static_cast<MathMLInlineContainerElement&>(nodeForNonAnonymous()); }
 
 private:
-    bool isRenderMathMLFenced() const override { return true; }
-    const char* renderName() const override { return "RenderMathMLFenced"; }
-    void addChild(RenderObject* child, RenderObject* beforeChild) override;
-    void updateFromElement() override;
+    bool isRenderMathMLFenced() const final { return true; }
+    const char* renderName() const final { return "RenderMathMLFenced"; }
+    void addChild(RenderObject* child, RenderObject* beforeChild) final;
+    void updateFromElement() final;
 
     RenderPtr<RenderMathMLOperator> createMathMLOperator(const String& operatorString, MathMLOperatorDictionary::Form, MathMLOperatorDictionary::Flag);
     void makeFences();
@@ -58,5 +57,3 @@ private:
 }
 
 #endif // ENABLE(MATHML)
-
-#endif // RenderMathMLFenced_h
index ca1761d..b54da87 100644 (file)
@@ -25,8 +25,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef RenderMathMLFraction_h
-#define RenderMathMLFraction_h
+#pragma once
 
 #if ENABLE(MATHML)
 
@@ -42,16 +41,14 @@ public:
     MathMLInlineContainerElement& element() { return static_cast<MathMLInlineContainerElement&>(nodeForNonAnonymous()); }
     float relativeLineThickness() const { return m_defaultLineThickness ? m_lineThickness / m_defaultLineThickness : LayoutUnit(0); }
 
-    void layoutBlock(bool relayoutChildren, LayoutUnit pageLogicalHeight = 0) final;
     void updateFromElement() final;
 
-protected:
-    void computePreferredLogicalWidths() final;
-
 private:
     bool isRenderMathMLFraction() const final { return true; }
     const char* renderName() const final { return "RenderMathMLFraction"; }
 
+    void computePreferredLogicalWidths() final;
+    void layoutBlock(bool relayoutChildren, LayoutUnit pageLogicalHeight = 0) final;
     Optional<int> firstLineBaseline() const final;
     void paint(PaintInfo&, const LayoutPoint&) final;
     RenderMathMLOperator* unembellishedOperator() final;
@@ -94,5 +91,3 @@ private:
 SPECIALIZE_TYPE_TRAITS_RENDER_OBJECT(RenderMathMLFraction, isRenderMathMLFraction())
 
 #endif // ENABLE(MATHML)
-
-#endif // RenderMathMLFraction_h
index 0e2729d..9e2b3fa 100644 (file)
@@ -23,8 +23,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef RenderMathMLMath_h
-#define RenderMathMLMath_h
+#pragma once
 
 #if ENABLE(MATHML)
 
@@ -37,8 +36,8 @@ public:
     RenderMathMLMath(Element&, RenderStyle&&);
 
 private:
-    bool isRenderMathMLMath() const override { return true; }
-    const char* renderName() const override { return "RenderMathMLMath"; }
+    bool isRenderMathMLMath() const final { return true; }
+    const char* renderName() const final { return "RenderMathMLMath"; }
 };
 
 }
@@ -46,4 +45,3 @@ private:
 SPECIALIZE_TYPE_TRAITS_RENDER_OBJECT(RenderMathMLMath, isRenderMathMLMath())
 
 #endif // ENABLE(MATHML)
-#endif // RenderMathMLMath_h
index 24fb9f8..685d9e8 100644 (file)
@@ -24,8 +24,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef RenderMathMLMenclose_h
-#define RenderMathMLMenclose_h
+#pragma once
 
 #if ENABLE(MATHML)
 #include "MathMLMencloseElement.h"
@@ -58,4 +57,3 @@ private:
 SPECIALIZE_TYPE_TRAITS_RENDER_OBJECT(RenderMathMLMenclose, isRenderMathMLMenclose())
 
 #endif // ENABLE(MATHML)
-#endif // RenderMathMLMenclose_h
index 89eadb1..2da12f5 100644 (file)
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef RenderMathMLOperator_h
-#define RenderMathMLOperator_h
+#pragma once
 
 #if ENABLE(MATHML)
 
-#include "Font.h"
-#include "GlyphPage.h"
 #include "MathMLElement.h"
 #include "MathMLOperatorDictionary.h"
 #include "MathOperator.h"
-#include "OpenTypeMathData.h"
 #include "RenderMathMLToken.h"
 
 namespace WebCore {
 
-class RenderMathMLOperator : public RenderMathMLToken {
+class RenderMathMLOperator final : public RenderMathMLToken {
 public:
     RenderMathMLOperator(MathMLElement&, RenderStyle&&);
     RenderMathMLOperator(Document&, RenderStyle&&, const String& operatorString, MathMLOperatorDictionary::Form, unsigned short flags = 0);
@@ -54,25 +50,23 @@ public:
     bool isVertical() const { return m_isVertical; }
     LayoutUnit italicCorrection() const { return m_mathOperator.italicCorrection(); }
 
-    void styleDidChange(StyleDifference, const RenderStyle* oldStyle) final;
-
-    void paint(PaintInfo&, const LayoutPoint&) final;
-
     void updateTokenContent(const String& operatorString);
     void updateTokenContent() final;
     void updateOperatorProperties();
     void updateFromElement() final;
     UChar textContent() const { return m_textContent; }
 
-protected:
+private:
     virtual void setOperatorProperties();
+    void styleDidChange(StyleDifference, const RenderStyle* oldStyle) final;
     void computePreferredLogicalWidths() final;
     void layoutBlock(bool relayoutChildren, LayoutUnit pageLogicalHeight = 0) final;
+    void paint(PaintInfo&, const LayoutPoint&) final;
+
     void setLeadingSpace(LayoutUnit leadingSpace) { m_leadingSpace = leadingSpace; }
     void setTrailingSpace(LayoutUnit trailingSpace) { m_trailingSpace = trailingSpace; }
 
-private:
-    const char* renderName() const override { return isAnonymous() ? "RenderMathMLOperator (anonymous)" : "RenderMathMLOperator"; }
+    const char* renderName() const final { return isAnonymous() ? "RenderMathMLOperator (anonymous)" : "RenderMathMLOperator"; }
     void paintChildren(PaintInfo& forSelf, const LayoutPoint&, PaintInfo& forChild, bool usePrintRect) final;
     bool isRenderMathMLOperator() const final { return true; }
     // The following operators are invisible: U+2061 FUNCTION APPLICATION, U+2062 INVISIBLE TIMES, U+2063 INVISIBLE SEPARATOR, U+2064 INVISIBLE PLUS.
@@ -109,4 +103,3 @@ private:
 SPECIALIZE_TYPE_TRAITS_RENDER_OBJECT(RenderMathMLOperator, isRenderMathMLOperator())
 
 #endif // ENABLE(MATHML)
-#endif // RenderMathMLOperator_h
index 8a0cf95..690e18f 100644 (file)
@@ -23,8 +23,6 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef RenderMathMLPadded_h
-#define RenderMathMLPadded_h
 #pragma once
 
 #if ENABLE(MATHML)
@@ -61,4 +59,3 @@ private:
 SPECIALIZE_TYPE_TRAITS_RENDER_OBJECT(RenderMathMLPadded, isRenderMathMLPadded())
 
 #endif // ENABLE(MATHML)
-#endif // RenderMathMLPadded_h
index 0b1adea..d3aaec1 100644 (file)
@@ -24,8 +24,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef RenderMathMLRoot_h
-#define RenderMathMLRoot_h
+#pragma once
 
 #if ENABLE(MATHML)
 
 
 namespace WebCore {
 
-class RenderMathMLMenclose;
-
 // Render base^(1/index), or sqrt(base) using radical notation.
-class RenderMathMLRoot : public RenderMathMLRow {
-
-friend class RenderMathMLRootWrapper;
+class RenderMathMLRoot final : public RenderMathMLRow {
 
 public:
     RenderMathMLRoot(Element&, RenderStyle&&);
-    void styleDidChange(StyleDifference, const RenderStyle* oldStyle) final;
     void updateFromElement() final;
-
-    void computePreferredLogicalWidths() final;
-    void layoutBlock(bool relayoutChildren, LayoutUnit pageLogicalHeight = 0) final;
     void updateStyle();
 
-protected:
-    void paint(PaintInfo&, const LayoutPoint&) final;
-
 private:
     bool isValid() const;
     RenderBox& getBase() const;
@@ -61,6 +49,12 @@ private:
     bool isRenderMathMLRoot() const final { return true; }
     const char* renderName() const final { return "RenderMathMLRoot"; }
 
+    void styleDidChange(StyleDifference, const RenderStyle* oldStyle) final;
+
+    void computePreferredLogicalWidths() final;
+    void layoutBlock(bool relayoutChildren, LayoutUnit pageLogicalHeight = 0) final;
+    void paint(PaintInfo&, const LayoutPoint&) final;
+
     MathOperator m_radicalOperator;
     LayoutUnit m_verticalGap;
     LayoutUnit m_ruleThickness;
@@ -81,5 +75,3 @@ private:
 SPECIALIZE_TYPE_TRAITS_RENDER_OBJECT(RenderMathMLRoot, isRenderMathMLRoot())
 
 #endif // ENABLE(MATHML)
-
-#endif // RenderMathMLRoot_h
index e3e5d0c..a1b3e0a 100644 (file)
@@ -44,11 +44,6 @@ RenderMathMLRow::RenderMathMLRow(Element& element, RenderStyle&& style)
 {
 }
 
-RenderMathMLRow::RenderMathMLRow(Document& document, RenderStyle&& style)
-    : RenderMathMLBlock(document, WTFMove(style))
-{
-}
-
 void RenderMathMLRow::updateOperatorProperties()
 {
     for (auto* child = firstChildBox(); child; child = child->nextSiblingBox()) {
index 01e6c95..0636f1c 100644 (file)
@@ -24,8 +24,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef RenderMathMLRow_h
-#define RenderMathMLRow_h
+#pragma once
 
 #if ENABLE(MATHML)
 
 
 namespace WebCore {
 
-class RenderMathMLRoot;
-
 class RenderMathMLRow : public RenderMathMLBlock {
 public:
     RenderMathMLRow(Element&, RenderStyle&&);
-    RenderMathMLRow(Document&, RenderStyle&&);
 
     void updateOperatorProperties();
 
+protected:
     void layoutBlock(bool relayoutChildren, LayoutUnit pageLogicalHeight = 0) override;
     Optional<int> firstLineBaseline() const override;
 
-protected:
     void layoutRowItems(LayoutUnit& ascent, LayoutUnit& descent);
     void computeLineVerticalStretch(LayoutUnit& ascent, LayoutUnit& descent);
     void computePreferredLogicalWidths() override;
@@ -60,4 +56,3 @@ private:
 SPECIALIZE_TYPE_TRAITS_RENDER_OBJECT(RenderMathMLRow, isRenderMathMLRow())
 
 #endif // ENABLE(MATHML)
-#endif // RenderMathMLRow_h
index 36c8ab1..8856ec6 100644 (file)
@@ -25,8 +25,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef RenderMathMLScripts_h
-#define RenderMathMLScripts_h
+#pragma once
 
 #if ENABLE(MATHML)
 
@@ -38,18 +37,18 @@ class RenderMathMLScripts : public RenderMathMLBlock {
 public:
     RenderMathMLScripts(Element&, RenderStyle&&);
     RenderMathMLOperator* unembellishedOperator() final;
-    Optional<int> firstLineBaseline() const final;
-    void layoutBlock(bool relayoutChildren, LayoutUnit pageLogicalHeight = 0) override;
 
 protected:
     bool isRenderMathMLScripts() const override { return true; }
     const char* renderName() const override { return "RenderMathMLScripts"; }
     void computePreferredLogicalWidths() override;
+    void layoutBlock(bool relayoutChildren, LayoutUnit pageLogicalHeight = 0) override;
 
     enum ScriptsType { Sub, Super, SubSup, Multiscripts, Under, Over, UnderOver };
     ScriptsType m_scriptType;
 
 private:
+    Optional<int> firstLineBaseline() const final;
     bool getBaseAndScripts(RenderBox*& base, RenderBox*& firstPostScript, RenderBox*& firstPreScript);
     LayoutUnit spaceAfterScript();
     LayoutUnit italicCorrection(RenderBox* base);
@@ -61,5 +60,3 @@ private:
 SPECIALIZE_TYPE_TRAITS_RENDER_OBJECT(RenderMathMLScripts, isRenderMathMLScripts())
 
 #endif // ENABLE(MATHML)
-
-#endif // RenderMathMLScripts_h
index ffd497c..0eb2056 100644 (file)
@@ -23,8 +23,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef RenderMathMLSpace_h
-#define RenderMathMLSpace_h
+#pragma once
 
 #if ENABLE(MATHML)
 
@@ -55,4 +54,3 @@ private:
 SPECIALIZE_TYPE_TRAITS_RENDER_OBJECT(RenderMathMLSpace, isRenderMathMLSpace())
 
 #endif // ENABLE(MATHML)
-#endif // RenderMathMLSpace_h
index 44bb89f..9caf48c 100644 (file)
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef RenderMathMLToken_h
-#define RenderMathMLToken_h
+#pragma once
 
 #if ENABLE(MATHML)
 
 #include "MathMLTextElement.h"
 #include "RenderMathMLBlock.h"
-#include "RenderText.h"
 
 namespace WebCore {
 
@@ -42,10 +40,10 @@ public:
 
     MathMLTextElement& element() { return static_cast<MathMLTextElement&>(nodeForNonAnonymous()); }
 
-    bool isChildAllowed(const RenderObject&, const RenderStyle&) const override { return true; };
     virtual void updateTokenContent();
     void updateFromElement() override;
 
+protected:
     void paint(PaintInfo&, const LayoutPoint&) override;
     void paintChildren(PaintInfo& forSelf, const LayoutPoint&, PaintInfo& forChild, bool usePrintRect) override;
     Optional<int> firstLineBaseline() const override;
@@ -55,6 +53,7 @@ public:
 private:
     bool isRenderMathMLToken() const final { return true; }
     const char* renderName() const override { return "RenderMathMLToken"; }
+    bool isChildAllowed(const RenderObject&, const RenderStyle&) const final { return true; };
     void styleDidChange(StyleDifference, const RenderStyle* oldStyle) override;
     void updateMathVariantGlyph();
     void setMathVariantGlyphDirty()
@@ -71,4 +70,3 @@ private:
 SPECIALIZE_TYPE_TRAITS_RENDER_OBJECT(RenderMathMLToken, isRenderMathMLToken())
 
 #endif // ENABLE(MATHML)
-#endif // RenderMathMLToken_h
index 56b1a4c..86c1927 100644 (file)
@@ -24,8 +24,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef RenderMathMLUnderOver_h
-#define RenderMathMLUnderOver_h
+#pragma once
 
 #if ENABLE(MATHML)
 
@@ -36,13 +35,14 @@ namespace WebCore {
 class RenderMathMLUnderOver final : public RenderMathMLScripts {
 public:
     RenderMathMLUnderOver(Element&, RenderStyle&&);
-    void computePreferredLogicalWidths() final;
-    void layoutBlock(bool relayoutChildren, LayoutUnit pageLogicalHeight = 0) final;
 
 private:
     bool isRenderMathMLScripts() const final { return false; }
-    bool isRenderMathMLUnderOver() const override { return true; }
-    const char* renderName() const override { return "RenderMathMLUnderOver"; }
+    bool isRenderMathMLUnderOver() const final { return true; }
+    const char* renderName() const final { return "RenderMathMLUnderOver"; }
+
+    void computePreferredLogicalWidths() final;
+    void layoutBlock(bool relayoutChildren, LayoutUnit pageLogicalHeight = 0) final;
 
     void computeOperatorsHorizontalStretch();
     bool isValid() const;
@@ -61,5 +61,3 @@ private:
 SPECIALIZE_TYPE_TRAITS_RENDER_OBJECT(RenderMathMLUnderOver, isRenderMathMLUnderOver())
 
 #endif // ENABLE(MATHML)
-
-#endif // RenderMathMLUnderOver_h