Added an application framework for demos. This framework currently only suppo...

Added an application framework for demos. This framework currently only supports stand-alone apps. It will be extended to support trusted pepper and untrusted nacl apps.
BUG=26099
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=35758

[vangelis, 2010-01-07 19:15:33]

some nits

http://codereview.chromium.org/525083/diff/1/6
File gpu/demos/app_framework/application.h (right):

http://codereview.chromium.org/525083/diff/1/6#newcode32
gpu/demos/app_framework/application.h:32: // Returns the size of rendering
surface.
For consistency with the coding style guide, there typically needs to be a
separate description per method:
// Returns the width of the rendering surface 

// Returns the height of the rendering surface

http://codereview.chromium.org/525083/diff/1/3
File gpu/demos/app_framework/gles2_utils.cc (right):

http://codereview.chromium.org/525083/diff/1/3#newcode14
gpu/demos/app_framework/gles2_utils.cc:14: GLuint LoadShader(GLenum type, const
char* shader_src) {
Is there a good reason why these methods are implemented as part of the
framework?  Shouldn't they be in the demos' common directory?

[alokp, 2010-01-07 20:37:04]

http://codereview.chromium.org/525083/diff/1/6
File gpu/demos/app_framework/application.h (right):

http://codereview.chromium.org/525083/diff/1/6#newcode32
gpu/demos/app_framework/application.h:32: // Returns the size of rendering
surface.
On 2010/01/07 19:15:33, vangelis wrote:
> For consistency with the coding style guide, there typically needs to be a
> separate description per method:
> // Returns the width of the rendering surface 
> 
> // Returns the height of the rendering surface
> 
> 

Done.

http://codereview.chromium.org/525083/diff/1/3
File gpu/demos/app_framework/gles2_utils.cc (right):

http://codereview.chromium.org/525083/diff/1/3#newcode14
gpu/demos/app_framework/gles2_utils.cc:14: GLuint LoadShader(GLenum type, const
char* shader_src) {
These are not used so I will just remove this file.

[vangelis, 2010-01-07 23:15:06]

LGTM