1673884 - Trace logged when reaching MAX_ATTEMPTS in MarionetteFrameProxy throws

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, defect, P3)

Description

[Julian Descottes [:jdescottes]]

In bug 1671347, I added a trace when the proxy reaches the maximum amount of attempts:

                logger.trace(
                  `[${this.browsingContext.id}] Query "${methodName}" reached the limit of retry attempts (${MAX_ATTEMPTS})`
                );

However this.browsingContext.id is not defined in this context and the trace will fail if we ever reach it.

Comment 1

[Julian Descottes [:jdescottes]]

Attachment: Bug 1673884 - [marionette] Fix MAX_ATTEMPTS trace log in MarionetteFrameActorProxy

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f4f3628b57a2
[marionette] Fix MAX_ATTEMPTS trace log in MarionetteFrameActorProxy r=marionette-reviewers,whimboo

Comment 3

[Cristina Coroiu [:ccoroiu]]

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