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

This bug was filed from the Socorro interface and is 
report bp-0a09a2e1-8786-46fa-98b4-f2a9d2150925.
=============================================================

Looks like it has been around since bug 1196460 landed.

More reports here: http://is.gd/71v46H

I think we need to move this check outside the 64 bit only section of GetChildIDFor:
#ifdef _WIN64
  if (!aAccessible ...
    return 0;

	Pulsebot
	Comment 2 • 9 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/106456a09909

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 3 • 9 years ago

https://hg.mozilla.org/mozilla-central/rev/106456a09909

	Robert Kaiser
	Comment 4 • 9 years ago

[Tracking Requested - why for this release]:
This is the top crash for browser processes in Dev Edition 43, can it be uplifted?

	Liz Henry (:lizzard) (relman/hg->git project)
	Comment 5 • 9 years ago

Trevor, can you file an uplift approval request for aurora? Will this apply cleanly to aurora?

	Trevor Saunders (:tbsaunde) Assignee
	Comment 6 • 9 years ago

(In reply to Liz Henry (:lizzard) (needinfo? me) from comment #5)
> Trevor, can you file an uplift approval request for aurora? Will this apply
> cleanly to aurora?

I think so.

	Trevor Saunders (:tbsaunde) Assignee
	Comment 7 • 9 years ago

Comment on attachment 8666898 [details] [diff] [review]
null check aAccessible in GetChildIDFor() for 32 bit as well as 64 bit windows

Approval Request Comment
[Feature/regressing bug #]: a11y e10s work for windows
[User impact if declined]: crashes
[Describe test coverage new/current, TreeHerder]: its been on m-c for a couple days
[Risks and why]: low just adding a null check
[String/UUID change made/needed]:

	Liz Henry (:lizzard) (relman/hg->git project)
	Comment 8 • 9 years ago

Comment on attachment 8666898 [details] [diff] [review]
null check aAccessible in GetChildIDFor() for 32 bit as well as 64 bit windows

Fix for top crash; let's take this for aurora

	Carsten Book [:Tomcat]
	Comment 9 • 9 years ago

https://hg.mozilla.org/releases/mozilla-aurora/rev/2c7950c98ec4

	Liz Henry (:lizzard) (relman/hg->git project)
	Comment 10 • 9 years ago

Tracking this since it's a topcrash (and in case it reopens)