884202 - "ASSERTION: Can't create an accessible for the document!" with pagehide event abuse

Status: RESOLVED WORKSFORME

(Core :: Disability Access APIs, defect)

Description

[Jesse Ruderman]

Attachment: testcase

With accessibility enabled, the testcase hits:

###!!! ASSERTION: Can't create an accessible for the document!: 'parentDocAcc', file accessible/src/base/DocManager.cpp, line 389

Comment 1

[Trevor Saunders (:tbsaunde)]

We should fix some day, but seems just case of we've torn down docunment and then somebody adds stuff to DOM.    Probablynot too important.

Comment 2

[alexander :surkov (:asurkov)]

(In reply to Trevor Saunders (:tbsaunde) from comment #1)
> We should fix some day, but seems just case of we've torn down docunment and
> then somebody adds stuff to DOM.    Probablynot too important.

what would a fix look like? just remove an assertion or a11y logic changes?

Comment 3

[Trevor Saunders (:tbsaunde)]

(In reply to alexander :surkov from comment #2)
> (In reply to Trevor Saunders (:tbsaunde) from comment #1)
> > We should fix some day, but seems just case of we've torn down docunment and
> > then somebody adds stuff to DOM.    Probablynot too important.
> 
> what would a fix look like? just remove an assertion or a11y logic changes?

haven't investigated enough to be sure, but removing the assert doesn't sound totally crazy.

Comment 4

[alexander :surkov (:asurkov)]

I think I see this assertion in a real life and it's annoying. I'd be good either fix the problem or remove it.

Comment 5

[Trevor Saunders (:tbsaunde)]

(In reply to alexander :surkov from comment #4)
> I think I see this assertion in a real life and it's annoying. I'd be good
> either fix the problem or remove it.

I didn't say *you* shouldn't care :-p

The no frame for text getting one gets hit a lot more often for me and annoys me a lot more.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

This still asserts when e10s is disabled, but doesn't with e10s enabled. Per discussion with eeejay, I'm just going to call this bug WFM at this point and land the testcase as a crashtest since e10s isn't far away from being the only supported mode anyway.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/34c4c264c904
Add crashtest. r=me

Comment 8

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/34c4c264c904