Closed Bug 679271 Opened 13 years ago Closed 13 years ago

Intermittent a11y/accessible/events/test_tree.xul | Test timed out, sometimes with a leak

Categories

(Core :: Disability Access APIs, defect)

Product:
Core
Component:
Disability Access APIs
Platform:
x86_64
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla10

People

(Reporter: philor, Assigned: surkov)

References

(Blocks 1 open bug)

Details

(Keywords: intermittent-failure)

Attachments

(2 files)

enable logging
3.25 KB, patch
MarcoZ
: review+
Details | Diff | Splinter Review
patch
1.23 KB, patch
MarcoZ
: review+
Details | Diff | Splinter Review 
http://tinderbox.mozilla.org/showlog.cgi?log=Mozilla-Inbound/1313451229.1313454933.20357.gz
Rev3 Fedora 12x64 mozilla-inbound debug test mochitest-other on 2011/08/15 16:33:49
s: talos-r3-fed64-026

2995 INFO TEST-PASS | chrome://mochitests/content/a11y/accessible/events/test_tree.xul | test with ID = 'insertRow: TreeRowCountChanged' failed. No TreeRowCountChanged event.
2996 INFO TEST-PASS | chrome://mochitests/content/a11y/accessible/events/test_tree.xul | test with ID = 'TreeInvalidated on insertRow' failed. No TreeInvalidated event.
--DOMWINDOW == 61 (0x35007b8) [serial = 115] [outer = 0x2cbf820] [url = about:blank]
--DOMWINDOW == 60 (0x2b85a98) [serial = 114] [outer = 0x2cbf820] [url = about:blank]
--DOMWINDOW == 59 (0x2cbf898) [serial = 113] [outer = (nil)] [url = about:blank]
2997 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/events/test_tree.xul | Test timed out.
Summary: Intermittent a11y/accessible/events/test_tree.xul | Test timed out. → Intermittent a11y/accessible/events/test_tree.xul | Test timed out, sometimes with a leak
enable logging for the test on inbound https://hg.mozilla.org/integration/mozilla-inbound/rev/d5954910540e

please don't mark as fixed
I had to backout the logging you pushed in comment 48 since it caused mochitest-other on Windows and Linux debug to permanetly leak... I wonder if you should take a deeper look at what a11y logging does if it causes perma leaks.
I have logs and will look closer, thank you for backing out it.
Attached patch enable logging — — Splinter Review 

        Attachment #564874 -
        Flags: review?(marco.zehe)

    
  

        Attachment #564874 -
        Flags: review?(marco.zehe) → review+

    
    

    
      
  

    
    

    
  
(In reply to alexander surkov from comment #53)
> enable loggin inbound land
> https://hg.mozilla.org/integration/mozilla-inbound/rev/2db0dafb8f30

https://hg.mozilla.org/mozilla-central/rev/2db0dafb8f30

Leaving open since logging only.
Assignee: nobody → surkov.alexander
Status: NEW → ASSIGNED

    
    

    
      
  

    
    

    
      
  

    
    

    
      
  

    
    

    
      
  

    
    

    
      
  

    
    

    
      
  

    
    

    
      
  

    
    

    
      
  

    
    

    
  

      
      
      
      

        Attached patch
          
          
        patch
        — 
          — Splinter Review
      

    


  
a log:

Event queue:
  invoke: invalidate row
registered: event type: name changed, target: no target description
registered: event type: name changed, target: no target description
registered: event type: TreeInvalidated, target: no target description

Event type: name changed. Target: ['tree@id="tree" node', address: 0x8605008, role: gridcell, name: 'row1_col1', address: 0x7b6d290]. Listeners count: 1
EQ matched: name changed

4285 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/events/test_tree.xul | Test timed out.

insertRow test: change from "row0_col0"/"row1_col1" to "aloha 1x0"/"aloha 1x1"

name changed is fired for old value. I see that mouse move over the tree restores all previous values in the tree what means data.colsText didn't keep new values in  reachable way. It appears this bug is caused by this behavior.

        Attachment #566126 -
        Flags: review?(marco.zehe)

    
    

    
      
  

    
    

    
  
Comment on attachment 566126 [details] [diff] [review]
patch

r=me. Curious that this is a matter of parameters that this fails intermittently. Thanks for finding it!

        Attachment #566126 -
        Flags: review?(marco.zehe) → review+

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/6bfe61a76eb0
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10

    
    

    
  
Alex, do you want to disable logging again for this now that the actual fix landed?

    
    

    
      
  

    
    

    
      
  

    
    

    
      
  

    
    

    
      
  

    
    

    
      
  
Whiteboard: [orange]

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: