729562 - Make SVG paint using an nsRenderingContext, and kill nsSVGRenderState

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Jonathan Watt [:jwatt]]

To fix bug 614732 (make SVG use display lists for painting, hit-testing and invalidation) SVG needs to stop painting using an nsSVGRenderState and just use a plain nsRenderingContext. nsRenderingContext is all that the display list code knows about, and it's all that it will pass down for painting.

To be able to get rid of nsSVGRenderState, nsRenderingContext will need to be able to carry two more pieces of state for SVG. We need to know whether we're painting normally/for clip/for masking; and sometimes we need to know if we're painting to a window.

Comment 1

[Jonathan Watt [:jwatt]]

Attachment: patch

Comment 2

[Jonathan Watt [:jwatt]]

Attachment: patch, using new nsRenderingContext::SetUserData method

Bas asked me on IRC to do this by adding facilities to nsRenderingContext to allow it to support user data. This should make future Azure migration work slightly easier.

Comment 3

[Jonathan Watt [:jwatt]]

Attachment: patch, using new nsRenderingContext::SetUserData method

I had to tweak things a little to get the patch to pass Try.

Comment 4

[Bas Schouten (:bas.schouten)]

Comment on attachment 599921 [details] [diff] [review]
patch, using new nsRenderingContext::SetUserData method

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

As far as I can see this looks fine, although this code isn't my area of expertise.

Comment 5

[Jonathan Watt [:jwatt]]

Thanks, Bas!

For the record, before checkin I fixed a bug in the UserData::Remove method, where it was reading too far, and possibly reading past the end of the allocated memory.

Comment 6

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/412c948ce100