Closed j1elo closed 3 years ago
Im also getting the same issue chrome and firefox both every time im using this extension... OS: Ubuntu 20.04
@j1elo @sakyb7 Thanks for your interest in Find+!
Unfortunately this is a known issue and my hands are tied. I've put a lot of work into trying to optimize the parsing and searching logic as much as possible, but I'm kinda at the limitation of what kind of computation you can do in a chrome extension. The logic for searching text in pages is really complex, it's not very easy to do.
For huge pages, I recommend using other tools (grep, for example). See further: https://github.com/brandon1024/find/issues/310
First of all, thank you very much for this extension! it is awesome and bring a sorely lacking functionality to the browser's find function.
Issue Description
The Firefox script execution limits seem to be fairly low, so this extension will trigger the warning dialog as soon as the page is a bit large. This issue renders the extension unusable for these pages, because the dialog will re-appear just 3 to 5 seconds later.
Steps to Reproduce
Screenshots and Documentation
Firefox version: 85.0 OS: Ubuntu linux
Workaround
Giving more time to the extension will work around the issue, although ideally it shouldn't be needed:
about:config
.dom.max_ext_content_script_run_time
.0
to grant unlimited time (be careful, as this applies to all extension scripts).Comments
Are there maybe any low hanging fruits to improve the performance of the internal regex engine? As right now it seems to be quite slow given that the test page is purely static text and the amount of text is not really that much.