Bug 1373381 - Update RGBA8 to BGRA8 since it was renamed in WR cset 264d53d. r?kvark draft
authorKartikaya Gupta <kgupta@mozilla.com>
Tue, 20 Jun 2017 09:38:14 -0400
changeset 597336 af0f9b5a0a2776226206d1b55e1c4648a363c47b
parent 597335 146dae21e2671b5656ef215b85ec58f15c8aa498
child 597337 49124ea0d79940964af08d1d0bbe09000be7b6f4
push id64916
push userkgupta@mozilla.com
push dateTue, 20 Jun 2017 13:40:40 +0000
reviewerskvark
bugs1373381
milestone56.0a1
Bug 1373381 - Update RGBA8 to BGRA8 since it was renamed in WR cset 264d53d. r?kvark MozReview-Commit-ID: 9iGffjS0sKs
gfx/webrender_bindings/WebRenderTypes.h
--- a/gfx/webrender_bindings/WebRenderTypes.h
+++ b/gfx/webrender_bindings/WebRenderTypes.h
@@ -46,39 +46,39 @@ inline Epoch NewEpoch(uint32_t aEpoch) {
   return e;
 }
 
 inline Maybe<WrImageFormat>
 SurfaceFormatToWrImageFormat(gfx::SurfaceFormat aFormat) {
   switch (aFormat) {
     case gfx::SurfaceFormat::R8G8B8X8:
       // TODO: use RGBA + opaque flag
-      return Some(WrImageFormat::RGBA8);
+      return Some(WrImageFormat::BGRA8);
     case gfx::SurfaceFormat::B8G8R8X8:
       // TODO: WebRender will have a BGRA + opaque flag for this but does not
       // have it yet (cf. issue #732).
     case gfx::SurfaceFormat::B8G8R8A8:
-      return Some(WrImageFormat::RGBA8);
+      return Some(WrImageFormat::BGRA8);
     case gfx::SurfaceFormat::B8G8R8:
       return Some(WrImageFormat::RGB8);
     case gfx::SurfaceFormat::A8:
       return Some(WrImageFormat::A8);
     case gfx::SurfaceFormat::R8G8:
       return Some(WrImageFormat::RG8);
     case gfx::SurfaceFormat::UNKNOWN:
       return Some(WrImageFormat::Invalid);
     default:
       return Nothing();
   }
 }
 
 inline gfx::SurfaceFormat
 WrImageFormatToSurfaceFormat(ImageFormat aFormat) {
   switch (aFormat) {
-    case ImageFormat::RGBA8:
+    case ImageFormat::BGRA8:
       return gfx::SurfaceFormat::B8G8R8A8;
     case ImageFormat::A8:
       return gfx::SurfaceFormat::A8;
     case ImageFormat::RGB8:
       return gfx::SurfaceFormat::B8G8R8;
     default:
       return gfx::SurfaceFormat::UNKNOWN;
   }