84338 ERROR TEST-UNEXPECTED-FAIL | /tests/modules/plugin/test/test_painting.html | partially clipped plugin painted once - got 2, expected 1 84339 ERROR TEST-UNEXPECTED-FAIL | /tests/modules/plugin/test/test_painting.html | painted after invalidate - got 4, expected 2

With Direct2D on windowless plugins will paint in two passes, so twice as many paint events are to be expected.

Comment on attachment 466082 [details] [diff] [review] Expect 2 and 4 painting events for D2D > function invalidate() { >- is(clipped.getPaintCount(), 1, "partially clipped plugin painted once"); >+ netscape.security.PrivilegeManager.enablePrivilege("UniversalXPConnect"); >+ if (Components.classes["@mozilla.org/gfx/info;1"].getService(Components.interfaces.nsIGfxInfo).D2DEnabled) { The GfxInfo structure's only available on Windows, so you'll have to wrap this in a try/catch block. >+ // With D2D we paint windowless plugin with 2 passes, 2 paints are expected. >+ is(clipped.getPaintCount(), 2, "partially clipped plugin painted once"); Painted twice? Or should we say one because it's logically one? >+ } else { >+ is(clipped.getPaintCount(), 1, "partially clipped plugin painted once"); >+ } > function done() { >- is(clipped.getPaintCount(), 2, "painted after invalidate"); >+ netscape.security.PrivilegeManager.enablePrivilege("UniversalXPConnect"); >+ if (Components.classes["@mozilla.org/gfx/info;1"].getService(Components.interfaces.nsIGfxInfo).D2DEnabled) { Same problem - wrap in try/catch.

Updated.

Comment on attachment 466215 [details] [diff] [review] Expect 2 and 4 painting events for D2D v2 much better!