Title: [96476] trunk/Source/WebCore
Revision
96476
Author
[email protected]
Date
2011-10-02 12:41:08 -0700 (Sun, 02 Oct 2011)

Log Message

Remove an unused member in RenderTable::ColumnStruct
https://bugs.webkit.org/show_bug.cgi?id=69119

Reviewed by Darin Adler.

No test as there should be no change in behavior.

The member was added several years ago and its users were removed or changed. The code would
need a refactoring but this would be tricky to get right. This change just removes the member
to reap the short-term benefits.

* rendering/RenderTable.h:
(WebCore::RenderTable::ColumnStruct::ColumnStruct): Remove anything associated with m_width.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (96475 => 96476)


--- trunk/Source/WebCore/ChangeLog	2011-10-02 17:38:09 UTC (rev 96475)
+++ trunk/Source/WebCore/ChangeLog	2011-10-02 19:41:08 UTC (rev 96476)
@@ -1,3 +1,19 @@
+2011-10-02  Julien Chaffraix  <[email protected]>
+
+        Remove an unused member in RenderTable::ColumnStruct
+        https://bugs.webkit.org/show_bug.cgi?id=69119
+
+        Reviewed by Darin Adler.
+
+        No test as there should be no change in behavior.
+
+        The member was added several years ago and its users were removed or changed. The code would
+        need a refactoring but this would be tricky to get right. This change just removes the member
+        to reap the short-term benefits.
+
+        * rendering/RenderTable.h:
+        (WebCore::RenderTable::ColumnStruct::ColumnStruct): Remove anything associated with m_width.
+
 2011-10-02  Dan Bernstein  <[email protected]>
 
         REGRESSION (r95502): Assertion failure in CSSPrimitiveValue::computeLengthDouble() when media query specifies unit-less length

Modified: trunk/Source/WebCore/rendering/RenderTable.h (96475 => 96476)


--- trunk/Source/WebCore/rendering/RenderTable.h	2011-10-02 17:38:09 UTC (rev 96475)
+++ trunk/Source/WebCore/rendering/RenderTable.h	2011-10-02 19:41:08 UTC (rev 96476)
@@ -126,18 +126,12 @@
     virtual void addChild(RenderObject* child, RenderObject* beforeChild = 0);
 
     struct ColumnStruct {
-        enum {
-            WidthUndefined = 0xffff
-        };
-
         ColumnStruct()
             : span(1)
-            , width(WidthUndefined)
         {
         }
 
         unsigned span;
-        unsigned width; // the calculated position of the column
     };
 
     Vector<ColumnStruct>& columns() { return m_columns; }
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to