Initial checkin of the compositor_model_bench tool, which simulates the GPU demands of Chromium's...

gpu/tools/compositor_model_bench/forward_render_model.cc

Index: gpu/tools/compositor_model_bench/forward_render_model.cc
===================================================================
--- gpu/tools/compositor_model_bench/forward_render_model.cc	(revision 0)
+++ gpu/tools/compositor_model_bench/forward_render_model.cc	(revision 0)
@@ -0,0 +1,76 @@
+// Copyright (c) 2011 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 "gpu/tools/compositor_model_bench/forward_render_model.h"
+
+#include <cstdlib>
+#include <vector>
+
+#include "gpu/tools/compositor_model_bench/render_model_utils.h"
+
+using std::vector;
+
+class ForwardRenderNodeVisitor : public RenderNodeVisitor {
+ public:
+  ForwardRenderNodeVisitor() {}
+
+  virtual void BeginVisitRenderNode(RenderNode* v) OVERRIDE {
+    NOTREACHED();
+  }
+
+  virtual void BeginVisitCCNode(CCNode* v) OVERRIDE {
+    if (!v->drawsContent)
+      return;
+    ConfigAndActivateShaderForNode(v);
+    DrawQuad(v->width, v->height);
+  }
+
+  virtual void BeginVisitContentLayerNode(ContentLayerNode* l) OVERRIDE {
+    if (!l->drawsContent)
+      return;
+    ConfigAndActivateShaderForTiling(l);
+    // Now that we capture root layer tiles, a layer without tiles
+    // should not get drawn.
+    if (l->tiles.size()) {
+      typedef vector<Tile>::iterator tile_itr;
+      for (tile_itr i = l->tiles.begin(); i != l->tiles.end(); ++i) {
+        DrawTileQuad(i->texID, i->x, i->y);
+      }
+    } else {
+      NOTREACHED();
+    }
+  }
+};
+
+ForwardRenderSimulator::ForwardRenderSimulator(RenderNode* root,
+                                               int window_width,
+                                               int window_height)
+    : RenderModelSimulator(root),
+      visitor_(NULL) {
+  textures_.reset(new TextureGenerator(root));
+  visitor_ = new ForwardRenderNodeVisitor();
+  glViewport(0, 0, window_width, window_height);
+  glDisable(GL_DEPTH_TEST);
+  glDisable(GL_CULL_FACE);
+  glEnable(GL_BLEND);
+  glBlendFunc(GL_SRC_ALPHA, GL_ONE_MINUS_SRC_ALPHA);
+}
+
+ForwardRenderSimulator::~ForwardRenderSimulator() OVERRIDE {
+  delete visitor_;

[piman, 2011/08/26 02:24:16]

use scoped_ptr instead?

[joshtrask, 2011/08/26 04:12:35]

I had been doing so at one point, but it occurred to me today that putting
visitor_ in the instance variables means I was using a forward declaration to
specialize the scoped_ptr template, and so I believe scoped_ptr's deletion
behavior is undefined. I could separate the ForwardRenderNodeVisitor into a
different file so that it's available in ForwardRenderModel.h, but I'm not too
keen on separating the visitor from the rest of the render model simulator. What
do you think?

+}
+
+void ForwardRenderSimulator::Update() OVERRIDE {
+  glClearColor(0, 0, 1, 1);
+  glColorMask(true, true, true, true);
+  glClear(GL_COLOR_BUFFER_BIT);
+  glColorMask(true, true, true, false);
+  BeginFrame(10, 11);

[piman, 2011/08/26 02:24:16]

I was initially confused about the magic "10" and "11". I think they don't have
to leak out of shaders.cc (see comments there).

+  if (visitor_)

[piman, 2011/08/26 02:24:16]

visitor_ can't be NULL, right? You should be able to skip this test

[joshtrask, 2011/08/26 04:29:04]

Is that true? What about std::bad_alloc?

On 2011/08/26 02:24:16, piman wrote:

[joshtrask, 2011/08/26 04:30:07]

Well, I suppose if that was going to give me a problem, it would've been before
I got this far...

On 2011/08/26 04:29:04, joshtrask wrote:

+    root_->Accept(visitor_);
+}
+
+void ForwardRenderSimulator::Resize(int width, int height) OVERRIDE {
+  glViewport(0, 0, width, height);
+}