Comment F Me (Score 1) 110
So instead of only people who look at online PDFs having to wait for the module to load, now every Firefox user has to waste CPU and memory loading a module they may or may not use. Do people really think this is an improvement? How do I teleport to a saner parallel dimension?
I skimmed the bug report to see if the preload was on browser startup or just earlier in the processes of loading a PDF. I couldn't tell, so maybe my comment above is wrong. But I did see this very ugly bit of code: !hash.toLowerCase().includes("disablerange=true") Converting a string to lower case then searching the entire string to find a setting with a specific value? Repeating that every time you need to check a setting? Eww. Glad that got replaced. Not glad that such code existed in the first place. No wonder FF has so many issues.
The replacement code looks like it has better performance but it's still using magic strings (risks typos) instead of constants: searchParams.get("disablerange") !== "true")