Closed Bug 790164 Opened 9 years ago Closed 9 years ago
Shell::Dispatch Touch Event should keep content Pres Shell alive while using it
I mentioned the problem few times in review comments, yet we have now an sg:crit bug ;) https://bugzilla.mozilla.org/show_bug.cgi?id=732052#c44
Assignee: nobody → wjohnston
Attachment #660146 - Flags: review?(bugs)
Whoops. Sorry about that :(
Comment on attachment 660147 [details] [diff] [review] Patch v2 No need to assign nullptr.
Attachment #660147 - Flags: review?(bugs) → review+
Comment on attachment 660147 [details] [diff] [review] Patch v2 I assume we want to move this forward. [Approval Request Comment] Bug caused by (feature/regressing bug #): bug 732052 User impact if declined: Security risks if the presShell is destroyed by the touch event. Testing completed (on m-c, etc.): landed on mc today (9/11/12) Risk to taking this patch (and alternatives if risky): Low risk. String or UUID changes made by this patch: None.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
sec-critical regression in FF16 - approved for landing on branches.
You need to log in before you can comment on or make changes to this bug.