Replaced funky state machines with classes. This is in preparation of doing the same to gl_Bench.

Replaced teartest state machines with classes.  Fixed/commented out GLES2 code.

[piman, 2010-04-30 19:44:02]

http://codereview.chromium.org/1732030/diff/1/3
File client/deps/glbench/src/teartest.cc (right):

http://codereview.chromium.org/1732030/diff/1/3#newcode30
client/deps/glbench/src/teartest.cc:30: TestRegistry()->insert(std::pair<const
char*, Test*>(name, t));
you can use std::make_pair(name, t) instead.
You can also use map[key] = value.

http://codereview.chromium.org/1732030/diff/1/4
File client/deps/glbench/src/teartest.h (right):

http://codereview.chromium.org/1732030/diff/1/4#newcode12
client/deps/glbench/src/teartest.h:12: public:
you need a virtual destructor here.

http://codereview.chromium.org/1732030/diff/1/4#newcode25
client/deps/glbench/src/teartest.h:25: TestAdder test_##name(#name, new
test_cls())
so, using static initialization is frowned upon by the style guide.
Can you change it into an init function that will add all the tests to the map ?

[Alexey Marinichev, 2010-04-30 23:20:52]

Also #ifdefed out stuff so that it compiles with GLES2/gl2.h.

http://codereview.chromium.org/1732030/diff/1/4
File client/deps/glbench/src/teartest.h (right):

http://codereview.chromium.org/1732030/diff/1/4#newcode12
client/deps/glbench/src/teartest.h:12: public:
On 2010/04/30 19:44:02, piman wrote:
> you need a virtual destructor here.

Done.

http://codereview.chromium.org/1732030/diff/1/4#newcode25
client/deps/glbench/src/teartest.h:25: TestAdder test_##name(#name, new
test_cls())
On 2010/04/30 19:44:02, piman wrote:
> so, using static initialization is frowned upon by the style guide.
> Can you change it into an init function that will add all the tests to the map
?

Done.

[piman, 2010-05-01 00:12:09]

LGTM