1344396 - Build display lists directly instead of using an intermediate representation

Status: RESOLVED FIXED

(Core :: Graphics: WebRender, enhancement)

Description

[Jeff Muizelaar [:jrmuizel]]

This make us build display lists directly instead of through the now pointless IPC goop.

Comment 1

[Jeff Muizelaar [:jrmuizel]]

Attachment: Build the display lists directly

This is a largely uninteresting patch that just uses the DisplayListBuilder directly. A wonderful cleanup patch will come after this. One of the more interesting pieces is the use of PushBuiltDisplayList. This is needed for handling empty transactions. See https://github.com/servo/webrender/pull/934 for more info.

I'll take the first review that I can get.

Comment 2

[Kartikaya Gupta (email:[e-mail])]

Comment on attachment 8843500 [details] [diff] [review]
Build the display lists directly

Review of attachment 8843500 [details] [diff] [review]:
-----------------------------------------------------------------

Remove the commented-out mCommands in WebRenderDisplayItemLayer.h. Also at some point we'll want to rename all the CreateWebRenderCommand functions but i guess we can wait until the dust settles.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/projects/graphics/rev/0d411df025f5
Build the display lists directly. r=kats

Comment 4

[Kartikaya Gupta (email:[e-mail])]

https://hg.mozilla.org/mozilla-central/rev/0d411df025f5