1606613 - Stop logging handled errors

Status: RESOLVED FIXED

(Remote Protocol :: Agent, defect, P1)

Description

[Andreas Tolfsen ❲:ato❳]

The remote agent currently logs handled errors to log.error() and there’s no need for that when the errors are being sent back to the client.

This is a left-over from when the remote agent was in a prototype stage and we couldn’t always rely on the Puppeteer client to interact with Firefox correctly.

There might be an argument to be made that we should be differentiating between RemoteAgentError derived errors and other errors, but the situation currently is that we use rely on several base errors (as well as NS result exceptions) to control the code flow.

Comment 1

[Andreas Tolfsen ❲:ato❳]

Attachment: bug 1606613: remote: stop logging handled errors

Errors that arise from use of the protocol, for example by the client
passing in the wrong argument or an operationg being unsupported, are
returned to the client. Since these are not inherent implementation
problems there is no point to double-logging them in the server.

This is a left-over from when the remote agent was in a prototype
stage and we couldn't always rely on the Puppeteer client to
interact with Firefox correctly.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ca9d646f56ba
remote: stop logging handled errors r=remote-protocol-reviewers,whimboo

Comment 3

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/ca9d646f56ba