Closed Bug 1401561 Opened 3 years ago Closed 3 years ago
Make sure the JAWS prompt doesn't trigger in non-e10s
Thought of this last night and it looks like it may be possible, I need a e10s check in here  somewhere.  https://bugzilla.mozilla.org/attachment.cgi?id=8905940&action=diff
3 years ago
Priority: -- → P1
Comment on attachment 8911327 [details] [diff] [review] patch Add a check to make sure we don't prompt if e10s is off. Looks like felipe is on pto, Jared, would you mind looking this over? It's a very straight forward change.
Attachment #8911327 - Flags: review?(jaws)
Attachment #8911327 - Flags: review?(jaws) → review+
Double checking, jimm, jaws, is this for 57 only?
If bug 1385991 gets uplifted to 56 then we will need to uplift this to 56 as well.
(In reply to Liz Henry (:lizzard) (needinfo? me) from comment #3) > Double checking, jimm, jaws, is this for 57 only? 57 only.
- updated commit message with r+ from jared
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/d8d8c0972cb4 Avoid prompting about compatibility issues with JAWS when e10s is pref'd off. r=jaws
Hi Jim, should we uplift this fix to Beta57?
Comment on attachment 8911946 [details] [diff] [review] patch Approval Request Comment [Feature/Bug causing the regression]: bug 1385991 [User impact if declined]: buggy prompting code for a11y [Is this code covered by automated tests?]: no [Has the fix been verified in Nightly?]: yes [Needs manual test from QE? If yes, steps to reproduce]: yes, STR: 1) disable e10s in 57, close firefox 2) launch old 8.0 JAWS client 3) launch firefox expected result: no e10s+a11y prompt [List of other uplifts needed for the feature/fix]: [Is the change risky?]: no [Why is the change risky/not risky?]: well understood change [String changes made/needed]: none
Attachment #8911946 - Flags: approval-mozilla-beta?
Comment on attachment 8911946 [details] [diff] [review] patch Improve jaws support, taking it. Should be in 57b5
Attachment #8911946 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Verified fixed in Beta 57.b13. There is no prompt showing up in Nightly with e10s ON or OFF. This may be because of Bug 1413599 comment 5.
Verified Fixed in 58.b6. No prompt is shown when e10s is off.
You need to log in before you can comment on or make changes to this bug.