Adding in most of the unit tests.

Adding in most of the unit tests.  This adds 406 of our unit tests
into the gyp build.  21 of them fail, but that is only because they don't
have test input yet -- I haven't added the build code that copies
the test inputs into the build dir yet.

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

[vangelis, 2009-07-01 17:03:23]

Don't know much about GYP but changes look reasonable.  Just some comments
inline on things that look a bit strange.

http://codereview.chromium.org/147129/diff/2009/2012
File plugin/plugin.gyp (right):

http://codereview.chromium.org/147129/diff/2009/2012#newcode111
Line 111: '../../<(cgdir)/lib/cg.lib',
Do we need the Cg library dependencies here? The plugin shouldn't require Cg
unless we're doing the GL build.

http://codereview.chromium.org/147129/diff/2009/2012#newcode122
Line 122: '../breakpad/breakpad.gyp:o3dBreakpad',
won't the breakpad dependency be covered by the clause above this one ('OS ==
"win"') ?

http://codereview.chromium.org/147129/diff/2009/2013
File tests/build.scons (right):

http://codereview.chromium.org/147129/diff/2009/2013#newcode4
Line 4: # Redistribution and use in source and binary forms, with or without
I don't see any diffs in this file.

[Greg Spencer, 2009-07-01 19:25:13]

http://codereview.chromium.org/147129/diff/2009/2012
File plugin/plugin.gyp (right):

http://codereview.chromium.org/147129/diff/2009/2012#newcode111
Line 111: '../../<(cgdir)/lib/cg.lib',
On 2009/07/01 17:03:23, vangelis wrote:
> Do we need the Cg library dependencies here? The plugin shouldn't require Cg
> unless we're doing the GL build.

You're right, removed it.  [This was a leftover from me copy-pasting the
converter .gyp file.]

http://codereview.chromium.org/147129/diff/2009/2012#newcode122
Line 122: '../breakpad/breakpad.gyp:o3dBreakpad',
On 2009/07/01 17:03:23, vangelis wrote:
> won't the breakpad dependency be covered by the clause above this one ('OS ==
> "win"') ? 

Yes.  Removed.

http://codereview.chromium.org/147129/diff/2009/2013
File tests/build.scons (right):

http://codereview.chromium.org/147129/diff/2009/2013#newcode4
Line 4: # Redistribution and use in source and binary forms, with or without
On 2009/07/01 17:03:23, vangelis wrote:
> I don't see any diffs in this file.  

There were some extra carriage return chracters on six lines starting at line
153.  I just fixed them to have proper line endings.
I guess rietveld diff tool doesn't display line ending differences.  Look at the
unified diff.

[vangelis, 2009-07-01 21:48:47]

LGTM