Created attachment 469362 [details] [diff] [review] add layersGPUAccelerated This patch adds a "layersGPUAccelerated" variable to the reftests, along the lines of "d2d" and "gtk2Widget" that can be used in fails-if, etc.
Created attachment 469363 [details] [diff] [review] add layersGPUAccelerated now with less patch
Comment on attachment 469363 [details] [diff] [review] add layersGPUAccelerated Looks great! (and works great too with 591150) It might be worthwhile to have a way to distinguish between accelerated layer manager types, OGL will almost certainly have a different set of random failures. This should work great for the moment, about to push my patch queue to try.
I expect we will do platform-specific failure modes, like (layersGPUAccelerated&&linux).
Created attachment 469965 [details] [diff] [review] Just use gWindowUtils Jeff pointed out that we already have a gWindowUtils, so let's just use that instead.
Created attachment 470064 [details] [diff] [review] update for changes in bug 590841
Comment on attachment 470064 [details] [diff] [review] update for changes in bug 590841 r=dbaron, although I wonder why we have code to deal with the possibility of not having window-utils or with it not having a compareCanvases method. I'd rather like to see that code go away. (And I think it has to go away for the fuzzy patch as well.) Maybe something like "accelLayers" would be shorter than "layersGPUAccelerated", though I'm ok with it either way.
Created attachment 471963 [details] [diff] [review] reverse condition Frig. I accidentally reversed the condition here.