668428 - move test_focusrings.xul to mochitest-chrome

Status: RESOLVED WONTFIX

(Core :: General, defect)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

we should move test_focusrings.xul to mochitest-chrome.

Comment 1

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: move test_focusrings.xul to chrome (1.0)

Neil, can you review this patch and ensure that if it is moved to chrome we will gain the same test coverage?

Comment 2

[Neil Deakin]

Comment on attachment 543038 [details] [diff] [review]
move test_focusrings.xul to chrome (1.0)

>-      prefs.clearUserPref("tabfocus");
>+    if (SpecialPowers.prefHasUserValue("tabfocus")) {

This one method isn't defined.

Comment 3

[Joel Maher ( :jmaher ) (UTC -8)]

this is an old bug, shall we close it as wontfix?