Convert the PDFObjects class to use a Map internally#20673
Convert the PDFObjects class to use a Map internally#20673Snuffleupagus merged 1 commit intomozilla:masterfrom
PDFObjects class to use a Map internally#20673Conversation
This patch also adds unconditional `Map.prototype.getOrInsertComputed()` usage, which should be fine since it's [supported in the latest browsers](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Map/getOrInsertComputed#browser_compatibility) and it'll be polyfilled (via core-js) in the `legacy` builds.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #20673 +/- ##
==========================================
- Coverage 59.78% 59.78% -0.01%
==========================================
Files 169 169
Lines 90165 90164 -1
==========================================
- Hits 53907 53903 -4
- Misses 36258 36261 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
/botio test |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.241.84.105:8877/3566ba1d5804340/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_test from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.193.163.58:8877/21d7cb9d4e13f9f/output.txt |
From: Bot.io (Linux m4)FailedFull output at http://54.241.84.105:8877/3566ba1d5804340/output.txt Total script time: 41.68 mins
Image differences available at: http://54.241.84.105:8877/3566ba1d5804340/reftest-analyzer.html#web=eq.log |
From: Bot.io (Windows)FailedFull output at http://54.193.163.58:8877/21d7cb9d4e13f9f/output.txt Total script time: 83.32 mins
Image differences available at: http://54.193.163.58:8877/21d7cb9d4e13f9f/reftest-analyzer.html#web=eq.log |
This patch also adds unconditional
Map.prototype.getOrInsertComputed()usage, which should be fine since it's supported in the latest browsers and it'll be polyfilled (via core-js) in thelegacybuilds.