Closed
Bug 961874
Opened 10 years ago
Closed 10 years ago
Python client should delete marionette session in cleanup()
Categories
(Remote Protocol :: Marionette, defect)
Tracking
(b2g-v1.3 fixed)
RESOLVED
FIXED
mozilla29
Tracking | Status | |
---|---|---|
b2g-v1.3 | --- | fixed |
People
(Reporter: wlach, Assigned: wlach)
Details
Attachments
(1 file)
477 bytes,
patch
|
mdas
:
review+
|
Details | Diff | Splinter Review |
Per discussion in bug 960063: https://bugzilla.mozilla.org/show_bug.cgi?id=960063#c8 https://bugzilla.mozilla.org/show_bug.cgi?id=960063#c9 Let's delete the session if it exists during cleanup.
Assignee | ||
Comment 1•10 years ago
|
||
Simple patch. Will run through try.
Attachment #8362722 -
Flags: review?(mdas)
Assignee | ||
Comment 2•10 years ago
|
||
Try run: https://tbpl.mozilla.org/?tree=Try&rev=35f3f349e091 (seems to be some sort of weird thing going on with pushlog, so that link looks broken to me atm... if it still is later tonight, I will repush)
Updated•10 years ago
|
Attachment #8362722 -
Flags: review?(mdas) → review+
Assignee | ||
Comment 3•10 years ago
|
||
Ok, the above try push didn't have the right params, but I did another one that worked and shows that we're in good shape. https://tbpl.mozilla.org/?tree=Try&rev=395dda7e4b17
Keywords: checkin-needed
Comment 4•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/4488b21a7d59
Keywords: checkin-needed
Comment 5•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/4488b21a7d59
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Updated•10 years ago
|
status-b2g-v1.3:
--- → fixed
Updated•1 year ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•