mozilla / pdf.js

PDF Reader in JavaScript

https://mozilla.github.io/pdf.js/

Apache License 2.0

47.09k stars 9.81k forks source link

Use `waitForEvent` in the `pasteFromClipboard` integration test helper function #18326

Closed timvandermeij closed 5 days ago

timvandermeij commented 6 days ago

This code contains the same bug that the previous commit fixed in waitForEvent, namely that we don't clear the timeout if the event is triggered. By using the now fixed waitForEvent function we not only deduplicate this code but we also fix this issue so that no incorrect timeout logs show up anymore.

Note that this patch gets rid of the incorrect timeout logs, but this doesn't solve the problem of the correct timeout logs yet (but it's preparatory work for follow-up patches that address that problem).

Fixes a part of #17931.

timvandermeij commented 6 days ago

/botio integrationtest