[Webkit-unassigned] [Bug 257082] Changing 'DisplayType::InlineGrid' and 'DisplayType::InlineFlex' to output properly in StyleAdjuster.cpp

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jul 4 10:17:27 PDT 2023


https://bugs.webkit.org/show_bug.cgi?id=257082

--- Comment #5 from Ahmad Saleem <ahmad.saleem792 at gmail.com> ---
Creating another 'Static' function in StyleAdjuster.cpp:

static DisplayType equivalentDisplayForLegend(const RenderStyle& style)
{
    switch (auto display = style.display()) {
    case DisplayType::Block:
    case DisplayType::Table:
    case DisplayType::Box:
    case DisplayType::FlowRoot:
    case DisplayType::ListItem:
        return display;
    case DisplayType::InlineTable:
        return DisplayType::Table;
    case DisplayType::InlineBox:
        return DisplayType::InlineBox;
    case DisplayType::Flex:
        return DisplayType::Flex;
    case DisplayType::InlineFlex:
        return DisplayType::InlineFlex;
    case DisplayType::Grid:
        return DisplayType::Grid;
    case DisplayType::InlineGrid:
        return DisplayType::InlineGrid;
    case DisplayType::Inline:
    case DisplayType::InlineBlock:
    case DisplayType::TableRowGroup:
    case DisplayType::TableHeaderGroup:
    case DisplayType::TableFooterGroup:
    case DisplayType::TableRow:
    case DisplayType::TableColumnGroup:
    case DisplayType::TableColumn:
    case DisplayType::TableCell:
    case DisplayType::TableCaption:
        return DisplayType::Block;
    case DisplayType::Contents:
        ASSERT_NOT_REACHED();
        return DisplayType::Contents;
    case DisplayType::None:
        ASSERT_NOT_REACHED();
        return DisplayType::None;
    }
    ASSERT_NOT_REACHED();
    return DisplayType::Block;
}

and then modifying Line 363 to:

style.setEffectiveDisplay(equivalentDisplayForLegend(style));

__________________

@Antti , @Tim , @Alan - any suggestion? It will fix some of these tests.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20230704/54ce7cd2/attachment.htm>


More information about the webkit-unassigned mailing list