Basic pixel tests for cc

cc/gl_renderer_pixeltest.cc

+// Copyright 2012 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "cc/gl_renderer.h"
+
+#include "base/file_util.h"
+#include "base/path_service.h"
+#include "cc/cc_paths.h"
+#include "cc/draw_quad.h"
+#include "cc/prioritized_resource_manager.h"
+#include "cc/resource_provider.h"
+#include "cc/test/pixel_test_graphics_context.h"
+#include "cc/test/pixel_test_utils.h"
+#include "testing/gtest/include/gtest/gtest.h"
+#include "ui/gfx/codec/png_codec.h"
+#include "ui/gl/gl_implementation.h"
+
+namespace cc {
+namespace {
+
+class FakeRendererClient : public RendererClient {
+ public:
+  FakeRendererClient()
+  {
+  }
+
+  // RendererClient methods.
+  virtual const gfx::Size& deviceViewportSize() const OVERRIDE {
+    static gfx::Size fakeSize(200, 200);

[danakj, 2012/12/03 19:14:10]

nit: fake_size

+    return fakeSize;
+  }
+  virtual const LayerTreeSettings& settings() const OVERRIDE {
+    static LayerTreeSettings fakeSettings;

[danakj, 2012/12/03 19:14:10]

nit: fake_settings

+    return fakeSettings;
+  }
+  virtual void didLoseContext() OVERRIDE { }
+  virtual void onSwapBuffersComplete() OVERRIDE { }
+  virtual void setFullRootLayerDamage() OVERRIDE { }
+  virtual void setManagedMemoryPolicy(const ManagedMemoryPolicy&) OVERRIDE {}
+  virtual void enforceManagedMemoryPolicy(
+      const ManagedMemoryPolicy&) OVERRIDE {}
+  virtual bool hasImplThread() const OVERRIDE { return false; }
+};
+
+
+class GLRendererPixeltest : public testing::Test {

[enne (OOO), 2012/12/03 21:40:29]

Can you be consistent in your capitalization of Test? Pixeltest (here) vs.
PerfTest and PixelTest (elsewhere)?

+ protected:
+  GLRendererPixeltest() {}
+
+  virtual void SetUp() {
+    gfx::InitializeGLBindings(gfx::kGLImplementationOSMesaGL);
+    context_ = PixelTestGraphicsContext::create();
+    resource_provider_ = ResourceProvider::create(context_.get());
+    renderer_ = GLRenderer::create(&fake_client_, resource_provider_.get());
+  }
+
+  scoped_ptr<GraphicsContext> context_;
+  FakeRendererClient fake_client_;
+  scoped_ptr<ResourceProvider> resource_provider_;
+  scoped_ptr<GLRenderer> renderer_;
+};
+
+TEST_F(GLRendererPixeltest, simpleGreenRect) {
+  gfx::Rect rect(0, 0, 200, 200);
+
+  RenderPass::Id id(1, 1);
+  scoped_ptr<RenderPass> pass = RenderPass::Create();
+  pass->SetNew(id, rect, rect, gfx::Transform());
+
+  scoped_ptr<SharedQuadState> shared_state = SharedQuadState::Create();
+  shared_state->SetAll(gfx::Transform(),
+                       rect,

[danakj, 2012/12/03 19:14:10]

can you use temp vars for these so we can see what's being set from the caller?

+                       rect,
+                       rect,
+                       false,
+                       1);
+
+  scoped_ptr<SolidColorDrawQuad> color_quad = SolidColorDrawQuad::Create();
+  color_quad->SetNew(shared_state.get(), rect, SK_ColorGREEN);
+
+  pass->quad_list.append(color_quad.PassAs<DrawQuad>());
+
+  RenderPassList pass_list;
+  pass_list.push_back(pass.get());
+  RenderPassIdHashMap pass_map;
+  pass_map.add(id, pass.PassAs<RenderPass>());
+
+  renderer_->drawFrame(pass_list, pass_map);
+
+  SkBitmap bitmap;
+  bitmap.setConfig(SkBitmap::kARGB_8888_Config, 200, 200);
+  bitmap.allocPixels();
+  unsigned char* pixels = static_cast<unsigned char*>(bitmap.getPixels());
+  renderer_->getFramebufferPixels(pixels, gfx::Rect(0, 0, 200, 200));
+
+  FilePath test_data_dir;
+  ASSERT_TRUE(PathService::Get(cc::DIR_TEST_DATA, &test_data_dir));
+  // test::WritePNGFile(bitmap, test_data_dir.AppendASCII("green.png"));
+  EXPECT_TRUE(test::IsSameAsPNGFile(bitmap,
+                                    test_data_dir.AppendASCII("green.png")));
+}
+
+}  // namespace
+}  // namespace cc