Benchmark tool for GPU-accelerated video rendering

media/tools/shader_bench/gl_painter.h

Index: media/tools/shader_bench/gl_painter.h
diff --git a/media/tools/shader_bench/gl_painter.h b/media/tools/shader_bench/gl_painter.h
new file mode 100644
index 0000000000000000000000000000000000000000..7f5d7abc11ee7afd9af6e380169cd94eda5e4636
--- /dev/null
+++ b/media/tools/shader_bench/gl_painter.h
@@ -0,0 +1,26 @@
+// Copyright (c) 2010 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.
+
+#ifndef MEDIA_TOOLS_SHADER_BENCH_GL_PAINTER_H_
+#define MEDIA_TOOLS_SHADER_BENCH_GL_PAINTER_H_
+
+#include "app/gfx/gl/gl_context.h"
+#include "base/scoped_ptr.h"
+#include "media/base/video_frame.h"
+#include "media/tools/shader_bench/painter.h"
+
+class GLPainter : public Painter {

[Alpha Left Google, 2010/11/15 20:43:51]

Looking into Painter I found it's all related to GL, is it possible that you
merge two classes?

+ public:
+  GLPainter();
+  virtual ~GLPainter();
+  virtual void Initialize(int width, int height);
+  virtual void Paint(scoped_refptr<media::VideoFrame> video_frame);
+  virtual void SetGLContext(gfx::GLContext* context);

[Alpha Left Google, 2010/11/15 20:43:51]

Painter class also has a lot of GL stuff in it, maybe assign GLContext in
Initialize() and remove this method?

+ private:
+  gfx::GLContext* context_;
+  GLuint program_id_;
+  GLuint textures_[3];
+};
+
+#endif  // MEDIA_TOOLS_SHADER_BENCH_GL_PAINTER_H_