Benchmark tool for GPU-accelerated video rendering

media/tools/shader_bench/window.h

+// 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_WINDOW_H_
+#define MEDIA_TOOLS_SHADER_BENCH_WINDOW_H_
+
+#include "gfx/native_widget_types.h"
+
+class Painter;
+class Task;
+
+namespace media {
+
+class Window {
+ public:
+  Window(int width, int height);
+

[scherkus (not reviewing), 2010/11/30 23:36:10]

dtor?

[vrk (LEFT CHROMIUM), 2010/12/01 02:46:11]

Talked about it, don't think it's needed in this case.

+  // Creates and returns a handle to a native window of the given dimensions.
+  gfx::NativeWindow CreateNativeWindow(int width, int height);
+
+  // Returns the NPAPI plugin window handle of the window.
+  gfx::PluginWindowHandle PluginWindow();
+
+  // Kicks off frame painting with the given limit, painter, and
+  // callback to run when painting task is complete.
+  virtual void Start(int limit, Task* done_task, Painter* painter);
+
+  // Called when window is expected to paint self.
+  virtual void OnPaint();
+
+  // Main loop for window.
+  virtual void MainLoop();
+
+ protected:

[scherkus (not reviewing), 2010/11/30 23:36:10]

bit confused... I don't think anyone inherits from Window, right?  can we make
these private?

[vrk (LEFT CHROMIUM), 2010/12/01 02:46:11]

Yup, sorry! When I first wrote the class I was going to have the OS-specific
Windows inherit from Window, but then I saw the gpu/demos/ example and changed
my mind, then blindly kept things protected. Also weirdly kept the above methods
virutal haha, no wonder people were confused. Changed both!

+  // Task to run when frame painting is completed. Will be deleted after
+  // running.
+  Task* done_task_;
+
+  // Reference to painter Window uses to paint frames.
+  Painter* painter_;
+
+  // Number of frames to paint before closing the window.
+  int limit_;
+
+  // Number of frames currently painted.
+  int count_;
+
+  // True if the window is painting video frames to the screen, false otherwise.
+  bool running_;
+
+  // This window's native handle.
+  gfx::NativeWindow window_handle_;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(Window);
+};
+
+}  // namespace media
+
+#endif  // MEDIA_TOOLS_SHADER_BENCH_WINDOW_H_