GN: Create target for video_capture_tests on Linux.

GN: Create target for video_capture_tests on Linux.

BUG=webrtc:6042
Committed: https://crrev.com/171c7fef0a8b7a1598ac87e0349e210372cb974c
Cr-Commit-Position: refs/heads/master@{#13638}

[sakal, 2016-08-03 14:27:16]

The CQ bit was checked by sakal@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-03 14:27:21]

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

[sakal, 2016-08-03 14:27:50]

Description was changed from

==========
GN: Create target for video_capture_tests on Linux.

BUG=webrtc:6042
==========

to

==========
GN: Create target for video_capture_tests on Linux.

BUG=webrtc:6042
==========

[sakal, 2016-08-03 14:27:50]

sakal@webrtc.org changed reviewers:
+ mflodman@webrtc.org

[sakal, 2016-08-03 14:28:27]

Magnus, PTAL.

[commit-bot: I haz the power, 2016-08-03 14:37:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-03 14:37:02]

Dry run: Try jobs failed on following builders:
  android_arm64_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/android_arm64_rel/build...)

[mflodman, 2016-08-04 09:51:04]

One comment, then LGTM.

https://codereview.webrtc.org/2207463004/diff/1/webrtc/modules/video_capture/...
File webrtc/modules/video_capture/BUILD.gn (right):

https://codereview.webrtc.org/2207463004/diff/1/webrtc/modules/video_capture/...
webrtc/modules/video_capture/BUILD.gn:169: if (is_linux) {
I thikn we need to also add "&& rtc_include_tests" to the statement.

[sakal, 2016-08-04 12:35:11]

The CQ bit was checked by sakal@webrtc.org

[sakal, 2016-08-04 12:35:11]

The patchset sent to the CQ was uploaded after l-g-t-m from mflodman@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2207463004/#ps20001 (title:
"Changes according to comments")

[commit-bot: I haz the power, 2016-08-04 12:35:15]

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

[sakal, 2016-08-04 12:35:59]

https://codereview.webrtc.org/2207463004/diff/1/webrtc/modules/video_capture/...
File webrtc/modules/video_capture/BUILD.gn (right):

https://codereview.webrtc.org/2207463004/diff/1/webrtc/modules/video_capture/...
webrtc/modules/video_capture/BUILD.gn:169: if (is_linux) {
On 2016/08/04 09:51:04, mflodman_OOO wrote:
> I thikn we need to also add "&& rtc_include_tests" to the statement.

Done.

[commit-bot: I haz the power, 2016-08-04 13:29:28]

Description was changed from

==========
GN: Create target for video_capture_tests on Linux.

BUG=webrtc:6042
==========

to

==========
GN: Create target for video_capture_tests on Linux.

BUG=webrtc:6042
==========

[commit-bot: I haz the power, 2016-08-04 13:29:29]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-04 13:29:35]

Description was changed from

==========
GN: Create target for video_capture_tests on Linux.

BUG=webrtc:6042
==========

to

==========
GN: Create target for video_capture_tests on Linux.

BUG=webrtc:6042

Committed: https://crrev.com/171c7fef0a8b7a1598ac87e0349e210372cb974c
Cr-Commit-Position: refs/heads/master@{#13638}
==========

[commit-bot: I haz the power, 2016-08-04 13:29:36]

Patchset 2 (id:??) landed as
https://crrev.com/171c7fef0a8b7a1598ac87e0349e210372cb974c
Cr-Commit-Position: refs/heads/master@{#13638}

[guidou, 2016-08-04 13:48:08]

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

The reason for reverting is: Breaks bots. 
See
https://build.chromium.org/p/chromium.webrtc.fyi/builders/Android%20Builder%2....