-
Notifications
You must be signed in to change notification settings - Fork 9.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Firefox] Avoid displaying the indeterminate loadingBar when disableStream=true
is set (PR 10714 follow-up)
#10742
[Firefox] Avoid displaying the indeterminate loadingBar when disableStream=true
is set (PR 10714 follow-up)
#10742
Conversation
…Stream=true` is set (PR 10714 follow-up) While PR 10714 did address the `disableRange=true` case, it also managed to "break" the `disableStream=true` case instead since the indeterminate loadingBar is now displayed when it shouldn't; sorry about that! The solution is simple enough though, don't attempt to fallback to `_fullRequestReader.onProgress` when handling "incomplete" loading information.
bf04f18
to
5e9b606
Compare
/botio unittest |
From: Bot.io (Linux m4)ReceivedCommand cmd_unittest from @timvandermeij received. Current queue size: 0 Live output at: http://54.67.70.0:8877/fd747d4faccacad/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_unittest from @timvandermeij received. Current queue size: 0 Live output at: http://54.215.176.217:8877/0cc8bf2274066de/output.txt |
From: Bot.io (Linux m4)SuccessFull output at http://54.67.70.0:8877/fd747d4faccacad/output.txt Total script time: 2.75 mins
|
From: Bot.io (Windows)SuccessFull output at http://54.215.176.217:8877/0cc8bf2274066de/output.txt Total script time: 5.76 mins
|
/botio-linux preview |
From: Bot.io (Linux m4)ReceivedCommand cmd_preview from @timvandermeij received. Current queue size: 0 Live output at: http://54.67.70.0:8877/271b7dbe85d2260/output.txt |
From: Bot.io (Linux m4)SuccessFull output at http://54.67.70.0:8877/271b7dbe85d2260/output.txt Total script time: 1.88 mins Published |
Thank you! |
While PR #10714 did address the
disableRange=true
case, it also managed to "break" thedisableStream=true
case instead since the indeterminate loadingBar is now displayed when it shouldn't; sorry about that!The solution is simple enough though, don't attempt to fallback to
_fullRequestReader.onProgress
when handling "incomplete" loading information. Edit: Smaller diff with https://github.com/mozilla/pdf.js/pull/10742/files?w=1/cc @timvandermeij