GN: Use rtc_include_tests variable to exclude tests.

GN: Use rtc_include_tests variable to exclude tests.

This is closer to what the GYP build does, and is more readable.

BUG=webrtc:5949
TBR=phoglund@webrtc.org
NOTRY=True

Committed: https://crrev.com/e72db175568083c97528fcc74c37a35992c3bc44
Cr-Commit-Position: refs/heads/master@{#13054}

[kjellander_webrtc, 2016-06-06 20:52:34]

Description was changed from

==========
GN: Use rtc_include_tests variable to exclude tests.

This is closer to what the GYP build does, and is more readable.

BUG=webrtc:5949
==========

to

==========
GN: Use rtc_include_tests variable to exclude tests.

This is closer to what the GYP build does, and is more readable.

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

[kjellander_webrtc, 2016-06-06 20:52:34]

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

[kjellander_webrtc, 2016-06-06 20:54:36]

Patchset #2 (id:20001) has been deleted

[kjellander_webrtc, 2016-06-06 20:55:03]

Description was changed from

==========
GN: Use rtc_include_tests variable to exclude tests.

This is closer to what the GYP build does, and is more readable.

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

to

==========
GN: Use rtc_include_tests variable to exclude tests.

This is closer to what the GYP build does, and is more readable.

BUG=webrtc:5949
TBR=phoglund@webrtc.org
NOTRY=True
==========

[kjellander_webrtc, 2016-06-06 20:58:22]

The CQ bit was checked by kjellander@webrtc.org

[commit-bot: I haz the power, 2016-06-06 20:58:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2043873003/1

[commit-bot: I haz the power, 2016-06-06 21:00:06]

Description was changed from

==========
GN: Use rtc_include_tests variable to exclude tests.

This is closer to what the GYP build does, and is more readable.

BUG=webrtc:5949
TBR=phoglund@webrtc.org
NOTRY=True
==========

to

==========
GN: Use rtc_include_tests variable to exclude tests.

This is closer to what the GYP build does, and is more readable.

BUG=webrtc:5949
TBR=phoglund@webrtc.org
NOTRY=True
==========

[commit-bot: I haz the power, 2016-06-06 21:00:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-06 21:00:17]

Description was changed from

==========
GN: Use rtc_include_tests variable to exclude tests.

This is closer to what the GYP build does, and is more readable.

BUG=webrtc:5949
TBR=phoglund@webrtc.org
NOTRY=True
==========

to

==========
GN: Use rtc_include_tests variable to exclude tests.

This is closer to what the GYP build does, and is more readable.

BUG=webrtc:5949
TBR=phoglund@webrtc.org
NOTRY=True

Committed: https://crrev.com/e72db175568083c97528fcc74c37a35992c3bc44
Cr-Commit-Position: refs/heads/master@{#13054}
==========

[commit-bot: I haz the power, 2016-06-06 21:00:18]

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

[phoglund, 2016-06-08 07:17:00]

This patch is wrong but I see you noticed it in a later patch :)

https://codereview.webrtc.org/2043873003/diff/1/webrtc/BUILD.gn
File webrtc/BUILD.gn (right):

https://codereview.webrtc.org/2043873003/diff/1/webrtc/BUILD.gn#newcode382
webrtc/BUILD.gn:382: if (!rtc_include_tests) {
Uh oh.