366689 - Submenu popups seem to have incorrect frame coordinates?

Status: NEW

(Core :: Web Painting, defect)

Description

[Boris Zbarsky [:bzbarsky]]

STEPS TO REPRODUCE:

1)  Apply the patch from https://bugzilla.mozilla.org/attachment.cgi?id=251032
2)  Try to select an item from a submenu (either the main menus or the context
    menu; simple thing is to try to do "View frame source" on the attachment in
    https://bugzilla.mozilla.org/attachment.cgi?id=251032&action=edit).

ACTUAL RESULTS:  No response from submenu for hover or click.

EXPECTED RESULTS:  Submenus work.

Comment 1

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

Oh yeah. Menus just position their views and don't update their frame coordinates. Bah. Something to fix when I remove views.

Anyway, as I said in that other bug, we're not ready for that patch yet anyway because if view geometry is wrong, other things will get messed up.

Comment 2

[Boris Zbarsky [:bzbarsky]]

Yeah.  I'm just making sure we have tracking bugs on the issues that need to be resolved to remove views...

I should note that there _is_ the code at http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/layout/xul/base/src/nsMenuPopupFrame.cpp&rev=1.297&mark=1012-1017#1010 that looks like it's trying to deal with this.  I assume it fails because GetOriginToViewOffset() is a bloody mess?

Comment 3

[Neil Deakin]

Is this still a problem?