vcl/osx/salinst.cxx | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-)
New commits: commit 90ea55957de9cae0c1159c999fb19bd1295f65d3 Author: Patrick Luby <[email protected]> AuthorDate: Sun Mar 1 03:35:22 2026 +0100 Commit: Patrick Luby <[email protected]> CommitDate: Sun Mar 1 15:17:06 2026 +0100 Revert "Related: tdf#155092 don't rely on cached "in live resize" value" This reverts commit 321fa22e87a6258039561fd01d04eb1eee0933e7. Reason for revert: While doing a Time Profile in LibreOffice, I found that -[NSEvent window] is a very expensive call. It appears to do a linear search of the entire list of windows each time. So revert and reimplement in a separate patch. Change-Id: I15c6e0bafbc31924a4b7b091f28724a17b8a6cb2 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/200689 Tested-by: Jenkins Reviewed-by: Patrick Luby <[email protected]> diff --git a/vcl/osx/salinst.cxx b/vcl/osx/salinst.cxx index 3b6965301cab..4c1e554397f6 100644 --- a/vcl/osx/salinst.cxx +++ b/vcl/osx/salinst.cxx @@ -586,14 +586,7 @@ bool AquaSalInstance::DoYield(bool bWait, bool bHandleAllCurrentEvents) // the front of the event queue so no more events will be // dispatched until live resizing ends. Surprisingly, live // resizing appears to end in the next mouse down event. - // In certain cases, ImplGetSVData()->mpWinData->mbIsLiveResize - // remains set to true because the current event needs to be - // dispatched for ImplGetSVData()->mpWinData->mbIsLiveResize - // to be reset to false. This causes dispatching to become an - // infinite wait for the current NSEventTypeLeftMouseUp event - // to get dispatched. So query the native window directly to - // get the real live resizing status. - if ( [pEvent type] == NSEventTypeLeftMouseUp && [pEvent window] && [[pEvent window] inLiveResize] ) + if ( ImplGetSVData()->mpWinData->mbIsLiveResize && [pEvent type] == NSEventTypeLeftMouseUp ) { [NSApp postEvent: pEvent atStart: YES]; return false;
