Closed
Bug 659340
Opened 14 years ago
Closed 14 years ago
Add back paranoia on Frame life from last patch of bug 655084
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
People
(Reporter: jesup, Assigned: jesup)
Details
Attachments
(1 file)
|
1.08 KB,
patch
|
ehsan.akhgari
:
review+
mounir
:
checkin+
|
Details | Diff | Splinter Review |
In bug 655084, I had simplified the code at ehsan's request in a way that adds hidden assumptions about other code and the lifetime of a Frame. It's more paranoid and safe to revert the the final version of that piece of the patch. See the end of that bug for discussion.
| Assignee | ||
Updated•14 years ago
|
Status: NEW → ASSIGNED
| Assignee | ||
Comment 1•14 years ago
|
||
Is this sufficient, or do we need to block scripts too?
Attachment #534869 -
Flags: review?(ehsan)
Updated•14 years ago
|
Attachment #534869 -
Flags: review?(ehsan) → review+
| Assignee | ||
Updated•14 years ago
|
Assignee: nobody → rjesup
| Assignee | ||
Comment 2•14 years ago
|
||
Fix checked in as http://hg.mozilla.org/mozilla-central/rev/b3f26adf6b42
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
| Assignee | ||
Updated•14 years ago
|
Attachment #534869 -
Flags: checked-in+
Updated•14 years ago
|
Attachment #534869 -
Flags: checked-in+ → checkin+
You need to log in
before you can comment on or make changes to this bug.
Description
•