Closed Bug 881022 Opened 7 years ago Closed 7 years ago
potential for crash in mozilla::layout::Scrollbar
Activity::Set Is Fading
+++ This bug was initially created as a clone of Bug #877097 +++ Bug 877097 revealed that ScrollbarActivity::SetIsFading may crash in similar fashion as ScrollbarActivity::UpdateOpacity used to. A similar fix should be written for ScrollbarActivity::SetIsFading.
Attachment #760151 - Flags: review?(roc) → review+
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
Comment on attachment 760151 [details] [diff] [review] Patch [Approval Request Comment] Bug caused by (feature/regressing bug #): Bug 636564 User impact if declined: Potential for crashes, similar to the ones reported in bug 877097. Testing completed (on m-c, etc.): Has been on m-c for a few days. Risk to taking this patch (and alternatives if risky): low, since the fix closely matches what was done for bug 877097. String or IDL/UUID changes made by this patch: none
Attachment #760151 - Flags: approval-mozilla-aurora?
Attachment #760151 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Requesting checkin for aurora.
Verified as fixed on Firefox 23 beta 8 - I could not find any crashes in Socorro with the signature: [@ mozilla::layout::ScrollbarActivity::SetIsFading].
(In reply to Simona B [QA] from comment #7) > Verified as fixed on Firefox 23 beta 8 - I could not find any crashes in > Socorro with the signature: [@ > mozilla::layout::ScrollbarActivity::SetIsFading]. Verified FF 24b8
You need to log in before you can comment on or make changes to this bug.