potential for crash in mozilla::layout::ScrollbarActivity::SetIsFading

VERIFIED FIXED in Firefox 23

Status

()

Product:
Core
Component:
Layout
Type:
defect
Importance:
--
critical
Status:
VERIFIED FIXED
Opened:
6 years ago
Updated:
6 years ago

People

(Reporter: spohl, Assigned: spohl)

Tracking

({crash, regression})

Version:
23 Branch
Target:
mozilla24
Platform:
x86_64
macOS
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox21 unaffected, firefox22 unaffected, firefox23+ verified, firefox24+ verified, firefox-esr17 unaffected)

Details

(Whiteboard: [lion-scrollbars+])

Attachments

(1 attachment)

Patch
3.97 KB, patch
roc
: review+
akeybl
: approval-mozilla-aurora+
Details | Diff | Splinter Review 
+++ 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.
Posted patch Patch — — Splinter Review
Attachment #760151 - Flags: review?(roc)
Status: NEW → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/625de2ded691
Status: ASSIGNED → RESOLVED
Closed: 6 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.
Keywords: checkin-needed
Verified as fixed on Firefox 23 beta 8 - I could not find any crashes in Socorro with the signature:  [@ mozilla::layout::ScrollbarActivity::SetIsFading].
status-firefox23: fixed → verified
(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
Status: RESOLVED → VERIFIED
status-firefox24: fixed → verified
You need to log in before you can comment on or make changes to this bug.