Closed
Bug 1190195
Opened 9 years ago
Closed 9 years ago
Cleanup ffprocess management in Talos
Categories
(Testing :: Talos, defect)
Testing
Talos
Tracking
(firefox42 affected)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox42 | --- | affected |
People
(Reporter: parkouss, Assigned: parkouss)
References
Details
Attachments
(1 file)
21.87 KB,
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
ffpprocess*.py files in Talos currently are classes doing stuff with processes and files.
We should rework this, separate the two concerns and remove the class pattern as instances does not make sense here now that android support is dropped.
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8642183 -
Flags: review?(jmaher)
Comment 2•9 years ago
|
||
Comment on attachment 8642183 [details] [diff] [review]
cleanup_ffprocess.patch
Review of attachment 8642183 [details] [diff] [review]:
-----------------------------------------------------------------
this is wonderful! I think you have been reading my mind.
Attachment #8642183 -
Flags: review?(jmaher) → review+
Assignee | ||
Comment 3•9 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•