ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/attributes/test_obj_group.xul | Can't get accessible for menu_item2.1, An error occurred - acc is null

RESOLVED FIXED in Firefox 7

Status

()

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

People

(Reporter: mwu, Assigned: surkov)

Tracking

(Blocks: 2 bugs, {intermittent-failure})

Trunk
mozilla8
x86
Windows XP
intermittent-failure
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox7 fixed)

Details

(Whiteboard: [qa-])

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1306544552.1306547991.888.gz

s: talos-r3-xp-044

844 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/attributes/test_obj_group.xul | Can't get accessible for menu_item2.1
845 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/attributes/test_obj_group.xul | [SimpleTest/SimpleTest.js, window.onerror] An error occurred - acc is null at chrome://mochitests/content/a11y/accessible/attributes.js:43
(Reporter)

Comment 1

6 years ago
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1306575383.1306577347.14234.gz

s: talos-r3-xp-032
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Summary: ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/attributes/test_obj_group.xul | Can't get accessible for menu_item2.1 → ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/attributes/test_obj_group.xul | Can't get accessible for menu_item2.1, An error occurred - acc is null
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Assignee)

Updated

6 years ago
Blocks: 518190
(Assignee)

Updated

6 years ago
Blocks: 650585
Comment hidden (Treeherder Robot)
(Assignee)

Comment 170

6 years ago
Created attachment 547935 [details] [diff] [review]
patch
Assignee: nobody → surkov.alexander
Status: NEW → ASSIGNED
Attachment #547935 - Flags: review?(trev.saunders)
Comment on attachment 547935 [details] [diff] [review]
patch

why is  the popup show event listener running at the wrong time?  

>+      this.menuNode = getNode(aID),

';' right? or does ','  do something very special?

>+      this.menuNode = getNode(aID),

same

>+  <hbox flex="1" style="overflow: auto;">

nit, why this addition?

>+  <vbox flex="1">

same
(Assignee)

Comment 172

6 years ago
(In reply to comment #171)
> Comment on attachment 547935 [details] [diff] [review] [review]
> patch
> 
> why is  the popup show event listener running at the wrong time?  

it doesn't running at wrong time, just accessible tree creation happens async, so you can't rely on DOM events dealing with accessible tree, you should use accessible events.

> >+      this.menuNode = getNode(aID),
> 
> ';' right? or does ','  do something very special?

right, nothing special

> >+  <hbox flex="1" style="overflow: auto;">
> 
> nit, why this addition?

to keep test and test results separately and allow scrolling when content exceed the window size, shortly just be able to see tests results, this construction is used through all XUL tests, except oldies like this one.
(In reply to comment #172)
> (In reply to comment #171)
> > Comment on attachment 547935 [details] [diff] [review] [review] [review]
> > patch
> > 
> > why is  the popup show event listener running at the wrong time?  
> 
> it doesn't running at wrong time, just accessible tree creation happens
> async, so you can't rely on DOM events dealing with accessible tree, you
> should use accessible events.

ok, fair enough, but I feel like js / xpcom should be able to get an accessible for any existing content,  do you think its reasonable that thisn't the case always like this one? 

> to keep test and test results separately and allow scrolling when content
> exceed the window size, shortly just be able to see tests results, this
> construction is used through all XUL tests, except oldies like this one.

ok, fair enough
Attachment #547935 - Flags: review?(trev.saunders) → review+
(Assignee)

Comment 174

6 years ago
(In reply to comment #173)

> > it doesn't running at wrong time, just accessible tree creation happens
> > async, so you can't rely on DOM events dealing with accessible tree, you
> > should use accessible events.
> 
> ok, fair enough, but I feel like js / xpcom should be able to get an
> accessible for any existing content,  do you think its reasonable that
> thisn't the case always like this one? 

while it may be nice but it's not big deal. Accessibility processing happens async and no way to force it to run in sync mode, 'cause it complicates the code and actually I don't see any consumers for this behavior.
(Assignee)

Comment 175

6 years ago
landed - http://hg.mozilla.org/mozilla-central/rev/99644756e69e
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
Comment hidden (Treeherder Robot)
(Assignee)

Comment 177

6 years ago
Comment on attachment 547935 [details] [diff] [review]
patch

can we move this test to aurora (do I need approval for testing at all)?
Attachment #547935 - Flags: approval-mozilla-aurora?

Comment 178

6 years ago
Comment on attachment 547935 [details] [diff] [review]
patch

Approved for releases/mozilla-aurora
Attachment #547935 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment hidden (Treeherder Robot)
(Assignee)

Comment 180

6 years ago
Marco, can you land the patch on aurora please?
Comment hidden (Treeherder Robot)
Landed on Aurora on Alexander's behalf: http://hg.mozilla.org/releases/mozilla-aurora/rev/2ba82873fa9c
status-firefox7: --- → fixed
qa- for no QA verification necessary
Whiteboard: [orange] → [orange][qa-]
Keywords: intermittent-failure
Whiteboard: [orange][qa-] → [qa-]
You need to log in before you can comment on or make changes to this bug.