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)
Tracking
(Not tracked)
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
|
Assignee | |
Comment 1•7 years ago
|
||
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.
|
Reporter | |
Comment 2•7 years ago
|
||
Lets make it blocking bug 1440714 so we don't forget about it. Does that sound fair?
Blocks: 1440714
|
|
Assignee | |
Comment 3•7 years ago
|
||
Yes - that works for me.
|
|
Assignee | |
Updated•7 years ago
|
Assignee: nobody → gbrown
|
|
Assignee | |
Updated•7 years ago
|
Priority: -- → P1
|
|
Assignee | |
Updated•7 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•