Fix interchange of red and blue content when using rgba_4444_textures

cc/output/renderer_pixeltest.cc

Index: cc/output/renderer_pixeltest.cc
diff --git a/cc/output/renderer_pixeltest.cc b/cc/output/renderer_pixeltest.cc
index bd1a79a65e687f6ef9c72497be25a06871581823..fac65be09dfd37c572e09addd596aaa0615631d3 100644
--- a/cc/output/renderer_pixeltest.cc
+++ b/cc/output/renderer_pixeltest.cc
@@ -2115,6 +2115,48 @@ TYPED_TEST(RendererPixelTest, PictureDrawQuadNonIdentityScale) {
       ExactPixelComparator(true)));
 }
 
+TYPED_TEST(RendererPixelTest, PictureDrawQuadTexture4444) {

[enne (OOO), 2014/08/08 18:16:17]

This isn't relevant for the software renderer, so probably should be
TEST_F(GLRendererPixelTest, PictureDrawQuadTexture4444).

[USE eero AT chromium.org, 2014/08/11 11:33:58]

Done.

+  gfx::Size pile_tile_size(1000, 1000);
+  gfx::Rect viewport(this->device_viewport_size_);
+  ResourceFormat texture_format = RGBA_4444;

[enne (OOO), 2014/08/08 18:16:17]

Do all platforms support this format? Should there be a check and an early out?

[USE eero AT chromium.org, 2014/08/11 11:33:58]

No idea. I did not find any relevant function to be used for testing the
support. Is there any?

+
+  RenderPass::Id id(1, 1);
+  gfx::Transform transform_to_root;
+  scoped_ptr<RenderPass> pass =
+      CreateTestRenderPass(id, viewport, transform_to_root);
+
+  // One viewport-filling blue quad
+  scoped_refptr<FakePicturePileImpl> blue_pile =
+      FakePicturePileImpl::CreateFilledPile(pile_tile_size, viewport.size());
+  SkPaint blue_paint;
+  blue_paint.setColor(SK_ColorBLUE);
+  blue_pile->add_draw_rect_with_paint(viewport, blue_paint);
+  blue_pile->RerecordPile();
+
+  gfx::Transform blue_content_to_target_transform;
+  SharedQuadState* blue_shared_state = CreateTestSharedQuadState(
+      blue_content_to_target_transform, viewport, pass.get());
+
+  PictureDrawQuad* blue_quad = pass->CreateAndAppendDrawQuad<PictureDrawQuad>();
+  blue_quad->SetNew(blue_shared_state,
+                    viewport,
+                    gfx::Rect(),
+                    viewport,
+                    gfx::RectF(0.f, 0.f, 1.f, 1.f),
+                    viewport.size(),
+                    texture_format,
+                    viewport,
+                    1.f,
+                    PicturePileImpl::CreateFromOther(blue_pile));
+
+  RenderPassList pass_list;
+  pass_list.push_back(pass.Pass());
+
+  EXPECT_TRUE(this->RunPixelTest(&pass_list,
+                                 base::FilePath(FILE_PATH_LITERAL("blue.png")),
+                                 FuzzyPixelOffByOneComparator(true)));

[enne (OOO), 2014/08/08 18:16:17]

Use an exact comparator.

[USE eero AT chromium.org, 2014/08/11 11:33:58]

Done. For that to work, I changed the blue components of the blue.png image from
254 to 255.

+}
+
 TYPED_TEST(RendererPixelTest, WrapModeRepeat) {
   gfx::Rect rect(this->device_viewport_size_);