Closed Bug 1349847 Opened 7 years ago Closed 7 years ago

Crash in mozilla::a11y::Accessible::HasGenericType

Categories

(Core :: Disability Access APIs, defect)

x86
Windows 7
defect
Not set
critical

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox-esr52 53+ fixed
firefox53 blocking fixed
firefox54 + fixed
firefox55 + fixed

People

(Reporter: jchen, Assigned: surkov)

References

(Blocks 1 open bug)

Details

(4 keywords)

Crash Data

This bug was filed from the Socorro interface and is 
report bp-ef120168-9a59-49e7-a243-42a672170323.
=============================================================

Windows crash that seemed to have started suddenly with the 2017-03-21 Aurora build, but doesn't appear in Nightly builds. I didn't see any strong suspects in the regression range [1]. Any ideas?

[1] https://hg.mozilla.org/releases/mozilla-aurora/pushloghtml?fromchange=6a72a11138b9b41a02455a3cb32a8994a7f41934&tochange=0496e366a8f4d1393a8caba1db5f6d1f49da468d
Flags: needinfo?(surkov.alexander)
Crash Signature: [@ mozilla::a11y::Accessible::HasGenericType] → [@ mozilla::a11y::Accessible::HasGenericType] [@ OOM | large | mozilla::a11y::Accessible::HasGenericType]
Assignee: nobody → surkov.alexander
Depends on: 1350057
This is the #6 Windows topcrash in Aurora 20170331004006.
Hi :kanru,
Can you help find someone to look at this? This is the topcrash in Aurora.
Flags: needinfo?(kchen)
[Tracking Requested - why for this release]: this is now also the topcrash in early data for 53.0b10 where the signatures are accounting for 13% of all browser crashes.
timing-wise, this would correlate with the uplift of bug 1342433.
[Tracking Requested - why for this release]: bug 1342433 was also uplifted to esr52
The bug is on my list, and actually a part of 'broken accessible tree' problem we deal with currently. I try to hunt it down with varied success, but not yet caught all issues in the code.
Flags: needinfo?(surkov.alexander)
1000 crashes in one day of beta is probably release-blocking. It's an order of magnitude higher than other top crashes on beta. 

How about backing out the work from bug 1342433? What do you think? 
n-i to jesup and davidb for advice.
Flags: needinfo?(rjesup)
Flags: needinfo?(dbolter)
(In reply to Liz Henry (:lizzard) (needinfo? me) from comment #7)
> 1000 crashes in one day of beta is probably release-blocking. It's an order
> of magnitude higher than other top crashes on beta. 
> 
> How about backing out the work from bug 1342433? What do you think? 

maybe it's worth to try, however it should be a good idea to keep tracking of the related signatures bug 1321384, bug 1309686 and bug 1318645 for possible correlation.
IF we decide to back it out from release53, we will also need to back it out from esr52. Julien FYI.
Flags: needinfo?(jcristau)
(In reply to alexander :surkov from comment #8)
> maybe it's worth to try, however it should be a good idea to keep tracking
> of the related signatures bug 1321384, bug 1309686 and bug 1318645 for
> possible correlation.

in beta 9 all those signatures (including the one in this bug) accounted for ~0.4% of browser crashes.

while data from beta 10 is still quite early and not totally conclusive, it looks like signatures from bug 1321384 & bug 1309686 are no longer present - but the crashing volume here is obviously much worse than before.
(In reply to alexander :surkov from comment #8)
> (In reply to Liz Henry (:lizzard) (needinfo? me) from comment #7)
> > 1000 crashes in one day of beta is probably release-blocking. It's an order
> > of magnitude higher than other top crashes on beta. 
> > 
> > How about backing out the work from bug 1342433? What do you think? 
> 
> maybe it's worth to try, however it should be a good idea to keep tracking
> of the related signatures bug 1321384, bug 1309686 and bug 1318645 for
> possible correlation.

Based on volume analysis from philipp in comment 10, let's backout 1342433. Who generally does that for beta these days?
Flags: needinfo?(dbolter)
Ryan can do the backout for bug 1342433. I don't mind if we leave it on trunk (since 55 is going to remain in nightly for another 6 weeks), up to Alexander for that one
(In reply to Liz Henry (:lizzard) (needinfo? me) from comment #12)
> Ryan can do the backout for bug 1342433. I don't mind if we leave it on
> trunk (since 55 is going to remain in nightly for another 6 weeks), up to
> Alexander for that one

it is hard to argue against data, but this is surprising with me the bug 1342433 spikes the crashes, since it's supposed to be a good thing for the accessible tree correctness overall, so it's definitely worth to keep it on nightly.
Backed out bug 1342433 from everywhere but mozilla-central.
Flags: needinfo?(rjesup)
Flags: needinfo?(kchen)
Flags: needinfo?(jcristau)
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.