FloatWithRect has invalid and inaccessible default constructor
authorkrollin@apple.com <krollin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 31 Jan 2019 01:35:26 +0000 (01:35 +0000)
committerkrollin@apple.com <krollin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 31 Jan 2019 01:35:26 +0000 (01:35 +0000)
https://bugs.webkit.org/show_bug.cgi?id=194057
<rdar://problem/47685211>

Reviewed by Zalan Bujtas.

FloatWithRect has a default constructor declared as 'default'. This
constructor is invalid because FloatWithRect has a data member that's
a reference that will not get initialized with this constructor. Since
it's invalid, and since it's private and not accessed by anything,
remove this constructor.

No new tests since there should be no observable behavior difference.

* rendering/line/LineLayoutState.h:

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

Source/WebCore/ChangeLog
Source/WebCore/rendering/line/LineLayoutState.h

index afdc6a9..02d161e 100644 (file)
@@ -1,3 +1,21 @@
+2019-01-30  Keith Rollin  <krollin@apple.com>
+
+        FloatWithRect has invalid and inaccessible default constructor
+        https://bugs.webkit.org/show_bug.cgi?id=194057
+        <rdar://problem/47685211>
+
+        Reviewed by Zalan Bujtas.
+
+        FloatWithRect has a default constructor declared as 'default'. This
+        constructor is invalid because FloatWithRect has a data member that's
+        a reference that will not get initialized with this constructor. Since
+        it's invalid, and since it's private and not accessed by anything,
+        remove this constructor.
+
+        No new tests since there should be no observable behavior difference.
+
+        * rendering/line/LineLayoutState.h:
+
 2019-01-30  Daniel Bates  <dabates@apple.com>
 
         [iOS] Keyups for non-modifier keys identified as "Dead" when not focused in a content-editable element
index 792f54a..d0689b4 100644 (file)
@@ -54,8 +54,6 @@ public:
     void adjustRect(const LayoutRect& rect) { m_rect = rect; }
 
 private:
-    FloatWithRect() = default;
-    
     FloatWithRect(RenderBox& renderer)
         : m_renderer(renderer)
         , m_rect(LayoutRect(renderer.x() - renderer.marginLeft(), renderer.y() - renderer.marginTop(), renderer.width() + renderer.horizontalMarginExtent(), renderer.height() + renderer.verticalMarginExtent()))