Open Bug 1704670 Opened 3 years ago Updated 7 months ago

Investigate the necessity of flushing frames in nsXULElement::OpenMenu(true)

Categories

(Core :: XUL, task)

Product:
Core
Component:
XUL
Type:
task

Tracking

()

People

(Reporter: mstange, Unassigned)

Details

nsXULElement::OpenMenu flushes frames at the start of the function. It's not obvious to me that there's a need to flush here.

However, if the flush is removed, the following tests time out on Linux and Windows:

  • toolkit/content/tests/chrome/test_menulist_keynav.xhtml
  • toolkit/content/tests/chrome/test_popup_recreate.xhtml

This bug is to investigate why the flushing is necessary and whether it wallpapers over another bug.

Severity: -- → S4
You need to log in before you can comment on or make changes to this bug.