Move the fetchBuiltInCMap method to the PartialEvaluator.prototype

[Snuffleupagus]

Defining this inline in the "constructor" looks slightly weird (I really don't know why I wrote it like that originally), and it can simply be changed to a regular method instead.

[Snuffleupagus]

/botio test

[pdfjsbot]

From: Bot.io (Linux m4)

---

Received

Command cmd_test from @Snuffleupagus received. Current queue size: 0

Live output at: http://54.67.70.0:8877/4abe3abe609665f/output.txt

[pdfjsbot]

From: Bot.io (Windows)

---

Received

Command cmd_test from @Snuffleupagus received. Current queue size: 0

Live output at: http://54.215.176.217:8877/69439f459aa1417/output.txt

[pdfjsbot]

From: Bot.io (Linux m4)

---

Failed

Full output at http://54.67.70.0:8877/4abe3abe609665f/output.txt

Total script time: 25.68 mins

• Font tests: Passed
• Unit tests: Passed
• Regression tests: FAILED

Image differences available at: http://54.67.70.0:8877/4abe3abe609665f/reftest-analyzer.html#web=eq.log

[pdfjsbot]

From: Bot.io (Windows)

---

Failed

Full output at http://54.215.176.217:8877/69439f459aa1417/output.txt

Total script time: 29.30 mins

• Font tests: Passed
• Unit tests: Passed
• Regression tests: FAILED

Image differences available at: http://54.215.176.217:8877/69439f459aa1417/reftest-analyzer.html#web=eq.log

[Snuffleupagus]

The Firefox test-failures, on the Linux bot, look identical to those in the latest test-runs in PR #12012, hence those should be caused by a Firefox update rather than this patch.

[timvandermeij]

Thanks!