Remove LegacyPromise in src/core/chunked_stream.js

[Snuffleupagus]

No description provided.

[yurydelendik]

blocking by #4700 to have a good pattern of storing resolve/reject methods.

[yurydelendik]

#4700 merged, see also #4430 for patterns

[yurydelendik]

/botio test

[pdfjsbot]

From: Bot.io (Windows)

---

Received

Command cmd_test from @yurydelendik received. Current queue size: 1

Live output at: http://107.22.172.223:8877/de093507fff6fed/output.txt

[pdfjsbot]

From: Bot.io (Linux)

---

Received

Command cmd_test from @yurydelendik received. Current queue size: 1

Live output at: http://107.21.233.14:8877/d7e1874e7ce96af/output.txt

[Snuffleupagus]

Since we discussed on IRC that there currently is no way to reject the promise if there are errors, should we also in preparation for better error handling add a function that does that?
E.g. something like this:

onError: function ChunkedStreamManager_onError(err) {
  this._loadedStreamCapability.reject(err);
},

[yurydelendik]

Since we discussed on IRC that there currently is no way to reject the promise if there are errors, should we also in preparation for better error handling add a function that does that?

Yes. You are welcome to add the logic now or follow up on this later, when we start enforcing that.

[yurydelendik]

... otherwise looks good, you can land it as is

[pdfjsbot]

From: Bot.io (Windows)

---

Success

Full output at http://107.22.172.223:8877/de093507fff6fed/output.txt

Total script time: 22.35 mins

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

[pdfjsbot]

From: Bot.io (Linux)

---

Success

Full output at http://107.21.233.14:8877/d7e1874e7ce96af/output.txt

Total script time: 25.67 mins

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