Closed Bug 1441457 Opened 7 years ago Closed 7 years ago

[mozdevice] Fix return value of processExist in devicemanager.py

Categories

(Testing :: Mozbase, enhancement, P1)

Product:
Testing
Component:
Mozbase
Version:
Version 3
Type:
enhancement

Tracking

(Not tracked)

Status:
RESOLVED DUPLICATE of bug 1440714

People

(Reporter: whimboo, Assigned: gbrown)

Details

The method falsely says that it returns True if the process exists. Instead it returns the remote process id. We should fix that, or rename the method so it is more obvious what it does. Please note that several places make use of this method: https://dxr.mozilla.org/mozilla-central/search?q=processExist&=mozilla-central
I will be making a lot of mozdevice changes in bug 1440714, some time in March. I could easily roll this issue into that bug: resolve this as a dup, if you like.
Lets make it blocking bug 1440714 so we don't forget about it. Does that sound fair?
Blocks: 1440714
Yes - that works for me.
Assignee: nobody → gbrown
Priority: -- → P1
No longer blocks: 1440714
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.