Open
Bug 1682014
Opened 4 years ago
Updated 3 years ago
Switch contrast audit getImageCtx from drawWindow to drawSnapshot for Fission
Categories
(DevTools :: Accessibility Tools, task)
DevTools
Accessibility Tools
Tracking
(Fission Milestone:Future)
NEW
| Fission Milestone | Future |
People
(Reporter: smacleod, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: dt-fission-future)
CanvasRenderingContext2D.drawWindow() is synchronous and won't work with fission, so contrast.js[1] must be updated to use the asynchronous drawSnapshot[2]
|
||
Comment 1•4 years ago
|
||
Hmm, drawSnapshot solves the problem of combining the entire tab's process tree into one image. But I think contrast-checking can be process-local, unless you want to detect bad contrast in iframes that have transparent backgrounds. If a process-local detection is good enough, and if drawWindow is called in the correct process for the selected DOM node, then I think this is a case that can keep using drawWindow.
|
||
Updated•4 years ago
|
Whiteboard: dt-fission-future
|
|
||
Comment 3•4 years ago
|
||
Honza said this is not part of dt-fission-m3-mvp and is not considered Fission M7 blocker.
Fission Milestone: M7 → ---
|
||
Comment 4•4 years ago
|
||
dt-fission-future don't need to block Fission MVP.
Fission Milestone: --- → Future
| Comment hidden (Intermittent Failures Robot) |
You need to log in
before you can comment on or make changes to this bug.
Description
•