Don't deduct min-width from the available width for empty cells

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

 /*
  * Copyright (C) 2002 Lars Knoll (knoll@kde.org)
  *           (C) 2002 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2003, 2006, 2008, 2010 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License.
  *
@@ skipping 527 matching lines @@
             numFixed++;
             totalFixed += m_layoutStruct[i].clampedEffectiveMaxLogicalWidth();
             // fall through
             break;
         case Auto:
             if (m_layoutStruct[i].emptyCellsOnly) {
                 numAutoEmptyCellsOnly++;
             } else {
                 numAuto++;
                 totalAuto += m_layoutStruct[i].clampedEffectiveMaxLogicalWidth();
+            }
+            if (!m_layoutStruct[i].columnHasNoCells)
                 allocAuto += cellLogicalWidth;
-            }
             break;
         default:
             break;
         }
     }
 
     // allocate width to percent cols
     if (available > 0 && havePercent) {
         for (size_t i = 0; i < nEffCols; ++i) {
             Length& logicalWidth = m_layoutStruct[i].effectiveLogicalWidth;
@@ skipping 32 matching lines @@
         }
     }
 
     // Give each auto width column its share of the available width, non-empty columns then empty columns.
     if (available > 0 && numAuto) {
         available += allocAuto;
         distributeWidthToColumns<float, Auto, NonEmptyCells, InitialWidth, StartToEnd>(available, totalAuto);
     }
     if (available > 0 && numAutoEmptyCellsOnly) {
         unsigned total = numAutoEmptyCellsOnly;
+        available += allocAuto;

[mstensho (USE GERRIT), 2015/09/30 11:09:39]

|available| already contains |allocAuto|, if |numAuto| (a few lines up).

Maybe we should never enter both blocks, so that the code should better read:

    if (available > 0 && (numAuto || numAutoEmptyCellsOnly)) {
        available += allocAuto;
        if (numAuto) {
            distributeWidthToColumns<float, Auto, NonEmptyCells, InitialWidth,
StartToEnd>(available, totalAuto);
        } else {
            unsigned total = numAutoEmptyCellsOnly;
            distributeWidthToColumns<unsigned, Auto, EmptyCells, InitialWidth,
StartToEnd>(available, total);
        }
    }

         distributeWidthToColumns<unsigned, Auto, EmptyCells, InitialWidth, StartToEnd>(available, total);
     }
 
     // Any remaining available width expands fixed width, percent width, and non-empty auto width columns, in that order.
     if (available > 0 && numFixed)
         distributeWidthToColumns<float, Fixed, AllCells, ExtraWidth, StartToEnd>(available, totalFixed);
 
     if (available > 0 && m_hasPercent && totalPercent < 100)
         distributeWidthToColumns<float, Percent, AllCells, ExtraWidth, StartToEnd>(available, totalPercent);
 
@@ skipping 80 matching lines @@
             int reduce = available * minMaxDiff / logicalWidthBeyondMin;
             m_layoutStruct[i].computedLogicalWidth += reduce;
             available -= reduce;
             logicalWidthBeyondMin -= minMaxDiff;
             if (available >= 0)
                 break;
         }
     }
 }
 }