cc: UYVY video is not premultiplied rgba

cc/resources/video_resource_updater.cc

Index: cc/resources/video_resource_updater.cc
diff --git a/cc/resources/video_resource_updater.cc b/cc/resources/video_resource_updater.cc
index 6c08a31b9c4fa584c3cea2533a038a0f3d5eaec3..cc5249b03c9a176e3129d9dc5dbd0580f8962258 100644
--- a/cc/resources/video_resource_updater.cc
+++ b/cc/resources/video_resource_updater.cc
@@ -40,7 +40,7 @@ VideoFrameExternalResources::ResourceType ResourceTypeForVideoFrame(
     case media::PIXEL_FORMAT_UYVY:
       switch (video_frame->mailbox_holder(0).texture_target) {
         case GL_TEXTURE_2D:

[Daniele Castagna, 2017/02/28 03:17:53]

Isn't CrOS using TEXTURE_EXTERNAL_OES?

[dshwang, 2017/02/28 04:18:44]

Not always TEXTURE_EXTERNAL_OES

GPU video decoder uses TEXTURE_EXTERNAL_OES
GpuMemoryBufferVideoFramePool with native GMB uses TEXTURE_EXTERNAL_OES
GpuMemoryBufferVideoFramePool with software-fallback GMB uses GL_TEXTURE_2D

-          return (video_frame->format() == media::PIXEL_FORMAT_XRGB)
+          return (video_frame->format() != media::PIXEL_FORMAT_ARGB)
                      ? VideoFrameExternalResources::RGB_RESOURCE
                      : VideoFrameExternalResources::RGBA_PREMULTIPLIED_RESOURCE;
         case GL_TEXTURE_EXTERNAL_OES: