Closed Bug 1528987 Opened 5 years ago Closed 5 years ago

Tell WR about how much scrollframes have scrolled

Categories

(Core :: Graphics: WebRender, enhancement)

Other Branch
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla67
Tracking Status
firefox67 --- fixed

People

(Reporter: kats, Assigned: kats)

Details

(Whiteboard: [gfx-noted])

Attachments

(1 file, 1 obsolete file)

There was some discussion on IRC here: https://mozilla.logbot.info/gfx/20190218#c15982523

The gist of it is that WR can do stuff better if it knows how much all the display items got shifted by due to scrolling. It should be easy to provide WR with the scroll offset for each scrollframe, so that it can do this.

Attached patch WIP (obsolete) — Splinter Review

This patch (which I haven't tried compiling) should provide WR with the gecko scroll offset for each scrollframe when that scrollframe is defined. If you subtract off the display items positions by the scroll offset of the enclosing scrollframe that should give you a normalized value that should be consistent from one DL to the next. Hopefully this is enough for you to test out any changes you want to make.

Flags: needinfo?(gwatson)

I fixed up a couple of minor compile errors, and have been experimenting with this. It seems to provide the missing information that WR needs to simplify and optimize the existing picture caching code. I think we could land this as a standalone patch, it's certainly going to be useful to WR in one way or another.

Flags: needinfo?(gwatson)

Sounds good. I can clean it up and land it then.

Assignee: nobody → kats
Attachment #9045038 - Attachment is obsolete: true
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/59ec14e90a54
Send the gecko scroll offsets for scrollframes to WR. r=gw
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: