Let table row and section's self visual overflow cover whole collapsed borders

third_party/WebKit/Source/core/layout/LayoutTableRowTest.cpp

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 135 matching lines @@
 
   auto* row2 = GetRowByElementId("row2");
   EXPECT_EQ(LayoutRect(0, -10, 440, 220), row2->ContentsVisualOverflowRect());
   EXPECT_EQ(LayoutRect(0, 0, 450, 210), row2->SelfVisualOverflowRect());
 
   auto* row3 = GetRowByElementId("row3");
   EXPECT_EQ(LayoutRect(), row3->ContentsVisualOverflowRect());
   EXPECT_EQ(LayoutRect(0, 0, 450, 100), row3->SelfVisualOverflowRect());
 }
 
+TEST_F(LayoutTableRowTest, VisualOverflowWithCollapsedBorders) {
+  SetBodyInnerHTML(
+      "<style>"
+      "  table { border-collapse: collapse }"
+      "  td { border: 0px solid blue; padding: 0 }"
+      "  div { width: 100px; height: 100px }"
+      "</style>"
+      "<table>"
+      "  <tr id='row'>"
+      "    <td style='border-bottom-width: 10px;"
+      "        outline: 3px solid blue'><div></div></td>"
+      "    <td style='border-width: 3px 15px'><div></div></td>"
+      "  </tr>"
+      "</table>");
+
+  auto* row = GetRowByElementId("row");
+
+  // The row's self visual overflow covers the collapsed borders.
+  LayoutRect expected_self_visual_overflow = row->BorderBoxRect();
+  expected_self_visual_overflow.ExpandEdges(LayoutUnit(1), LayoutUnit(8),
+                                            LayoutUnit(5), LayoutUnit(0));
+  EXPECT_EQ(expected_self_visual_overflow, row->SelfVisualOverflowRect());
+
+  // The row's visual overflow covers self visual overflow and visual overflows
+  // of all cells.
+  LayoutRect expected_visual_overflow = row->BorderBoxRect();
+  expected_visual_overflow.ExpandEdges(LayoutUnit(3), LayoutUnit(8),
+                                       LayoutUnit(5), LayoutUnit(3));
+  EXPECT_EQ(expected_visual_overflow, row->VisualOverflowRect());
+}
+
 TEST_F(LayoutTableRowTest, LayoutOverflow) {
   SetBodyInnerHTML(
       "<table style='border-spacing: 0'>"
       "  <tr id='row'>"
       "    <td style='100px; height: 100px; padding: 0'>"
       "      <div style='position: relative; top: 50px; left: 50px;"
       "          width: 100px; height: 100px'></div>"
       "    </td>"
       "  </tr>"
       "</table>");
 
   EXPECT_EQ(LayoutRect(0, 0, 150, 150),
             GetRowByElementId("row")->LayoutOverflowRect());
 }
 
 }  // anonymous namespace
 
 }  // namespace blink