Created attachment 161303 [details] Example file from Excel Attached document is containing a lot of cells (78 columns x 10000 rows) formatted with a cell style. Using the replace functionality to change the style causes huge CPU and memory use. Steps to reproduce: 1. Open attached document 2. Open the Find and Replace dialog 3. Select the Cell Styles checkbox 4. Search for the Excel Built-in Accent2 style and Replace it with the Default style. 5. Push the “Replace All” button, close the Results dialog and push “Replace All” again. Actual results: CPU use goes to 100% for quite a while, LO “does not respond”. Memory use grows quickly to about 3.5GB, then the replace is successfully done. Closing the document also takes a lot of CPU time. Expected results: Smooth user experience: before the bibisected change this replace happened in ~1 second. LibreOffice details: Version: 7.0.0.0.alpha1+ (x64) Build ID: fd238380ae7820f12ac1f7c52d0f7180a93f3ba3 CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win Locale: hu-HU (hu_HU); UI: en-US Calc: CL Also happens in: Verzió: 6.3.0.4 (x86) Build az.: 057fc023c990d676a43019934386b85b21a9ee99 CPU szálak: 4; OS: Windows 6.3; Felületmegjelenítés: alapértelmezett; VCL: win; Területi beállítások: hu-HU (hu_HU); Felület nyelve: hu-HU Calc: CL But not in: Verzió: 6.2.0.3 Build az.: 98c6a8a1c6c7b144ce3cc729e34964b47ce25d62 CPU szálak: 4; OS: Windows 6.3; Felületmegjelenítés: GL; VCL: win; Területi beállítások: hu-HU (hu_HU); UI nyelve: hu-HU Calc: threaded Bibisected using bibisect-win32-6.3 to: URL: https://cgit.freedesktop.org/libreoffice/core/commit/?id=f920e86fbf3968104e1dfc0e9934e80652ed0837 author Caolán McNamara <caolanm@redhat.com> 2019-03-22 15:59:00 +0000 committer Caolán McNamara <caolanm@redhat.com> 2019-03-24 18:19:29 +0100 weld SvxSearchDialog Adding CC to: Caolán McNamara
Created attachment 161304 [details] Screenshot of the problem in Calc
its presumably a big accessibility relation, something that would only happen in past with a11y explicitly enabled vs always now
Caolán McNamara committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/13769dea65137fc3c537de6257d15cb87b51f8ae Related: tdf#133411 SetDocWin is using the previous search success state It will be available in 7.1.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Caolán McNamara committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/0b94169d820482434dc98a37c3c1633ca46fd0dc Resolves: tdf#133411 drop CONTENT_FLOWS_TO from dialog to search results It will be available in 7.1.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Caolán McNamara committed a patch related to this issue. It has been pushed to "libreoffice-7-0": https://git.libreoffice.org/core/commit/dc8d9b6c31acb9234e76dbca8873ae3f963dac4b Related: tdf#133411 SetDocWin is using the previous search success state It will be available in 7.0.0.1. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Caolán McNamara committed a patch related to this issue. It has been pushed to "libreoffice-7-0": https://git.libreoffice.org/core/commit/51b40687d41aab9576dd4e2620c3cd5231732d58 Resolves: tdf#133411 drop CONTENT_FLOWS_TO from dialog to search results It will be available in 7.0.0.1. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
this drops this a11y feature, it never really worked right. fixed in master, 7-0 and backport is in gerrit for 6-4
Super fast replacement in Version: 7.0.0.0.beta1+ Build ID: 2506d8221dd940dfd93d3d7c183430ba6ba3089d CPU threads: 4; OS: Linux 4.19; UI render: default; VCL: gtk3 Locale: en-US (en_US.UTF-8); UI: en-US Calc: threaded @Caolán, thanks for fixing this issue!!
Caolán McNamara committed a patch related to this issue. It has been pushed to "libreoffice-6-4": https://git.libreoffice.org/core/commit/f753baddc057a3bf4881ac78a15cd11fa847ee28 Resolves: tdf#133411 drop CONTENT_FLOWS_TO from dialog to search results It will be available in 6.4.5. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.