Added infrastructure for adding gtest-based tests. I need to use it for the new preprocessor.

Added infrastructure for adding gtest-based tests. I need to use it for the new preprocessor.
Committed: https://code.google.com/p/angleproject/source/detail?r=1024

[kbr1, 2012-04-09 22:58:04]

Thumbs up for more testing. LGTM, but please check the concerns below.

http://codereview.appspot.com/5988069/diff/1/DEPS
File DEPS (right):

http://codereview.appspot.com/5988069/diff/1/DEPS#newcode6
DEPS:6: "http://googletest.googlecode.com/svn/trunk@573", #release 1.6.0
Please make sure that this addition and the others don't break Chromium builds.
To test, you can patch these changes into the copy of ANGLE in your Chromium
repo, run gclient sync and build Chrome.

http://codereview.appspot.com/5988069/diff/1/tests/build_tests.gyp
File tests/build_tests.gyp (right):

http://codereview.appspot.com/5988069/diff/1/tests/build_tests.gyp#newcode19
tests/build_tests.gyp:19: 'target_name': 'pp_tests',
A more descriptive name like angle_preprocessor_tests might be better.

[Alok Priyadarshi, 2012-04-10 04:14:12]

http://codereview.appspot.com/5988069/diff/1/DEPS
File DEPS (right):

http://codereview.appspot.com/5988069/diff/1/DEPS#newcode6
DEPS:6: "http://googletest.googlecode.com/svn/trunk@573", #release 1.6.0
I do not see how this will affect chrome build. First of all chrome does not
build this target. But most importantly chrome pulls third-party libs with
--ignore-externals, which means this DEPS file is ignored.

On 2012/04/09 22:58:04, kbr1 wrote:
> Please make sure that this addition and the others don't break Chromium
builds.
> To test, you can patch these changes into the copy of ANGLE in your Chromium
> repo, run gclient sync and build Chrome.
>

http://codereview.appspot.com/5988069/diff/1/tests/build_tests.gyp
File tests/build_tests.gyp (right):

http://codereview.appspot.com/5988069/diff/1/tests/build_tests.gyp#newcode19
tests/build_tests.gyp:19: 'target_name': 'pp_tests',
On 2012/04/09 22:58:04, kbr1 wrote:
> A more descriptive name like angle_preprocessor_tests might be better.

Will do.

[kbr1, 2012-04-10 18:26:15]

http://codereview.appspot.com/5988069/diff/1/DEPS
File DEPS (right):

http://codereview.appspot.com/5988069/diff/1/DEPS#newcode6
DEPS:6: "http://googletest.googlecode.com/svn/trunk@573", #release 1.6.0
On 2012/04/10 04:14:12, Alok Priyadarshi wrote:
> I do not see how this will affect chrome build. First of all chrome does not
> build this target. But most importantly chrome pulls third-party libs with
> --ignore-externals, which means this DEPS file is ignored.

That's fine then. Just checking.

[dgkoch, 2012-04-11 16:05:23]

LGTM. 

Maybe you could add a wiki page on how to use this, once it's integrated as
well?

[Alok Priyadarshi, 2012-04-11 16:21:52]

I will. Thanks.
Please note that I have not added a hand-written build file because you really
need to use gclient to pull googletest into third_party.

On 2012/04/11 16:05:23, dgkoch wrote:
> LGTM. 
> 
> Maybe you could add a wiki page on how to use this, once it's integrated as
> well?