Closed Bug 824662 Opened 12 years ago Closed 11 years ago

Intermittent test_takeFocus.html | Can't get accessible for plugin

Categories

(Core :: Disability Access APIs, defect)

Product:
Core
Component:
Disability Access APIs
Platform:
All
Windows XP
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla22

People

(Reporter: Gavin, Assigned: surkov)

References

(Blocks 1 open bug)

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

patch
3.11 KB, patch
tbsaunde
: review+
Details | Diff | Splinter Review 
First seen on https://tbpl.mozilla.org/?tree=Mozilla-Inbound&rev=2aa1ecf204b1&jobname=Rev3%20WINNT%205.1%20mozilla-inbound%20pgo%20test%20mochitest-other .

https://tbpl.mozilla.org/php/getParsedLog.php?id=18263138&tree=Mozilla-Inbound&full=1


5621 INFO TEST-INFO | chrome://mochitests/content/a11y/accessible/focus/test_takeFocus.html | must wait for load
5622 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/focus/test_takeFocus.html | Can't get accessible for plugin
...
5644 INFO TEST-INFO | chrome://mochitests/content/a11y/accessible/focus/test_takeFocus.html | Invoke the 'takeFocus for  'plugin' ' test { expected 'focus' event; unexpected 'focus' event;  }
5645 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/focus/test_takeFocus.html | uncaught exception - TypeError: this.accessible is null at chrome://mochitests/content/a11y/accessible/focus/test_takeFocus.html:33
Blocks: a11yrandomorange
at least we need to enable 'tree' logging to see if it helps to understand the problem.
Alexander do you have any time for this one?
(In reply to David Bolter [:davidb] from comment #25)
> Alexander do you have any time for this one?
Flags: needinfo?(surkov.alexander)
enable logging: http://hg.mozilla.org/integration/mozilla-inbound/rev/5c174f3aed3c
Flags: needinfo?(surkov.alexander)
https://hg.mozilla.org/mozilla-central/rev/5c174f3aed3c
Assignee: nobody → surkov.alexander
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Whiteboard: [leave open]
log extraction:

10:21:27     INFO -  4390 INFO TEST-PASS | chrome://mochitests/content/a11y/accessible/focus/test_focusedChild.html | Wrong focused child
10:21:27     INFO -  4391 INFO TEST-PASS | chrome://mochitests/content/a11y/accessible/focus/test_focusedChild.html | Test with ID = 'focusedChild for application accessible' succeed. Event focus was handled.
10:21:27     INFO -  4392 INFO TEST-END | chrome://mochitests/content/a11y/accessible/focus/test_focusedChild.html | finished in 566ms
10:21:27     INFO -  4393 INFO TEST-START | chrome://mochitests/content/a11y/accessible/focus/test_takeFocus.html
10:21:27     INFO -  4394 INFO TEST-INFO | chrome://mochitests/content/a11y/accessible/focus/test_takeFocus.html | must wait for load
10:21:27     INFO -  module enabled: tree
10:21:27     INFO -  4395 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/focus/test_takeFocus.html | Can't get accessible for plugin
10:21:27     INFO -  Event type: reorder. Target: ['iframe@id="testframe" node', address: [object HTMLIFrameElement], role: internal frame, address: [xpconnect wrapped nsIAccessible]]
10:21:27     INFO -  A11Y TREE: process content insertion; 15:47.625
10:21:27     INFO -    {
10:21:27     INFO -      container: 05329B20, a@id='link', idx in parent: 19
10:21:27     INFO -      child: 053745D0, embed@id='plugin', idx in parent: 3
10:21:27     INFO -      child accessible: 053F62E0, node: 053745D0
10:21:27     INFO -      document: 028E0C70, node: 0537D000
10:21:27     INFO -      uri: chrome://mochitests/content/a11y/accessible/focus/test_takeFocus.html
10:21:27     INFO -    }
10:21:27     INFO -  Event type: hide. Target: ['embed@id="plugin" node', address: [object HTMLEmbedElement], role: text container, address: [xpconnect wrapped nsIAccessible]]
10:21:27     INFO -  Event type: text removed, start: 6, length: 1, removed text: . Target: ['a@id="link" node', address: chrome://mochitests/content/a11y/accessible/focus/test_takeFocus.html, role: link, name: 'link', address: [xpconnect wrapped nsIAccessible]]
10:21:27     INFO -  Event type: show. Target: ['embed@id="plugin" node', address: [object HTMLEmbedElement], role: embedded object, address: [xpconnect wrapped nsIAccessible]]
10:21:27     INFO -  Event type: text inserted, start: 6, length: 1, inserted text: . Target: ['a@id="link" node', address: chrome://mochitests/content/a11y/accessible/focus/test_takeFocus.html, role: link, name: 'link', address: [xpconnect wrapped nsIAccessible]]
10:21:27     INFO -  Event type: reorder. Target: ['a@id="link" node', address: chrome://mochitests/content/a11y/accessible/focus/test_takeFocus.html, role: link, name: 'link', address: [xpconnect wrapped nsIAccessible]]
10:21:28     INFO -  scenario #0, registered events number: 1
10:21:28     INFO -  registered: event type: focus, target: ['span@id="aria-link" node', address: [object HTMLSpanElement], role: link, name: 'link', address: [xpconnect wrapped nsIAccessible]]
10:21:28     INFO -  Event queue:
10:21:28     INFO -    invoke: takeFocus for  'aria-link'
plugin gets created in delay so we construct text_container accessible at initial document creation. Then an accessible for plugin element is recreated and doTest is called after text_container was removed and before plugin accessible is constructed. I suggest to delay the test until plugin accessible is constructed.
Attached patch patch — — Splinter Review 

        Attachment #721563 -
        Flags: review?(trev.saunders)

    
    

    
      
  

        Attachment #721563 -
        Flags: review?(trev.saunders) → review+

    
    

    
      
  

    
    

    
  
landed with minor change (no embed role checking to make linux/mac tests feel well): 
http://hg.mozilla.org/integration/mozilla-inbound/rev/c492b9c5c8d6
Whiteboard: [leave open]

    
    

    
      
  

    
    

    
  
(In reply to alexander :surkov from comment #82)
> landed with minor change (no embed role checking to make linux/mac tests
> feel well): 
> http://hg.mozilla.org/integration/mozilla-inbound/rev/c492b9c5c8d6

Thank you for looking at this :-)

    
    

    
      
  

    
    

    
      
  

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/c492b9c5c8d6
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED

    
    

    
      
  

    
    

    
      
  

    
    

    
      
  

    
    

    
      
  

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: