914505 - Make sure that ThebesLayerBuffer::SupportsAzureContent is correct for the first frame

Status: RESOLVED FIXED

(Core :: Graphics: Layers, defect)

Comment 1

[Matt Woodrow (:mattwoodrow)]

Attachment: WIP

Builds, will try find time later on tonight to get some testing done.

Comment 2

[Matt Woodrow (:mattwoodrow)]

Attachment: Fix

https://tbpl.mozilla.org/?tree=Try&rev=44c0bbc8bf58

Looks like we're ok, and this fixed the rest of my b2g reftest failures.

Comment 3

[Nick Cameron [:nrc]]

Comment on attachment 802803 [details] [diff] [review]
Fix

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

::: gfx/layers/ThebesLayerBuffer.h
@@ +246,5 @@
>    /**
>     * Return a new surface of |aSize| and |aType|.
>     * @param aFlags if ALLOW_REPEAT is set, then the buffer should be configured
>     * to allow repeat-mode, otherwise it should be in pad (clamp) mode
>     */

please comment on the out params

@@ +249,5 @@
>     * to allow repeat-mode, otherwise it should be in pad (clamp) mode
>     */
> +  virtual void
> +  CreateBuffer(ContentType aType, const nsIntRect& aRect, uint32_t aFlags,
> +               gfxASurface** aBlackSurface,gfxASurface** aWhiteSurface,

nit: space after comma

::: gfx/layers/client/ContentClient.h
@@ +152,5 @@
>      ThebesLayerBuffer::DrawTo(aLayer, aTarget, aOpacity, aMask, aMaskTransform);
>    }
>  
> +  virtual void CreateBuffer(ContentType aType, const nsIntRect& aRect, uint32_t aFlags,
> +                            gfxASurface** aBlackSurface,gfxASurface** aWhiteSurface,

nit: space after comma

@@ +153,5 @@
>    }
>  
> +  virtual void CreateBuffer(ContentType aType, const nsIntRect& aRect, uint32_t aFlags,
> +                            gfxASurface** aBlackSurface,gfxASurface** aWhiteSurface,
> +                            RefPtr<gfx::DrawTarget>* aBlackDT, RefPtr<gfx::DrawTarget>* aWhiteDT);

MOZ_OVERRIDE

@@ +233,5 @@
>      return ThebesLayerBuffer::BufferRotation();
>    }
>  
> +  virtual void CreateBuffer(ContentType aType, const nsIntRect& aRect, uint32_t aFlags,
> +                            gfxASurface** aBlackSurface,gfxASurface** aWhiteSurface,

nit again

@@ +234,5 @@
>    }
>  
> +  virtual void CreateBuffer(ContentType aType, const nsIntRect& aRect, uint32_t aFlags,
> +                            gfxASurface** aBlackSurface,gfxASurface** aWhiteSurface,
> +                            RefPtr<gfx::DrawTarget>* aBlackDT, RefPtr<gfx::DrawTarget>* aWhiteDT);

MOZ_OVERRIDE

::: gfx/layers/opengl/ThebesLayerOGL.cpp
@@ +357,5 @@
>  
>    // ThebesLayerBuffer interface
> +  void
> +  CreateBuffer(ContentType aType, const nsIntRect& aRect, uint32_t aFlags,
> +               gfxASurface** aBlackSurface,gfxASurface** aWhiteSurface,

nit: space after comma

@@ +358,5 @@
>    // ThebesLayerBuffer interface
> +  void
> +  CreateBuffer(ContentType aType, const nsIntRect& aRect, uint32_t aFlags,
> +               gfxASurface** aBlackSurface,gfxASurface** aWhiteSurface,
> +               RefPtr<gfx::DrawTarget>* aBlackDT, RefPtr<gfx::DrawTarget>* aWhiteDT)

MOZ_OVERRIDE

Comment 4

[Matt Woodrow (:mattwoodrow)]

http://hg.mozilla.org/integration/mozilla-inbound/rev/771e3bbcd721

Comment 5

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/771e3bbcd721