Commit cab3ce7b authored by Ian Kilpatrick's avatar Ian Kilpatrick Committed by Commit Bot

[cleanup] Remove IsTablePart check within ComputeMinAndMaxContentContribution

This check could never be reached as the we never reach this function
for a table-part.

Tables have their own (complex) rules for min-max sizing so removing
this to avoid future confusion.

Change-Id: I6345b82754e6e03615d8160e5a2b0738ac44971d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2042920Reviewed-by: default avatarChristian Biesinger <cbiesinger@chromium.org>
Reviewed-by: default avatarMorten Stenshorne <mstensho@chromium.org>
Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#739441}
parent 67f996c5
...@@ -332,7 +332,7 @@ MinMaxSize ComputeMinAndMaxContentContribution( ...@@ -332,7 +332,7 @@ MinMaxSize ComputeMinAndMaxContentContribution(
// sized different. So we just always let the table code handle this. // sized different. So we just always let the table code handle this.
// Replaced elements may size themselves using aspect ratios and block // Replaced elements may size themselves using aspect ratios and block
// sizes, so we pass that on as well. // sizes, so we pass that on as well.
if (box->IsTable() || box->IsTablePart() || box->IsLayoutReplaced()) { if (box->IsTable() || box->IsLayoutReplaced()) {
bool needs_size_reset = false; bool needs_size_reset = false;
if (!box->HasOverrideContainingBlockContentLogicalHeight()) { if (!box->HasOverrideContainingBlockContentLogicalHeight()) {
box->SetOverrideContainingBlockContentLogicalHeight( box->SetOverrideContainingBlockContentLogicalHeight(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment