(CANCELED) Fix TextAutosizer not to scheduleRelayout()

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 190 matching lines @@
         table = cell->table();
         if (cell->colSpan() > 1 || table->isLogicalWidthAuto())
             return false;
     }
     NOTREACHED();
     return true;
 }
 
 void TableLayoutAlgorithmAuto::computeIntrinsicLogicalWidths(LayoutUnit& minWidth, LayoutUnit& maxWidth)
 {
-    TextAutosizer::TableLayoutScope textAutosizerTableLayoutScope(m_table);
+    // TODO(kojii): We should give a SubtreeLayoutScope to TextAutosizer if in
+    // layout, but it's hard to get it here.

[cbiesinger, 2016/04/15 17:08:08]

Hmm yeah that one is tricky...

[kojii, 2016/04/15 17:58:44]

Yeah...or, what we really need is to avoid scheduleLayout() and
SubtreeLayoutScope is needed just because it makes so as of now. We could add
another value to MarkingBehavior enum as you and leviw were discussing in IRC.
It might look more reasonable?

+    TextAutosizer::TableLayoutScope textAutosizerTableLayoutScope(m_table, nullptr);
 
     fullRecalc();
 
     int spanMaxLogicalWidth = calcEffectiveLogicalWidth();
     minWidth = LayoutUnit();
     maxWidth = LayoutUnit();
     float maxPercent = 0;
     float maxNonPercent = 0;
     bool scaleColumns = shouldScaleColumns(m_table);
 
@@ skipping 472 matching lines @@
             int reduce = available * minMaxDiff / logicalWidthBeyondMin;
             m_layoutStruct[i].computedLogicalWidth += reduce;
             available -= reduce;
             logicalWidthBeyondMin -= minMaxDiff;
             if (available >= 0)
                 break;
         }
     }
 }
 } // namespace blink