Issue metadata
Sign in to add a comment
|
Implement messageerror event |
||||||||||||||||||||||||||||||||||||||||||||
Issue descriptionChange 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.)
,
May 8 2017
,
May 8 2017
,
May 8 2017
this change seems fairly limited in scope, so marking the security bit to NA
,
Jun 19 2017
,
Jun 22 2017
Hi, Ben, the above CL at comment#5 added messageerror event handler into several objects, but seems currently there are't any code firing messageerror event when needed(a message can't be decoded). If possible would you please help to share current status of this work? Thanks. The background of my question is that I'm working on the CL https://chromium-review.googlesource.com/c/538047/ which adds messageerror event handler into ServiceWorkerContainer and ServiceWorkerGlobalScope. I'm happy to get your comments/concerns for this CL, too.
,
Jun 22 2017
,
Jun 27 2017
Hi Leon, sorry for the delay in response. I haven't started on this work yet. The bug tracking this is here: https://crbug.com/714842. I took a quick look at your CL and it seemed fine to me. Did you have something in particular you were concerned about?
,
Jun 28 2017
Thanks Ben for kindly reply. I was not sure whether it's proper I just expose onmessageerror from ServiceWorkerContainer and ServiceWorkerGlobalScope but without any chance actually invoking execution of it. IIUC, I can just land my CL now, and after issue 714842 done, the onmessageerror handler of ServiceWorkerContainer and ServiceWorkerGlobalScope would also be invoked when needed, does it make sense? Thanks. |
|||||||||||||||||||||||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||||||||||||||||||||||
Comment 1 by binji@chromium.org
, May 3 2017Owner: binji@chromium.org
Status: Assigned (was: Unconfirmed)