New issue
Advanced search Search tips

Issue 822475 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 3
Type: Bug



Sign in to add a comment

Use CloseWithReason instead of RaiseError when a callback is discarded

Project Member Reported by reillyg@chromium.org, Mar 15 2018

Issue description

In helping a developer diagnose a bug in which they were not invoking a Mojo callback before it was destroyed I noticed that we could, instead of using InterfaceEndpointClient::RaiseError to close the pipe, call CloseWithReason and provide an error code and message indicating this failure mode. While C++ code does not normally look at these error reasons the JavaScript bindings will reject the Promise with an error containing this reason and this would be useful for debugging.

rockot@, opinions?
 
Cc: -roc...@chromium.org rockot@google.com

Sign in to add a comment