Remove unnecessary (and problematic) copy-constructor from LayoutUnit
[WebKit-https.git] / Source / WebCore / ChangeLog
index 0d5c9f0216cd4f31349ce90070e876bc74b54d17..3ba2007403f59df9f169bac6cecc8d68f1dfdb83 100644 (file)
@@ -1,3 +1,22 @@
+2013-02-19  Morten Stenshorne  <mstensho@opera.com>
+
+        Remove unnecessary (and problematic) copy-constructor from LayoutUnit
+        https://bugs.webkit.org/show_bug.cgi?id=110121
+
+        This copy constructor does what a default copy constructor would do, so
+        it's not necessary. Furthermore, this copy constructor is the only
+        reason why LayoutUnit becomes non-POD, and that triggers a gdb bug,
+        making it impossible to do things like "print location()" when inside
+        some RenderBlock, for instance.
+
+        Gdb bug http://sourceware.org/bugzilla/show_bug.cgi?id=15154 reported.
+
+        Reviewed by Benjamin Poulain.
+
+        No tests. Apart from dealing with buggy debuggers, this is just code cleanup.
+
+        * platform/LayoutUnit.h:
+
 2013-02-19  Emil A Eklund  <eae@chromium.org>
 
         LayoutUnit::epsilon shouldn't be necessary to place floats