568719 - [D2D] New surfaces need to be cleared before use

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[Bas Schouten (:bas.schouten)]

Attachment: Clear surfaces when created

There's several bugs particularly with SVG filters that point this problem out when using NVidia drivers and Direct2D without the debug layer. The drivers can choose to reuse existing video memory, which already contains data. We need to make sure our surfaces are empty before using them.

Comment 1

[Jeff Muizelaar [:jrmuizel]]

Comment on attachment 447888 [details] [diff] [review]
Clear surfaces when created

Seems like this should move into a helper function and add a comment about why it needs to be done. It also looks like this patch adds a Flush(). What's that for?

Comment 2

[Bas Schouten (:bas.schouten)]

(In reply to comment #1)
> (From update of attachment 447888 [details] [diff] [review])
> Seems like this should move into a helper function and add a comment about why
> it needs to be done. It also looks like this patch adds a Flush(). What's that
> for?

I can move it into a helper function. The flush needs to go away, it serves no purpose and ended up in there by accident.

Comment 3

[Bas Schouten (:bas.schouten)]

Attachment: Clear surfaces when created v2

Fixed review comments.

Comment 4

[Jeff Muizelaar [:jrmuizel]]

Comment on attachment 448074 [details] [diff] [review]
Clear surfaces when created v2

The comment should include why we need to clear surface. It does not need to include "This clears a D2D surface", that is obvious from the function name.

Comment 5

[Bas Schouten (:bas.schouten)]

Pushed http://hg.mozilla.org/mozilla-central/rev/d8a36dd3fc3c.