Closed Bug 1591341 Opened 2 months ago Closed 2 months ago

Always close the client connection in add_task() for browser chrome tests

Categories

(Remote Protocol :: Agent, defect, P1)

defect

Tracking

(firefox72 fixed)

RESOLVED FIXED
Tracking Status
firefox72 --- fixed

People

(Reporter: whimboo, Assigned: whimboo)

Details

Attachments

(1 file)

As noted by Maja over on https://phabricator.services.mozilla.com/D50233, I missed to add a client.close() call for the case when no new tab gets opened.

It might even be better to move all the calls into the finally block.

Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/291ee5d212aa
[remote] Always close the client connection in add_task() for browser chrome tests. r=remote-protocol-reviewers,maja_zf
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.