The default bug view has changed. See this FAQ.

Some subtests in test_doc.html fail with Bug 695480

RESOLVED FIXED in mozilla15

Status

()

Core
Disability Access APIs
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: khuey, Assigned: surkov)

Tracking

unspecified
mozilla15
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Created attachment 616442 [details] [diff] [review]
Test disabling

See the attached patch for the three failures.

Comment 1

5 years ago
So what exactly does bug 695480 do that makes these tests fail? I'm strictly against just disabling these tests since this is an indication that something may break us (and therefore screen readers) because of the patch in that bug.
Bug 695480 makes it so that you can't touch DOM objects in a Window (including nodes, etc) after the Window is navigated.  I'm not sure whether the bugs here are in the test or in the browser UI somewhere.  Unfortunately I can't really figure out what the tests are testing, so I was hoping someone on the a11y team could dig in here.

Comment 3

5 years ago
The tests perform all sorts of mutations on a document inside an iframe. The first of the ones that you disabled opens a new doc inside an iframe and tests whether the accessible tree built from the resulting DOM tree is what we expected, and if certain events are fired before testing that tree. The next one closes that same thing and checks whether afterwards, the tree conforms to the new expectations, mainly that the contents is gone. And the last test youdisabled tests whether content gets provided correctly if nodes are inserted without the document actually having a body element.
(Assignee)

Comment 4

5 years ago
(In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #2)
> Bug 695480 makes it so that you can't touch DOM objects in a Window
> (including nodes, etc) after the Window is navigated.

Could you give JS example?

>  I'm not sure whether
> the bugs here are in the test or in the browser UI somewhere.  Unfortunately
> I can't really figure out what the tests are testing, so I was hoping
> someone on the a11y team could dig in here.

failure log please?
(In reply to alexander :surkov from comment #4)
> (In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #2)
> > Bug 695480 makes it so that you can't touch DOM objects in a Window
> > (including nodes, etc) after the Window is navigated.
> 
> Could you give JS example?

Sure:

let win = window.open();
let documentElement = win.document.documentElement;
win.close();
documentElement.nodeType; // This now throws, because you can't touch things inside the window after it is closed.

> >  I'm not sure whether
> > the bugs here are in the test or in the browser UI somewhere.  Unfortunately
> > I can't really figure out what the tests are testing, so I was hoping
> > someone on the a11y team could dig in here.
> 
> failure log please?

The failure mode is just a timeout.
(Assignee)

Comment 6

5 years ago
(In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #5)
> > failure log please?
> The failure mode is just a timeout.

the log contains some extra info (it shows which test timed out) but please add
gA11yEventDumpToConsole = true;
to test_doc.html to log more details.
Running test_doc.html: http://khuey.pastebin.mozilla.org/1593587
(Assignee)

Comment 8

5 years ago
(In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #7)
> Running test_doc.html: http://khuey.pastebin.mozilla.org/1593587

thank you. I can't spot anything evident in events.js why events comparison could fail (we just compare pointers to accessible objects) so I'm going to make a local build with bug 695480 patch applied to see what happens.
(Assignee)

Comment 9

5 years ago
Created attachment 618195 [details] [diff] [review]
patch
Assignee: nobody → surkov.alexander
Attachment #616442 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #618195 - Flags: review?(marco.zehe)
(Assignee)

Comment 10

5 years ago
(In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #5)
> (In reply to alexander :surkov from comment #4)
> > (In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #2)
> > > Bug 695480 makes it so that you can't touch DOM objects in a Window
> > > (including nodes, etc) after the Window is navigated.
> > 
> > Could you give JS example?
> 
> Sure:
> 
> let win = window.open();
> let documentElement = win.document.documentElement;
> win.close();
> documentElement.nodeType; // This now throws, because you can't touch things
> inside the window after it is closed.

do I understand right, that's applicable to DOM node JS wrapper objects only?

Updated

5 years ago
Attachment #618195 - Flags: review?(marco.zehe) → review+
(Assignee)

Comment 11

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/6071b7a2cb39
Target Milestone: --- → mozilla15
Thanks for digging in here!
(Assignee)

Comment 13

5 years ago
yw!
https://hg.mozilla.org/mozilla-central/rev/6071b7a2cb39
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.