Closed Bug 1005846 Opened 6 years ago Closed Last year

Intermittent test_canvas_focusring.html | button 1 should not be focused

Categories

(Core :: Canvas: 2D, 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.diffSplinter 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: Last year
Flags: needinfo?(lsalzman)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.