Title: [249081] trunk/Source/WebCore
Revision
249081
Author
za...@apple.com
Date
2019-08-24 06:16:41 -0700 (Sat, 24 Aug 2019)

Log Message

[LFC] Remove redundant Layout::Box::ElementType::TableRowGroup
https://bugs.webkit.org/show_bug.cgi?id=201112
<rdar://problem/54663833>

Reviewed by Antti Koivisto.

Use Layout::Box::ElementType::TableBodyGroup instead.

* layout/layouttree/LayoutBox.cpp:
(WebCore::Layout::Box::isPaddingApplicable const):
* layout/layouttree/LayoutBox.h:
* layout/layouttree/LayoutTreeBuilder.cpp:
(WebCore::Layout::TreeBuilder::createLayoutBox):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (249080 => 249081)


--- trunk/Source/WebCore/ChangeLog	2019-08-24 03:31:55 UTC (rev 249080)
+++ trunk/Source/WebCore/ChangeLog	2019-08-24 13:16:41 UTC (rev 249081)
@@ -1,3 +1,19 @@
+2019-08-24  Zalan Bujtas  <za...@apple.com>
+
+        [LFC] Remove redundant Layout::Box::ElementType::TableRowGroup
+        https://bugs.webkit.org/show_bug.cgi?id=201112
+        <rdar://problem/54663833>
+
+        Reviewed by Antti Koivisto.
+
+        Use Layout::Box::ElementType::TableBodyGroup instead.
+
+        * layout/layouttree/LayoutBox.cpp:
+        (WebCore::Layout::Box::isPaddingApplicable const):
+        * layout/layouttree/LayoutBox.h:
+        * layout/layouttree/LayoutTreeBuilder.cpp:
+        (WebCore::Layout::TreeBuilder::createLayoutBox):
+
 2019-08-23  Simon Fraser  <simon.fra...@apple.com>
 
         RenderLayerModelObject should not call private RenderLayer functions

Modified: trunk/Source/WebCore/layout/layouttree/LayoutBox.cpp (249080 => 249081)


--- trunk/Source/WebCore/layout/layouttree/LayoutBox.cpp	2019-08-24 03:31:55 UTC (rev 249080)
+++ trunk/Source/WebCore/layout/layouttree/LayoutBox.cpp	2019-08-24 13:16:41 UTC (rev 249081)
@@ -366,8 +366,8 @@
         return false;
 
     auto elementType = m_elementAttributes.value().elementType;
-    return elementType != ElementType::TableRowGroup
-        && elementType != ElementType::TableHeaderGroup
+    return elementType != ElementType::TableHeaderGroup
+        && elementType != ElementType::TableBodyGroup
         && elementType != ElementType::TableFooterGroup
         && elementType != ElementType::TableRow
         && elementType != ElementType::TableColumnGroup

Modified: trunk/Source/WebCore/layout/layouttree/LayoutBox.h (249080 => 249081)


--- trunk/Source/WebCore/layout/layouttree/LayoutBox.h	2019-08-24 03:31:55 UTC (rev 249080)
+++ trunk/Source/WebCore/layout/layouttree/LayoutBox.h	2019-08-24 13:16:41 UTC (rev 249081)
@@ -50,7 +50,6 @@
         TableColumn,
         TableRow,
         TableColumnGroup,
-        TableRowGroup,
         TableHeaderGroup,
         TableBodyGroup,
         TableFooterGroup,

Modified: trunk/Source/WebCore/layout/layouttree/LayoutTreeBuilder.cpp (249080 => 249081)


--- trunk/Source/WebCore/layout/layouttree/LayoutTreeBuilder.cpp	2019-08-24 03:31:55 UTC (rev 249080)
+++ trunk/Source/WebCore/layout/layouttree/LayoutTreeBuilder.cpp	2019-08-24 13:16:41 UTC (rev 249081)
@@ -106,8 +106,6 @@
                 return Box::ElementAttributes { Box::ElementType::TableRow };
             if (element->hasTagName(HTMLNames::colgroupTag))
                 return Box::ElementAttributes { Box::ElementType::TableColumnGroup };
-            if (element->hasTagName(HTMLNames::tbodyTag))
-                return Box::ElementAttributes { Box::ElementType::TableRowGroup };
             if (element->hasTagName(HTMLNames::theadTag))
                 return Box::ElementAttributes { Box::ElementType::TableHeaderGroup };
             if (element->hasTagName(HTMLNames::tbodyTag))
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to