Closed Bug 679271 Opened 8 years ago Closed 8 years ago
_tree .xul | Test timed out, sometimes with a leak
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.
Attachment #564874 - Flags: review?(marco.zehe) → review+
enable loggin inbound land https://hg.mozilla.org/integration/mozilla-inbound/rev/2db0dafb8f30
(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
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+
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
Alex, do you want to disable logging again for this now that the actual fix landed?
You need to log in before you can comment on or make changes to this bug.