478594 - ireflow issue with <select>

Status: NEW

(Core :: Layout, defect)

Description

[Jesse Ruderman]

Attachment: testcase (based on layout/generic/crashtests/391053-1.xhtml)

Steps to reproduce:
1. Apply the ireflow patch (bug 67752 comment 56).
2. Set the following environment variables:
  export GECKO_REFLOW_INTERRUPT_MODE=counter
  export GECKO_REFLOW_INTERRUPT_CHECKS_TO_SKIP=1
  export GECKO_REFLOW_INTERRUPT_FREQUENCY=1
3. Load the testcase
4. Click the <select>

Result: clicking changes the size.

Based on CPU usage, I'm guessing ireflow never finishes.  Is that expected given the testcase and settings?

refdyn complained about layout/generic/crashtests/391053-1.xhtml, but this might not be the same issue that made refdyn unhappy.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Yeah.  Given the testcase and settings, I'd expect us to never complete a reflow here.  This shouldn't affect refdyn, since trying to snapshot will flush out a non-interruptible reflow.

I could disable interruption inside a listbox, but I think this shouldn't be an issue when using the user-event approach....

Comment 2

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Yeah, I think we should at least give it a go with user events and see if starvation is a problem in practice.