Added GPU unit tests to win, mac and linux builds. They aren't running yet....

Added GPU unit tests to win, mac and linux builds. They aren't running yet.
TEST=trybots
BUG=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=43013

[alokp, 2010-03-29 22:08:11]

lgtm

http://codereview.chromium.org/1577001/diff/1/3
File build/all.gyp (right):

http://codereview.chromium.org/1577001/diff/1/3#newcode96
build/all.gyp:96: '../gpu/demos/demos.gyp:*',
could you also move gpu/demos/demos.gyp.

http://codereview.chromium.org/1577001/diff/1/2
File gpu/gpu.gyp (right):

http://codereview.chromium.org/1577001/diff/1/2#newcode314
gpu/gpu.gyp:314: 'target_name': 'gpu_tests',
gpu_unittests may be a better name.

http://codereview.chromium.org/1577001/diff/1/2#newcode317
gpu/gpu.gyp:317: 'command_buffer/common',
I do not think this is necessary as long as your target is dependent on
command_buffer_common.

[apatrick_chromium, 2010-03-29 22:24:19]

http://codereview.chromium.org/1577001/diff/1/3
File build/all.gyp (right):

http://codereview.chromium.org/1577001/diff/1/3#newcode96
build/all.gyp:96: '../gpu/demos/demos.gyp:*',
On 2010/03/29 22:08:11, alokp wrote:
> could you also move gpu/demos/demos.gyp.

Sure

http://codereview.chromium.org/1577001/diff/1/2
File gpu/gpu.gyp (right):

http://codereview.chromium.org/1577001/diff/1/2#newcode314
gpu/gpu.gyp:314: 'target_name': 'gpu_tests',
On 2010/03/29 22:08:11, alokp wrote:
> gpu_unittests may be a better name.

Agreed. This seems to be more consistent with Chrome conventions though.

http://codereview.chromium.org/1577001/diff/1/2#newcode317
gpu/gpu.gyp:317: 'command_buffer/common',
On 2010/03/29 22:08:11, alokp wrote:
> I do not think this is necessary as long as your target is dependent on
> command_buffer_common.

Done.

[alokp, 2010-03-29 22:28:06]

http://codereview.chromium.org/1577001/diff/1/2
File gpu/gpu.gyp (right):

http://codereview.chromium.org/1577001/diff/1/2#newcode317
gpu/gpu.gyp:317: 'command_buffer/common',
really? I see a lot of _unittests targets in all.gyp. Not a big deal though.

[apatrick_chromium, 2010-03-29 22:38:23]

Okay I'll change it.