Make InlineBox::m_topLeft and m_logicalWidth protected.
authorzalan@apple.com <zalan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 29 Jun 2017 18:47:25 +0000 (18:47 +0000)
committerzalan@apple.com <zalan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 29 Jun 2017 18:47:25 +0000 (18:47 +0000)
https://bugs.webkit.org/show_bug.cgi?id=173973

Reviewed by Simon Fraser.

I don't think this reasoning from 10 years ago is valid anymore ->
  "FIXME: Would like to make this protected, but methods are accessing these members over in the part."
(comment was conveniently removed in a later commit).

No change in functionality.

* rendering/InlineBox.h:
(WebCore::InlineBox::InlineBox):

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

Source/WebCore/ChangeLog
Source/WebCore/rendering/InlineBox.h

index ad829d6..4707971 100644 (file)
@@ -1,3 +1,19 @@
+2017-06-29  Zalan Bujtas  <zalan@apple.com>
+
+        Make InlineBox::m_topLeft and m_logicalWidth protected.
+        https://bugs.webkit.org/show_bug.cgi?id=173973
+
+        Reviewed by Simon Fraser.
+
+        I don't think this reasoning from 10 years ago is valid anymore -> 
+          "FIXME: Would like to make this protected, but methods are accessing these members over in the part."
+        (comment was conveniently removed in a later commit).
+
+        No change in functionality.
+
+        * rendering/InlineBox.h:
+        (WebCore::InlineBox::InlineBox):
+
 2017-06-29  Wenson Hsieh  <wenson_hsieh@apple.com>
 
         [iOS DnD] Adopt +objectWithItemProviderData: for serializing NSItemProviderReading-conformant objects
index c5b96a1..bb59d12 100644 (file)
@@ -287,9 +287,7 @@ private:
 
     RenderObject& m_renderer;
 
-public:
-    FloatPoint m_topLeft;
-    float m_logicalWidth { 0 };
+    float m_expansion { 0 };
 
 #define ADD_BOOLEAN_BITFIELD(name, Name) \
     private:\
@@ -367,8 +365,6 @@ public:
     };
 #undef ADD_BOOLEAN_BITFIELD
 
-private:
-    float m_expansion { 0 };
     InlineBoxBitfields m_bitfields;
 
 protected:
@@ -382,9 +378,9 @@ protected:
         , m_prev(prev)
         , m_parent(parent)
         , m_renderer(renderer)
+        , m_bitfields(firstLine, constructed, dirty, extracted, isHorizontal)
         , m_topLeft(topLeft)
         , m_logicalWidth(logicalWidth)
-        , m_bitfields(firstLine, constructed, dirty, extracted, isHorizontal)
     {
     }
 
@@ -411,6 +407,10 @@ protected:
     // For InlineFlowBox and InlineTextBox
     bool extracted() const { return m_bitfields.extracted(); }
 
+protected:
+    FloatPoint m_topLeft;
+    float m_logicalWidth { 0 };
+
 #if !ASSERT_WITH_SECURITY_IMPLICATION_DISABLED
 private:
     static constexpr unsigned deletionSentinelNotDeletedValue = 0xF0F0F0F0U;