12094 - Click on a pull-down menu brings the browser to a foreground

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect, P3)

Description

[peters]

Clicking on any Editor's pull-down menu title causes Mozilla browser window to
appear on the foreground, with the Editor still being an active (focus is on it)
window.

Comment 1

[buster]

assigning to Simon for the moment, but I think this is really hyatt's problem.
Can you verify that, Mr. Fraser? If so, please change the component and assign
it to hyatt.

Comment 2

[Simon Fraser [no longer active]]

Something strange is going on with window layering and XUL menus. This is almost
certainly a hyatt, saari or danm problem.

Comment 3

[David Hyatt]

This is happening because the editor window is incorrectly being made into a
dependent window.  This is Dan's stuff.  Passing to him.

Comment 4

[Dan M]

Editor windows were being opened with parent windows.  Now that dependent windows are functional,
asking for a window with a parent means to make a dependent window.  So Editor windows opened from
the Tasks menu are no longer dependent, and that conceals whatever problem menus have with being
in a dependent window.  This will come back to haunt us some day.

Comment 5

[sujay]

still not fixed in 8/24 build....

when did this fix go in?

Comment 6

[Dan M]

...yesterday at 5:30.  Still working for me from a fresh pull and build this morning, on Windows and Linux.
If you have a particular case where it isn't working, please add details.

Comment 7

[sujay]

Dan, I grabbed this build:

ftp://sweetlou/products/client/seamonkey/windows/32bit/x86/current/base.zip

And I still see the problem as described in the bug report. Basically
I try an use the pull-down menus in the editor and then it brings the
browser to the foreground.

Comment 8

[Paul MacQuiddy]

The problem is, we are testing M9 builds, and it looks like the fix did not go
into M9. I don't imagine this is an M9 stopper, because they really don't want
to spin builds again. However, this bug makes menu items very painful to use in
Editor. Sujay, please add this to the M9 release note bug 11352.

Sujay, you can verify the fix on an M10 build.

Comment 9

[sujay]

will verify on M10 builds..

Comment 10

[sujay]

verified in 8/26 build.

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a9ac5b186828
[fenix] Fix ignored XML lint issues (https://github.com/mozilla-mobile/fenix/pull/12094)