Closed
Bug 1421385
Opened 7 years ago
Closed 21 days ago
Improve WR hit-testing correctness with respect to rounded corners
Categories
(Core :: Graphics: WebRender, enhancement, P3)
Tracking
()
RESOLVED
FIXED
People
(Reporter: kats, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: [wr-reserve] [gfx-noted])
Attachments
(1 file, 2 obsolete files)
5.07 KB,
patch
|
Details | Diff | Splinter Review |
When I wrote the code to generate the nsDisplayCompositorHitTestInfo display items I mistakenly assumed they would get clip chains populated on them to deal with things like masks and rounded corners and whatnot. But they don't. So we need to fix that, probably by either moving where these items are created (so that the relevant clip-related RAII things are on the stack) or by wrapping the current call sites with the necessary RAII things.
Updated•7 years ago
|
Whiteboard: [gfx-noted][wr-reserve][triage] → [wr-reserve] [gfx-noted]
Updated•7 years ago
|
Blocks: stage-wr-trains
Updated•6 years ago
|
Priority: P3 → P2
Reporter | ||
Comment 1•6 years ago
|
||
This is the mochitest I have half-written. There might be bugs in it.
Assignee: nobody → bugmail
Reporter | ||
Updated•6 years ago
|
Assignee: bugmail → nobody
Reporter | ||
Updated•6 years ago
|
Assignee: nobody → bugmail
Reporter | ||
Comment 2•6 years ago
|
||
Rebased the mochitest. It's still failing, for the same reason (code is wrong).
Attachment #8945953 -
Attachment is obsolete: true
Reporter | ||
Comment 4•6 years ago
|
||
If we haven't hit it so far maybe we don't need it for release, but we should do it for the "immediately after release" target.
Updated•5 years ago
|
Priority: P1 → P3
Reporter | ||
Comment 5•5 years ago
|
||
The plan for bug 1503046 will probably fix this.
Depends on: 1503046
Reporter | ||
Updated•4 years ago
|
Blocks: apz-hit-test
Reporter | ||
Comment 6•4 years ago
|
||
Rebased. Still fails.
Attachment #8986031 -
Attachment is obsolete: true
Reporter | ||
Updated•4 years ago
|
Assignee: kats → nobody
Updated•2 years ago
|
Severity: normal → S3
Comment 7•21 days ago
|
||
This happened.
Status: NEW → RESOLVED
Closed: 21 days ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•