Fix tools_unittests in GN.

Fix tools_unittests in GN.

tools_unittests were failing when generated by GN and running on a device.

BUG=webrtc:5949
NOTRY=True

Committed: https://crrev.com/4556b450765116072f9700e0619c86b0f9b06588
Cr-Commit-Position: refs/heads/master@{#13714}

[ehmaldonado_webrtc, 2016-08-10 13:32:18]

Description was changed from

==========
IN PROGRESS: tools unittests in GN.

BUG=
==========

to

==========
tools_unittests were failing when generated by GN and running on a device.

BUG=
==========

[ehmaldonado_webrtc, 2016-08-10 13:32:19]

ehmaldonado@webrtc.org changed reviewers:
+ kjellander@webrtc.org

[ehmaldonado_webrtc, 2016-08-10 13:34:36]

This makes the tests run.
I'm curious why there is not something similar in GYP, and how does GN figures
out the data dependencies on desktop?

[kjellander_webrtc, 2016-08-10 14:36:29]

On 2016/08/10 13:34:36, ehmaldonado_webrtc wrote:
> This makes the tests run.
> I'm curious why there is not something similar in GYP, 

In GYP these things are spread out to several targets:
https://cs.chromium.org/chromium/src/third_party/webrtc/tools/tools.gyp?rcl=0...
which includes the tools_unittests.isolate which in turn lists the file:
https://cs.chromium.org/chromium/src/third_party/webrtc/tools/tools_unittests...

It's great that with GN it's all kept in one target.

>and how does GN figures out the data dependencies on desktop?

On desktop, the resources/ dir is checked out and
https://cs.chromium.org/chromium/src/third_party/webrtc/test/testsupport/file...
has logic to figure out the root of the checkout so it can find the resources
folder. It requires all tests to use the ResourcePath method though.

[kjellander_webrtc, 2016-08-10 14:37:00]

lgtm (please add NOTRY=True to save some time for this tiny GN-only change)

[ehmaldonado_webrtc, 2016-08-10 14:45:37]

Description was changed from

==========
tools_unittests were failing when generated by GN and running on a device.

BUG=
==========

to

==========
Fix tools_unittests in GN.

tools_unittests were failing when generated by GN and running on a device.

BUG=webrtc:5949
NOTRY=True
==========

[ehmaldonado_webrtc, 2016-08-10 14:46:00]

The CQ bit was checked by ehmaldonado@webrtc.org

[commit-bot: I haz the power, 2016-08-10 14:46:04]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-08-10 14:47:17]

Description was changed from

==========
Fix tools_unittests in GN.

tools_unittests were failing when generated by GN and running on a device.

BUG=webrtc:5949
NOTRY=True
==========

to

==========
Fix tools_unittests in GN.

tools_unittests were failing when generated by GN and running on a device.

BUG=webrtc:5949
NOTRY=True
==========

[commit-bot: I haz the power, 2016-08-10 14:47:18]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-10 14:47:27]

Description was changed from

==========
Fix tools_unittests in GN.

tools_unittests were failing when generated by GN and running on a device.

BUG=webrtc:5949
NOTRY=True
==========

to

==========
Fix tools_unittests in GN.

tools_unittests were failing when generated by GN and running on a device.

BUG=webrtc:5949
NOTRY=True

Committed: https://crrev.com/4556b450765116072f9700e0619c86b0f9b06588
Cr-Commit-Position: refs/heads/master@{#13714}
==========

[commit-bot: I haz the power, 2016-08-10 14:47:27]

Patchset 1 (id:??) landed as
https://crrev.com/4556b450765116072f9700e0619c86b0f9b06588
Cr-Commit-Position: refs/heads/master@{#13714}