Remove content process crash annotation blacklist
Categories
(Toolkit :: Crash Reporting, task)
Tracking
()
Tracking | Status | |
---|---|---|
firefox77 | --- | fixed |
People
(Reporter: gsvelto, Assigned: gsvelto)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Crash annotations from content process crashes are taken from the child process, except when they're not set in which case they're taken from the parent process annotations. This is done transparently now but before bug 1547698 we wrote the .extra file in three distinct passes, each of which appended annotations to it. Since the main process appended its annotations after the content process some could overwrite the content ones as the earlier ones would be ignored when the file was parsed. To solve this problem we introduced a blacklist of annotations from the main process which should not be written out for content process crash reports.
Since bug 1547698 all annotations for a content process crash are taken from the content process with only the empty ones being filled up from the parent process. As such the blacklist is now useless.
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Assignee | ||
Comment 2•5 years ago
|
||
This needs rebasing after bug 1614933. Fortunately the rebased patch should be even simpler than the original.
Comment 4•5 years ago
|
||
bugherder |
Description
•