The content-shutdown callback should return quickly, to allow the dead child process to be reaped

RESOLVED FIXED

Status

Fennec Graveyard
General
--
major
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Brian Crowder, Assigned: mfinkle)

Tracking

Details

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
We're currently displaying a dialog here which can lead to some weirdness in pathological cases.  We should display the dialog afterwards.
(Reporter)

Updated

7 years ago
Assignee: nobody → mark.finkle
How much and what type of weirdness are we seeing on the edge/pathological cases, Brian?
(Reporter)

Comment 2

7 years ago
Well, when left running overnight, we will create a dozen or more child processes, all of which are zombies, and all of which must be dismissed in a similar dialog.  Also, over a shorter period of time, a weird, unlabeled dialog will appear with just "Ok|Cancel" options and no title.
Created attachment 495592 [details] [diff] [review]
patch

I don't see any extra corrupt dialogs with this fix. Can you test it too Crowder?
Attachment #495592 - Flags: review?(mbrubeck)
Attachment #495592 - Flags: review?(mbrubeck) → review+
pushed:
http://hg.mozilla.org/mobile-browser/rev/25904bf6e61e
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED

Comment 5

7 years ago
Can someone please provide some steps to reproduce?
You need to log in before you can comment on or make changes to this bug.