Bug 171216
| Summary: | Implement messageerror event | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Anne van Kesteren <annevk> |
| Component: | DOM | Assignee: | Chris Dumez <cdumez> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | binji, cdumez, fpizlo, philip, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | Safari Technology Preview | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=254777 | ||
Anne van Kesteren
Change to HTML: https://github.com/whatwg/html/pull/2530.
Change to service workers: https://github.com/w3c/ServiceWorker/pull/1130.
Tests: https://github.com/w3c/web-platform-tests/issues/5567.
(https://github.com/w3c/web-platform-tests/pull/5003 has more tests as part of SharedArrayBuffer postMessage() integration.)
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Ben Smith
I'm implementing this in Chromium (see intent to implement here: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/Z_XzejHJTrs). It would be useful to know if this is on the radar for WebKit.
Philip Jägenstedt
Search for "messageerror" shows that this isn't implemented yet. It is in EventNames.h, but never fired.
Unfortunately, these event handler attributes were added, which might confuse feature detection:
https://trac.webkit.org/browser/webkit/trunk/Source/WebCore/workers/service/ServiceWorkerContainer.idl?rev=267935#L49
https://trac.webkit.org/browser/webkit/trunk/Source/WebCore/workers/service/ServiceWorkerGlobalScope.idl?rev=267935#L48
Radar WebKit Bug Importer
<rdar://problem/96467919>
Chris Dumez
Pull request: https://github.com/WebKit/WebKit/pull/6654
EWS
Committed 256896@main (5e2beb55a112): <https://commits.webkit.org/256896@main>
Reviewed commits have been landed. Closing PR #6654 and removing active labels.