Added gyp file for examples. Removed unnecessary functions. Added header file...

Added gyp file for examples. Removed unnecessary functions. Added header files for two examples and changed a few function signatures so they can be compiled as static libraries.
BUG=26099
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=35856

[vangelis, 2010-01-07 19:10:06]

Do you need to modify the readme file to reflect on the additional changes you
made to the third party code? 

Some more comments inline.

http://codereview.chromium.org/527016/diff/3006/2009
File third_party/gles_book_examples/Chapter_2/Hello_Triangle/Hello_Triangle.c
(right):

http://codereview.chromium.org/527016/diff/3006/2009#newcode36
third_party/gles_book_examples/Chapter_2/Hello_Triangle/Hello_Triangle.c:36: //
TODO(alokp): Shaders containing "precision" do not compile.
If you add the TODO you should probably live the line with the precision keyword
in there, just commented out. Otherwise, there's not much context for people
reading the code to figure out what the TODO refers to.

http://codereview.chromium.org/527016/diff/3006/2009#newcode43
third_party/gles_book_examples/Chapter_2/Hello_Triangle/Hello_Triangle.c:43: //
TODO(alokp): Client-side vertex arrays do not work.
Similarly here.  For someone reading the code they won't be able to figure out
why you make this comment.  You could maybe expand to say that the original code
uses client-side vertex arrays that are not currently supported or something
similar.

http://codereview.chromium.org/527016/diff/3006/2019
File
third_party/gles_book_examples/Chapter_8/Simple_VertexShader/Simple_VertexShader.c
(right):

http://codereview.chromium.org/527016/diff/3006/2019#newcode34
third_party/gles_book_examples/Chapter_8/Simple_VertexShader/Simple_VertexShader.c:34:
// TODO(alokp): Shaders containing "precision" do not compile.
Either add back the line that uses the precision keyword and have it commented
out or remove the TODO

http://codereview.chromium.org/527016/diff/3006/2018
File
third_party/gles_book_examples/Chapter_8/Simple_VertexShader/Simple_VertexShader.h
(right):

http://codereview.chromium.org/527016/diff/3006/2018#newcode14
third_party/gles_book_examples/Chapter_8/Simple_VertexShader/Simple_VertexShader.h:14:
//    This is a simple example that draws a rotating cube in perspective
the comment doesn't match the actual contents of the file.

[alokp, 2010-01-07 20:35:58]

http://codereview.chromium.org/527016/diff/3006/2009
File third_party/gles_book_examples/Chapter_2/Hello_Triangle/Hello_Triangle.c
(right):

http://codereview.chromium.org/527016/diff/3006/2009#newcode36
third_party/gles_book_examples/Chapter_2/Hello_Triangle/Hello_Triangle.c:36: //
TODO(alokp): Shaders containing "precision" do not compile.
On 2010/01/07 19:10:06, vangelis wrote:
> If you add the TODO you should probably live the line with the precision
keyword
> in there, just commented out. Otherwise, there's not much context for people
> reading the code to figure out what the TODO refers to.

Done.

http://codereview.chromium.org/527016/diff/3006/2009#newcode43
third_party/gles_book_examples/Chapter_2/Hello_Triangle/Hello_Triangle.c:43: //
TODO(alokp): Client-side vertex arrays do not work.
I removed this TODO because Gregg said we will never support client-side vertex
arrays.

http://codereview.chromium.org/527016/diff/3006/2019
File
third_party/gles_book_examples/Chapter_8/Simple_VertexShader/Simple_VertexShader.c
(right):

http://codereview.chromium.org/527016/diff/3006/2019#newcode34
third_party/gles_book_examples/Chapter_8/Simple_VertexShader/Simple_VertexShader.c:34:
// TODO(alokp): Shaders containing "precision" do not compile.
On 2010/01/07 19:10:06, vangelis wrote:
> Either add back the line that uses the precision keyword and have it commented
> out or remove the TODO

Done.

http://codereview.chromium.org/527016/diff/3006/2018
File
third_party/gles_book_examples/Chapter_8/Simple_VertexShader/Simple_VertexShader.h
(right):

http://codereview.chromium.org/527016/diff/3006/2018#newcode14
third_party/gles_book_examples/Chapter_8/Simple_VertexShader/Simple_VertexShader.h:14:
//    This is a simple example that draws a rotating cube in perspective
what do you mean? I copied the comment from Simple_VertexShader.c.

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

Also modified README.chromium to reflect additional changes.

[vangelis, 2010-01-07 23:22:09]

LGTM

http://codereview.chromium.org/527016/diff/3006/2018
File
third_party/gles_book_examples/Chapter_8/Simple_VertexShader/Simple_VertexShader.h
(right):

http://codereview.chromium.org/527016/diff/3006/2018#newcode14
third_party/gles_book_examples/Chapter_8/Simple_VertexShader/Simple_VertexShader.h:14:
//    This is a simple example that draws a rotating cube in perspective
On 2010/01/07 20:36:00, alokp wrote:
> what do you mean? I copied the comment from Simple_VertexShader.c.

I meant that the comment doesn't really describe the contents of this header. 
It works for the .c file as that's where all the code is for the spinning cube. 
Maybe just remove the comment altogether?

[alokp, 2010-01-07 23:27:11]

http://codereview.chromium.org/527016/diff/3006/2018
File
third_party/gles_book_examples/Chapter_8/Simple_VertexShader/Simple_VertexShader.h
(right):

http://codereview.chromium.org/527016/diff/3006/2018#newcode14
third_party/gles_book_examples/Chapter_8/Simple_VertexShader/Simple_VertexShader.h:14:
//    This is a simple example that draws a rotating cube in perspective
Done. I also removed similar comment from Hello_Triangle.h.