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

Reporter

Description

•

13 years ago

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.

Marco Zehe (:MarcoZ)

Updated

•

13 years ago

Blocks: a11yrandomorange

Comment hidden (Legacy TBPL/Treeherder Robot)

philor https://tbpl.mozilla.org/php/getParsedLog.php?id=6359620 Rev3 WINNT 6.1 mozilla-inbound debug test mochitest-other on 2011-09-09 19:33:06 26004 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/toolkit/mozapps/downloads/tests/chrome/test_taskbarprogress_service.xul | The active taskbar progress is not the Download Manager - got true, expected false 26005 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/toolkit/mozapps/downloads/tests/chrome/test_taskbarprogress_service.xul | DownloadTaskbarProgress has the expected taskbar as active - got [xpconnect wrapped nsITaskbarProgress @ 0x17fec640 (native @ 0x2546f07c)], expected [xpconnect wrapped nsITaskbarProgress @ 0x252b5b58 (native @ 0x79ce2b4)] TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/mozapps/extensions/test/browser/browser_select_selection.js | Should have been in the right position - Got 38, expected 46 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/mozapps/extensions/test/browser/browser_select_selection.js | Should have been in the right position - Got 39, expected 47 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/mozapps/extensions/test/browser/browser_select_selection.js | Should have been in the right position - Got 40, expected 48 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/mozapps/extensions/test/browser/browser_select_selection.js | Should have been in the right position - Got 41, expected 49 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/mozapps/extensions/test/browser/browser_select_selection.js | Should have been in the right position - Got 42, expected 50 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/mozapps/extensions/test/browser/browser_select_selection.js | Should have been in the right position - Got 43, expected 51 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/mozapps/extensions/test/browser/browser_select_selection.js | Should have been in the right position - Got 44, expected 52 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/mozapps/extensions/test/browser/browser_select_selection.js | Should have been in the right position - Got 45, expected 38 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/mozapps/extensions/test/browser/browser_select_selection.js | Should have been in the right position - Got 46, expected 39 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/mozapps/extensions/test/browser/browser_select_selection.js | Should have been in the right position - Got 47, expected 40 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/mozapps/extensions/test/browser/browser_select_selection.js | Should have been in the right position - Got 48, expected 52 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/mozapps/extensions/test/browser/browser_select_selection.js | Should have been in the right position - Got 49, expected 42 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/mozapps/extensions/test/browser/browser_select_selection.js | Should have been in the right position - Got 50, expected 43 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/mozapps/extensions/test/browser/browser_select_selection.js | Should have been in the right position - Got 51, expected 44 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/mozapps/extensions/test/browser/browser_select_selection.js | Should have been in the right position - Got 52, expected 45 3029 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/events/test_tree.xul | Test timed out.

Comment hidden (Legacy TBPL/Treeherder Robot)

Phil Ringnalda (:philor)

Reporter

Updated

•

13 years ago

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

alexander :surkov (:asurkov)

Assignee

Comment 48

•

13 years ago

enable logging for the test on inbound https://hg.mozilla.org/integration/mozilla-inbound/rev/d5954910540e please don't mark as fixed

Marco Bonardo [:mak]

Comment 49

•

13 years ago

https://hg.mozilla.org/mozilla-central/rev/d5954910540e

Marco Bonardo [:mak]

Comment 50

•

13 years ago

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.

alexander :surkov (:asurkov)

Assignee

Comment 51

•

13 years ago

I have logs and will look closer, thank you for backing out it.

alexander :surkov (:asurkov)

Assignee

Comment 52

•

13 years ago

Attached patch enable logging (deleted) — Details — Splinter Review

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

Marco Zehe (:MarcoZ)

Updated

•

13 years ago

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

alexander :surkov (:asurkov)

Assignee

Comment 53

•

13 years ago

enable loggin inbound land https://hg.mozilla.org/integration/mozilla-inbound/rev/2db0dafb8f30

Comment hidden (Legacy TBPL/Treeherder Robot)

Ed Morley [:emorley]

Comment 55

•

13 years ago

(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

Comment hidden (Legacy TBPL/Treeherder Robot)

alexander :surkov (:asurkov)

Assignee

Comment 64

•

13 years ago

Attached patch patch (deleted) — Details — 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 hidden (Legacy TBPL/Treeherder Robot)

Marco Zehe (:MarcoZ)

Comment 66

•

13 years ago

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+

alexander :surkov (:asurkov)

Assignee

Comment 67

•

13 years ago

inbound land https://hg.mozilla.org/integration/mozilla-inbound/rev/6bfe61a76eb0

Ed Morley [:emorley]

Comment 68

•

13 years ago

https://hg.mozilla.org/mozilla-central/rev/6bfe61a76eb0

Status: ASSIGNED → RESOLVED

Closed: 13 years ago

Resolution: --- → FIXED

Target Milestone: --- → mozilla10

Marco Zehe (:MarcoZ)

Comment 69

•

13 years ago

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

Comment hidden (Legacy TBPL/Treeherder Robot)

Nobody; OK to take it and work on it

Updated

•

12 years ago

Keywords: intermittent-failure

Nobody; OK to take it and work on it

Updated

•

12 years ago

Whiteboard: [orange]

enable logging 13 years ago alexander :surkov (:asurkov) (deleted), patch	MarcoZ : review+	Details \| Diff \| Splinter Review
patch 13 years ago alexander :surkov (:asurkov) (deleted), patch	MarcoZ : review+	Details \| Diff \| Splinter Review