Closed Bug 1337401 Opened 7 years ago Closed 7 years ago

Enable e10s bc accessibility tests on Windows.

Categories

(Core :: Disability Access APIs, defect)

Product:
Core
Component:
Disability Access APIs
Platform:
All
Windows
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: yzen, Assigned: yzen)

References

Details

(Keywords: access, Whiteboard: aes+)

Attachments

(1 file)

1337401 patch
3.32 KB, patch
eeejay
: review+
Details | Diff | Splinter Review 
Right now all tests are enabled when e10s is on.
(In reply to Yura Zenevich [:yzen] from comment #0)
> Right now all tests are enabled when e10s is on.

s/enabled/disabled
Attached patch 1337401 patch — — Splinter Review 
Note the version number is for XP, we do not run these with e10s there.
Attachment #8834496 - Flags: review?(eitan)
Comment on attachment 8834496 [details] [diff] [review]
1337401 patch

Sorry for the wait!
Attachment #8834496 - Flags: review?(eitan) → review+
Pushed by yura.zenevich@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/27352df2ac2c
enabling all passing e10s windows bc tests. r=eeejay
(In reply to Yura Zenevich [:yzen] from comment #2)
> Created attachment 8834496 [details] [diff] [review]
> 1337401 patch
> 
> Note the version number is for XP, we do not run these with e10s there.

I don't think you needed those, we dropped support for XP in 53. Tests on XP should be turned off for mi, mc, and ma.
Whiteboard: aes+
Depends on: 1338034
I am closing this bug in favour of more specific bug 1347787.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Keywords: leave-open
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: