Trevor Saunders (:tbsaunde) Reporter
	Description • 9 years ago

This bug was filed from the Socorro interface and is 
report bp-840b56a3-64c0-4a40-9af4-d42822151006.
=============================================================

	alexander :surkov (:asurkov) Assignee
	Comment 1 • 9 years ago

any ideas why nsTArray<Accessible*>::Contains may crash, see [1] and [2]?

[1] http://hg.mozilla.org/mozilla-central/annotate/5f16c6c2b969/accessible/generic/DocAccessible.cpp#l1715
[2] http://hg.mozilla.org/mozilla-central/annotate/b56aeea0c470/xpcom/glue/nsTArray.h#l1098

	David Bolter [:davidb] (NeedInfo me for attention)
	Comment 2 • 9 years ago

A couple on Windows involving OOM:
report bp-5ad10efa-6833-413d-af8f-500812151009.
report bp-b0370a1b-6b09-4548-9c89-5e4372151007.

These crashes are currently very infrequent.

	Brian Birtles (:birtles, Feb 26~ parental leave)
	Comment 3 • 9 years ago

This just crashed facebook messenger for me in 44.0a2.
https://crash-stats.mozilla.com/report/index/7782c200-829c-43c4-8715-b52cd2151218

	David Bolter [:davidb] (NeedInfo me for attention)
	Comment 4 • 9 years ago

Alex can you recreate this in fb messenger?

	alexander :surkov (:asurkov) Assignee
	Comment 5 • 9 years ago

(In reply to David Bolter [:davidb] from comment #4)
> Alex can you recreate this in fb messenger?

no so far, I'll try on Windows, when I'll set up the machine

	Brian Birtles (:birtles, Feb 26~ parental leave)
	Comment 6 • 9 years ago

I'm trying to remember the exact steps that got me here. I think I typed someone's name in the search box, selected the name and then began typing the message. After about 1~2 seconds it crashed.

	Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)
	Comment 7 • 8 years ago

Surkov, this bug is a top crasher on Beta44. Hoping that we can get a fix/repro available soon. Thanks!

	alexander :surkov (:asurkov) Assignee
	Comment 8 • 8 years ago

(In reply to Ritu Kothari (:ritu) from comment #7)
> Surkov, this bug is a top crasher on Beta44. Hoping that we can get a
> fix/repro available soon. Thanks!

is the issue different from bug 1213281 which has been landed on release and aurora channels?

	Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)
	Comment 9 • 8 years ago

Surkov, bug 1213281 landed on Aurora44 (which is now Beta44) so this bug is a different issue.

	alexander :surkov (:asurkov) Assignee
	Comment 11 • 8 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=afc13b65b8e3

	alexander :surkov (:asurkov) Assignee
	Comment 12 • 8 years ago

Comment on attachment 8704270 [details] [diff] [review]
patch

Approval Request Comment
[Feature/regressing bug #]:aria-owns implementation initially landed on this beta
[User impact if declined]:top crash
[Describe test coverage new/current, TreeHerder]:not covered
[Risks and why]: small, couple of null checks
[String/UUID change made/needed]:no

	Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)
	Comment 13 • 8 years ago

Alexander, could you please describe the test coverage you did on this patch? Was it locally and manually tested to ensure there were no obvious regressions? I will not be able to take a patch with "no test coverage" to Beta44.

	alexander :surkov (:asurkov) Assignee
	Comment 14 • 8 years ago

(In reply to Ritu Kothari (:ritu) from comment #13)
> Alexander, could you please describe the test coverage you did on this
> patch? Was it locally and manually tested to ensure there were no obvious
> regressions? I will not be able to take a patch with "no test coverage" to
> Beta44.

I meant there's no test coverage for the patch, because no steps to reproduce. Try server build was ok, so all mochiteset passed. Thus no obvious regressions.

	Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)
	Comment 15 • 8 years ago

Comment on attachment 8704270 [details] [diff] [review]
patch

not null checks, beta44+

	Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)
	Comment 16 • 8 years ago

Surkov, could you also request uplift to Aurora45?

	alexander :surkov (:asurkov) Assignee
	Comment 17 • 8 years ago

(In reply to Ritu Kothari (:ritu) from comment #16)
> Surkov, could you also request uplift to Aurora45?

beta and aurora have a quite different code, aurora might not need a patch like this

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 18 • 8 years ago
bugherder uplift

https://hg.mozilla.org/releases/mozilla-beta/rev/ac1d763699d4

	Carsten Book [:Tomcat]
	Comment 19 • 8 years ago
bugherder uplift

https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/ac1d763699d4

	David Bolter [:davidb] (NeedInfo me for attention)
	Comment 20 • 8 years ago

Alex could you take another look here? We're still crashing in release (FF 44):
https://crash-stats.mozilla.com/signature/?product=Firefox&accessibility=__true__&signature=nsTArray_Impl%3CT%3E%3A%3AContains%3CT%3E+|+mozilla%3A%3Aa11y%3A%3ADocAccessible%3A%3ARemoveDependentIDsFor&_columns=date&_columns=product&_columns=version&_columns=build_id&_columns=platform&_columns=reason&_columns=address&page=1#reports

	alexander :surkov (:asurkov) Assignee
	Comment 21 • 8 years ago

it's not an issue for nightly, if it should be fixed for 44, then the null check should help

	BugBot [:suhaib / :marco/ :calixte]
	Comment 22 • 8 years ago

Crash volume for signature 'mozilla::a11y::DocAccessible::RemoveDependentIDsFor':
 - nightly (version 50): 4 crashes from 2016-06-06.
 - aurora  (version 49): 10 crashes from 2016-06-07.
 - beta    (version 48): 30 crashes from 2016-06-06.
 - release (version 47): 0 crashes from 2016-05-31.
 - esr     (version 45): 0 crashes from 2016-04-07.

Crash volume on the last weeks:
            W. N-1  W. N-2  W. N-3  W. N-4  W. N-5  W. N-6  W. N-7
 - nightly       0       0       1       1       0       1       0
 - aurora        0       2       1       0       2       3       1
 - beta          5       4       2       5       4       4       4
 - release       0       0       0       0       0       0       0
 - esr           0       0       0       0       0       0       0

Affected platform: Windows

	alexander :surkov (:asurkov) Assignee
	Comment 23 • 6 years ago

can see few crashes on 58 (https://crash-stats.mozilla.com/signature/?signature=mozilla%3A%3Aa11y%3A%3ADocAccessible%3A%3ARemoveDependentIDsFor&date=%3E%3D2017-12-05T10%3A42%3A09.000Z&date=%3C2018-03-05T10%3A42%3A09.000Z&_columns=date&_columns=product&_columns=version&_columns=build_id&_columns=platform&_columns=reason&_columns=address&_columns=install_time&_sort=-version&_sort=-date&page=1)

	BugBot [:suhaib / :marco/ :calixte]
	Comment 24 • 5 years ago

Closing because no crashes reported for 12 weeks.

	BugBot [:suhaib / :marco/ :calixte]
	Comment 25 • 5 years ago

Closing because no crashes reported for 12 weeks.

	Calixte Denizet (:calixte)
	Comment 26 • 5 years ago

There are still some crashes so reopen it.