GN: Include webrtc/api targets even if rtc_include_tests=false

GN: Include webrtc/api targets even if rtc_include_tests=false

The main purpose with the rtc_include_tests GN variable is to avoid
generating and compiling all the test targets.
Some of our examples have dependencies on the test headers in API,
so therefore this change is relaxing that condition.

BUG=webrtc:6828
NOTRY=True
TBR=ehmaldonado@webrtc.org,

Review-Url: https://codereview.webrtc.org/2725053008
Cr-Commit-Position: refs/heads/master@{#16989}
Committed: https://chromium.googlesource.com/external/webrtc/+/a769ceba656eff1d12df82ef3058306cf948b215

[kjellander_webrtc, 2017-03-03 06:18:38]

Description was changed from

==========
GN: Include webrtc/api targets even if rtc_include_tests=false

The main purpose with rtc_include_tests is to avoid generating
and compiling all the test targets.
Some of our examples have dependencies on the test headers in API,
so therefore this change is relaxing that condition.

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

to

==========
GN: Include webrtc/api targets even if rtc_include_tests=false

The main purpose with rtc_include_tests GN variable is to avoid
generating and compiling all the test targets.
Some of our examples have dependencies on the test headers in API,
so therefore this change is relaxing that condition.

BUG=webrtc:6828
NOTRY=True
TBR=ehmaldonado@webrtc.org,
==========

[kjellander_webrtc, 2017-03-03 06:18:38]

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

[kjellander_webrtc, 2017-03-03 06:18:51]

Description was changed from

==========
GN: Include webrtc/api targets even if rtc_include_tests=false

The main purpose with rtc_include_tests GN variable is to avoid
generating and compiling all the test targets.
Some of our examples have dependencies on the test headers in API,
so therefore this change is relaxing that condition.

BUG=webrtc:6828
NOTRY=True
TBR=ehmaldonado@webrtc.org,
==========

to

==========
GN: Include webrtc/api targets even if rtc_include_tests=false

The main purpose with the rtc_include_tests GN variable is to avoid
generating and compiling all the test targets.
Some of our examples have dependencies on the test headers in API,
so therefore this change is relaxing that condition.

BUG=webrtc:6828
NOTRY=True
TBR=ehmaldonado@webrtc.org,
==========

[kjellander_webrtc, 2017-03-03 06:19:55]

This fixes the breakage in
https://build.chromium.org/p/client.webrtc/builders/Linux%20%28more%20configs...
that happened because of https://codereview.webrtc.org/2714343002/

[kjellander_webrtc, 2017-03-03 06:23:01]

The CQ bit was checked by kjellander@webrtc.org

[commit-bot: I haz the power, 2017-03-03 06:23:09]

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, 2017-03-03 06:25:04]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1488522181824870,
"parent_rev": "dc9200e314a3ca66a26f64eb1870380e715250df", "commit_rev":
"a769ceba656eff1d12df82ef3058306cf948b215"}

[commit-bot: I haz the power, 2017-03-03 06:25:06]

Description was changed from

==========
GN: Include webrtc/api targets even if rtc_include_tests=false

The main purpose with the rtc_include_tests GN variable is to avoid
generating and compiling all the test targets.
Some of our examples have dependencies on the test headers in API,
so therefore this change is relaxing that condition.

BUG=webrtc:6828
NOTRY=True
TBR=ehmaldonado@webrtc.org,
==========

to

==========
GN: Include webrtc/api targets even if rtc_include_tests=false

The main purpose with the rtc_include_tests GN variable is to avoid
generating and compiling all the test targets.
Some of our examples have dependencies on the test headers in API,
so therefore this change is relaxing that condition.

BUG=webrtc:6828
NOTRY=True
TBR=ehmaldonado@webrtc.org,

Review-Url: https://codereview.webrtc.org/2725053008
Cr-Commit-Position: refs/heads/master@{#16989}
Committed:
https://chromium.googlesource.com/external/webrtc/+/a769ceba656eff1d12df82ef3...
==========

[commit-bot: I haz the power, 2017-03-03 06:25:07]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/webrtc/+/a769ceba656eff1d12df82ef3...

[kjellander_webrtc, 2017-03-03 06:26:08]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.webrtc.org/2728073002/ by kjellander@webrtc.org.

The reason for reverting is: Fails Chromium builds: 
b/c/b/linux/src/buildtools/linux64/gn gen //out/Release --check
  -> returned 1
ERROR at //third_party/webrtc/api/BUILD.gn:186:5: Can't load input file.
    "//webrtc/test:test_support",
    ^-------------------------.