Prepare for error reporting in workers.

[Ms2ger]

This change is

[highfive]

Heads up! This PR modifies the following files:

• @KiChjang: components/script/dom/serviceworker.rs, components/script/dom/bindings/global.rs, components/script/dom/abstractworker.rs, components/script/dom/worker.rs, components/script/dom/bindings/error.rs

[highfive]

Warning

• These commits modify script code, but no tests are modified. Please consider adding a test!

[nox]

One question but otherwise r=me.

-S-awaiting-review +S-awaiting-answer

---

Reviewed 2 of 2 files at r1, 3 of 3 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

---

components/script/dom/bindings/global.rs, line 295 [r1] (raw file):

        match *self {
            GlobalRef::Window(ref window) => window.report_an_error(error_info, value),
            GlobalRef::Worker(_) => (),

Will we ever report errors on Worker? If not, could we use if let here?

---

Comments from Reviewable

[Ms2ger]

@bors-servo r=nox

---

Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

---

components/script/dom/bindings/global.rs, line 295 [r1] (raw file):

Previously, nox (Anthony Ramine) wrote…

Will we ever report errors on Worker? If not, could we use if let here?

Yeah, working on that right now.

---

Comments from Reviewable

[bors-servo]

📌 Commit 59904e6 has been approved by nox

[bors-servo]

⌛ Testing commit 59904e6 with merge 20e1eec...

[bors-servo]

💔 Test failed - mac-rel-wpt

[highfive]

  ▶ TIMEOUT [expected PASS] /_mozilla/css/iframe/hide_layers2.html

[Ms2ger]

@bors-servo retry #13059

[bors-servo]

⌛ Testing commit 59904e6 with merge 019ac5b...

[bors-servo]

💔 Test failed - mac-rel-wpt

[highfive]

  ▶ TIMEOUT [expected PASS] /_mozilla/css/iframe/hide_layers2.html

[Ms2ger]

@bors-servo retry #13059

AAARGH

[bors-servo]

⌛ Testing commit 59904e6 with merge 3631e31...

[bors-servo]

☀️ Test successful - arm32, arm64, linux-dev, linux-rel, mac-dev-unit, mac-rel-css, mac-rel-wpt, windows-dev