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)

Other Branch
enhancement

Tracking

()

RESOLVED FIXED

People

(Reporter: kats, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [wr-reserve] [gfx-noted])

Attachments

(1 file, 2 obsolete files)

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.
Whiteboard: [gfx-noted][wr-reserve][triage] → [wr-reserve] [gfx-noted]
Attached patch WIP mochitest (obsolete) — Splinter Review
This is the mochitest I have half-written. There might be bugs in it.
Assignee: nobody → bugmail
Assignee: bugmail → nobody
Assignee: nobody → bugmail
Attached patch WIP mochitest (obsolete) — Splinter Review
Rebased the mochitest. It's still failing, for the same reason (code is wrong).
Attachment #8945953 - Attachment is obsolete: true
Kats, is this needed for release?
Flags: needinfo?(kats)
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.
Blocks: stage-wr-next
No longer blocks: stage-wr-trains
Flags: needinfo?(kats)
Priority: P2 → P1
Priority: P1 → P3

The plan for bug 1503046 will probably fix this.

Depends on: 1503046
Attached patch WIP mochitestSplinter Review

Rebased. Still fails.

Attachment #8986031 - Attachment is obsolete: true
Assignee: kats → nobody
Severity: normal → S3

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.

Attachment

General

Created:
Updated:
Size: