Closed Bug 1005846 Opened 11 years ago Closed 6 years ago

Intermittent test_canvas_focusring.html | button 1 should not be focused

Categories

(Core :: Graphics: Canvas2D, defect, P3)

x86
Windows XP
defect

Tracking

()

RESOLVED FIXED

People

(Reporter: cbook, Assigned: lsalzman)

References

()

Details

(Keywords: intermittent-failure, Whiteboard: [test disabled on Windows debug][leave open])

Attachments

(1 file)

Windows XP 32-bit mozilla-inbound debug test mochitest-1 on 2014-05-05 02:38:49 PDT for push ba31cd1620bd slave: t-xp32-ix-095 https://tbpl.mozilla.org/php/getParsedLog.php?id=39042836&tree=Mozilla-Inbound 2425 INFO TEST-UNEXPECTED-FAIL | /tests/content/canvas/test/test_canvas_focusring.html | button 1 should not be focused
Windows debug-only failure that affects 32+. Rick, any chance you could please take a look? :)
Flags: needinfo?(cabanier)
Test disabled on Windows debug due to ongoing failures and lack of responsiveness in the bug. https://hg.mozilla.org/integration/mozilla-inbound/rev/c5f6f799213f
Flags: needinfo?(cabanier)
Whiteboard: [test disabled on Windows debug][leave open]
I think this would be fixed by bug 1120371 (removal of drawCustomFocusRing).
Depends on: 1120371
Attached patch 1005846.diff (deleted) — Splinter Review
With the rewrite to pushPrefEnv (See bug 1056851, comment40), it makes this test a permanent failure on Windows. I think this test needs to be disabled on Windows, because it doesn't work there. In the end, this test will be removed anyway, as part of bug 1120371.
Attachment #8564962 - Flags: review?(ryanvm)
Comment on attachment 8564962 [details] [diff] [review] 1005846.diff This test is obviously unowned given the lack of response in this bug. Kill it with fire.
Attachment #8564962 - Flags: review?(ryanvm) → review+
Bulk assigning P3 to all open intermittent bugs without a priority set in Firefox components per bug 1298978.
Priority: -- → P3
The leave-open keyword is there and there is no activity for 6 months. :lsalzman, maybe it's time to close this bug?
Flags: needinfo?(lsalzman)
The particular failing reftest is now removed, thus solving this bug.
Assignee: nobody → lsalzman
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(lsalzman)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: