371446 - figure out API and behavior for 'overflow' on non-blocks

Status: NEW

(Core :: CSS Parsing and Computation, defect)

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

In bug 9458 comment 87, bzbarsky wrote:

> >diff -r 5df1530b6fe3 layout/base/nsCSSFrameConstructor.cpp
> >@@ -6378,6 +6387,7 @@ nsCSSFrameConstructor::ConstructFrameByD
> >   // XXX Ignore tables for the time being
> >   if (aDisplay->IsBlockLevel() &&
> >       aDisplay->mDisplay != NS_STYLE_DISPLAY_TABLE &&
> >+      aDisplay->mDisplay != NS_STYLE_DISPLAY_INLINE_TABLE &&
> >       aDisplay->IsScrollableOverflow() &&
> >       !propagatedScrollToViewport) {
> > 
> 
> Maybe that condition could be a method on nsStyleDisplay or something? E.g.
> part of IsScrollableOverflow()?
> 
> Followup bug if you agree.

I tend to think that shouldn't be part of IsScrollableOverflow, but we do need to do a bit of work to figure out how 'overflow' should behave on elements that aren't block-inside, and what the nsStyleDisplay API really should be to allow that.