New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 714582 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Last visit 15 days ago
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Launch-OWP
Launch-Accessibility: ----
Launch-Exp-Leadership: ----
Launch-Leadership: ----
Launch-Legal: ----
Launch-M-Approved: ----
Launch-M-Target: ----
Launch-Privacy: ----
Launch-Security: NA
Launch-Test: ----
Launch-UI: ----
Rollout-Type: ----

Blocking:
issue 709179



Sign in to add a comment

Implement messageerror event

Project Member Reported by annevank...@gmail.com, Apr 24 2017

Issue description

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.)
 

Comment 1 by binji@chromium.org, May 3 2017

Labels: -Type-Bug -Pri-3 -OS-Mac M-60 OS-All Pri-2 Type-Launch-OWP
Owner: binji@chromium.org
Status: Assigned (was: Unconfirmed)
Components: Blink>ServiceWorker Blink>Messaging

Comment 3 by evn@google.com, May 8 2017

Labels: Launch-Security-NA

Comment 4 by evn@google.com, May 8 2017

this change seems fairly limited in scope, so marking the security bit to NA

Comment 5 by binji@chromium.org, Jun 19 2017

Status: Fixed (was: Assigned)
This landed here: https://codereview.chromium.org/2860483002

Comment 6 by leon....@intel.com, Jun 22 2017

Cc: falken@chromium.org
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.

Comment 7 by leon....@intel.com, Jun 22 2017

Cc: leon....@intel.com

Comment 8 by binji@chromium.org, 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?

Comment 9 by leon....@intel.com, 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