Compute the correct overflow-x and overflow-y values for table elements

Source/core/css/resolver/StyleAdjuster.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 2004-2005 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2006, 2007 Nicholas Shanks (webkit@nickshanks.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. All rights reserved.
  * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org>
  * Copyright (C) 2007, 2008 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (c) 2011, Code Aurora Forum. All rights reserved.
  * Copyright (C) Research In Motion Limited 2011. All rights reserved.
@@ skipping 407 matching lines @@
         if (toHTMLPlugInElement(element).usePlaceholderContent()) {
             if (style.width().isAuto())
                 style.setWidth(Length(LayoutReplaced::defaultWidth, Fixed));
             if (style.height().isAuto())
                 style.setHeight(Length(LayoutReplaced::defaultHeight, Fixed));
         }
 
         return;
     }
 }
+inline bool overflowValueAllowedOnTable(EOverflow overflow)
+{
+    return overflow == OSCROLL || overflow == OAUTO || overflow == OOVERLAY || overflow == OPAGEDX || overflow == OPAGEDY;

[mstensho (USE GERRIT), 2015/07/27 09:33:30]

None of these values are allowed on tables (and the function is called
overflowValueAllowedOnTable()). These are the very values that are disallowed.

I don't think listing the disallowed values makes any sense. It's not
future-proof.

[rhogan, 2015/07/27 18:18:55]

Haha, just testing! ;)
I agree and have switched it back.

[Julien - ping for review, 2015/07/27 20:08:34]

Looks like I am overruled on that.

/me goes back to play with glue in a corner and let's Morten handle this review

+}
 
 void StyleAdjuster::adjustOverflow(ComputedStyle& style)
 {
     ASSERT(style.overflowX() != OVISIBLE || style.overflowY() != OVISIBLE);
 
-    // If either overflow value is not visible, change to auto.
-    if (style.overflowX() == OVISIBLE && style.overflowY() != OVISIBLE) {
+    if (style.display() == TABLE || style.display() == INLINE_TABLE) {
+        // Tables only support overflow:hidden and overflow:visible and ignore anything else,
+        // see http://dev.w3.org/csswg/css2/visufx.html#overflow. As a table is not a block
+        // container box the rules for resolving conflicting x and y values in CSS Overflow Module
+        // Level 3 do not apply. Arguably overflow-x and overflow-y aren't allowed on tables but
+        // all UAs allow it.
+        if (overflowValueAllowedOnTable(style.overflowX()))
+            style.setOverflowX(OVISIBLE);
+        if (overflowValueAllowedOnTable(style.overflowY()))
+            style.setOverflowY(OVISIBLE);
+        // If we are left with conflicting overflow values for the x and y axes on a table then resolve
+        // both to OVISIBLE. This is interoperable behaviour but is not specced anywhere.
+        if (style.overflowX() == OVISIBLE && style.overflowY() != OVISIBLE)

[mstensho (USE GERRIT), 2015/07/27 09:33:30]

No need for the style.overflowY() != OVISIBLE condition.

[rhogan, 2015/07/27 18:18:55]

Right enough!

+            style.setOverflowY(OVISIBLE);
+        else if (style.overflowY() == OVISIBLE && style.overflowX() != OVISIBLE)

[mstensho (USE GERRIT), 2015/07/27 09:33:30]

ditto.

+            style.setOverflowX(OVISIBLE);
+    } else if (style.overflowX() == OVISIBLE && style.overflowY() != OVISIBLE) {
+        // If either overflow value is not visible, change to auto.
         // FIXME: Once we implement pagination controls, overflow-x should default to hidden
         // if overflow-y is set to -webkit-paged-x or -webkit-page-y. For now, we'll let it
         // default to auto so we can at least scroll through the pages.
         style.setOverflowX(OAUTO);
     } else if (style.overflowY() == OVISIBLE && style.overflowX() != OVISIBLE) {
         style.setOverflowY(OAUTO);
     }
 
-    // Table rows, sections and the table itself will support overflow:hidden and will ignore scroll/auto.
-    // FIXME: Eventually table sections will support auto and scroll.
-    if (style.display() == TABLE || style.display() == INLINE_TABLE
-        || style.display() == TABLE_ROW_GROUP || style.display() == TABLE_ROW) {
-        if (style.overflowX() != OVISIBLE && style.overflowX() != OHIDDEN)
-            style.setOverflowX(OVISIBLE);
-        if (style.overflowY() != OVISIBLE && style.overflowY() != OHIDDEN)
-            style.setOverflowY(OVISIBLE);
-    }
-
     // Menulists should have visible overflow
     if (style.appearance() == MenulistPart) {
         style.setOverflowX(OVISIBLE);
         style.setOverflowY(OVISIBLE);
     }
 }
 
 void StyleAdjuster::adjustStyleForDisplay(ComputedStyle& style, const ComputedStyle& parentStyle)
 {
     if (style.display() == BLOCK && !style.isFloating())
@@ skipping 25 matching lines @@
     if (style.writingMode() != TopToBottomWritingMode && (style.display() == BOX || style.display() == INLINE_BOX))
         style.setWritingMode(TopToBottomWritingMode);
 
     if (parentStyle.isDisplayFlexibleOrGridBox()) {
         style.setFloating(NoFloat);
         style.setDisplay(equivalentBlockDisplay(style.display(), style.isFloating(), !m_useQuirksModeStyles));
     }
 }
 
 }