Created attachment 80388 [details] Stacktrace Writer crashes here when reproducing those steps: 1) start LO and create a new Writer document 2) File -> New and create a new Presentation 3) Move the Presentation Windows, and click File -> Open on Writer _without_ focusing the windows first
Created attachment 80389 [details] bibisect4.0+ result
And I'm using Gnome3 - maybe it's relevant here. Also, the crash is reproducible when using "SAL_USE_VCLPLUGIN=gen" too.
Reverting 8eb1fb9eff4d4eea9c0c1c2e8d3b380e227b5d50 seems to fix the issue.
Created attachment 80396 [details] Minimal patch to fix the issue This patch prevents the crash, but it probably breaks rollover effect added in 8eb1fb9eff4d4eea9c0c1c2e8d3b380e227b5d50
Adding original author to CC
On pc Debian x86-64 with master sources updated today, I could reproduce this crash.
Increase importance since it's a crash + regression
Oops Reproduced on Ubuntu 13.04 x64 commit: 61ba279242ef4ee8f8989c26c6c7291e07949ada
Caolán: the stacktrace retrieved by Pierre-Eric shows vcl part. Moreover he bibisected + provided a patch. I don't know if it has an impact on other env than Linux but with Isamu and me, we're 2 people to reproduce this bug. Would you have some time to take a look?
Created attachment 80470 [details] Patch to keep it alive rollover effect
Current code calls GrabFocus() on background window's MenuBarWindow. And it seems to be illegal and additionally no good ui behavior. https://bugs.freedesktop.org/attachment.cgi?id=80470 will fix it. But this patch disables line 5520: "// #58935# #73659# Focus, if no popup underneath...".
Please replace: s/MenuBarWindow/MenuBarWindow during rollover/
#11 makes it not crash, but then (for me under GNOME) the impress window I moved out of the way pops into the foreground and the file open dialog appears between the writer and impress window, i.e. in the middle of the stack of windows.
Created attachment 80473 [details] smallest version of p-e's patch how about this minimal patch, does this break the rollover stuff ?
Unfortunatelly patch #14 disables rollover effect on windows8 x64.
Created attachment 80524 [details] Not to call Menubar selection logic New patch uploaded. I believe that this patch completely fixes bug 65450. I tested on Windows8, WindowsXP, Ubuntu13.04. Please review it. This patch makes that rollover effect doesn't call MenuBar's selection logic. Currently when we rollovers on MenuBar, MenuBar's selection logic (ChangeHighlightItem et al) is called. But calling these logic on background window seems to illegal and causes fdo#65450.
Isamu Mogi committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=29474304550512f56ef597464bfdb5304bf56bb1 fdo#65450 Not to call MenuBar's selection logic for rollover effect The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Isamu Mogi committed a patch related to this issue. It has been pushed to "libreoffice-4-1": http://cgit.freedesktop.org/libreoffice/core/commit/?id=cbfc61a6ca075eb3fbd9a41dea7f9489c5d56770&h=libreoffice-4-1 fdo#65450 Not to call MenuBar's selection logic for rollover effect It will be available in LibreOffice 4.1. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
that works for me anyway, so lets give it a go