simplify faulty's valid process logic

RESOLVED FIXED in Firefox 66

Status

()

enhancement
RESOLVED FIXED
3 months ago
3 months ago

People

(Reporter: froydnj, Assigned: froydnj)

Tracking

(Blocks 2 bugs)

unspecified
mozilla66
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox66 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 months ago

There's no need to explicit list out all the child process types that we
care about, when a simple is-this-the-parent-process check can do the
job just as well.

(Assignee)

Comment 1

3 months ago
Attachment #9035754 - Flags: review?(choller)
(Assignee)

Updated

3 months ago
Blocks: 1514039

Comment on attachment 9035754 [details] [diff] [review]
simplify faulty's valid process logic

Thanks, that seems right to me.

Faulty is :posidron's tool, so I'd like to make sure he is aware of this. Christoph, any objections?

Attachment #9035754 - Flags: review?(choller)
Attachment #9035754 - Flags: review+
Attachment #9035754 - Flags: feedback?
Attachment #9035754 - Flags: feedback? → feedback?(cdiehl)
Attachment #9035754 - Flags: feedback?(cdiehl) → feedback+

Comment 3

3 months ago
Pushed by nfroyd@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/97c36c63095b
simplify faulty's valid process logic; r=decoder,f=posidron

Comment 4

3 months ago
bugherder
Status: NEW → RESOLVED
Last Resolved: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
You need to log in before you can comment on or make changes to this bug.