Modifying shouldScaleColumns for proper scaling of nested table columns

Source/core/rendering/AutoTableLayout.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 180 matching lines @@
         if ((tw.isAuto() || tw.isPercent()) && !table->isOutOfFlowPositioned()) {
             RenderBlock* cb = table->containingBlock();
             while (cb && !cb->isRenderView() && !cb->isTableCell() &&
                 cb->style()->width().isAuto() && !cb->isOutOfFlowPositioned())
                 cb = cb->containingBlock();
 
             table = 0;
             if (cb && cb->isTableCell() &&
                 (cb->style()->width().isAuto() || cb->style()->width().isPercent())) {
                 RenderTableCell* cell = toRenderTableCell(cb);
-                if (cell->colSpan() > 1 || cell->table()->style()->width().isAuto())
+                Length tableWidth = cell->table()->style()->width();
+                if (cell->colSpan() > 1 || tableWidth.isAuto() || (tableWidth.isSpecified() && tableWidth.isZero())
+                    || cell->table()->isFloatingOrOutOfFlowPositioned())

[Julien - ping for review, 2013/05/29 20:56:14]

How about some testing for these cases?

                     scale = false;
                 else
                     table = cell->table();
             }
         }
         else
             table = 0;
     }
     return scale;
 }
@@ skipping 544 matching lines @@
 
     int pos = 0;
     for (size_t i = 0; i < nEffCols; ++i) {
         m_table->setColumnPosition(i, pos);
         pos += m_layoutStruct[i].computedLogicalWidth + m_table->hBorderSpacing();
     }
     m_table->setColumnPosition(m_table->columnPositions().size() - 1, pos);
 }
 
 }