Reland of Enabling `gn check` on webrtc/test

Reland of Enabling `gn check` on webrtc/test (patchset #1 id:1 of https://codereview.webrtc.org/2920763002/ )

Reason for revert:
Fixing and re-landing.

Original issue's description:
> Revert of Enabling `gn check` on webrtc/test (patchset #9 id:160001 of https://codereview.webrtc.org/2911203002/ )
> 
> Reason for revert:
> ERROR at //webrtc/test/testsupport/fileutils_unittest.cc:20:11: Can't include this header from here.
> #include "webrtc/base/checks.h"
>           ^-------------------
> The target:
>   //webrtc/test:fileutils_unittests
> is including a file from the target:
>   //webrtc/base:rtc_base_approved
> 
> It's usually best to depend directly on the destination target.
> In some cases, the destination target is considered a subcomponent
> of an intermediate target. In this case, the intermediate target
> should depend publicly on the destination to forward the ability
> to include headers.
> 
> Dependency chain (there may also be others):
>   //webrtc/test:fileutils_unittests -->
>   //webrtc/test:fileutils --[private]-->
>   //webrtc/base:rtc_base_approved
> 
> 
> Original issue's description:
> > Enabling `gn check` on webrtc/test
> > 
> > BUG=webrtc:6828
> > NOTRY=True
> > 
> > Review-Url: https://codereview.webrtc.org/2911203002
> > Cr-Commit-Position: refs/heads/master@{#18372}
> > Committed: https://chromium.googlesource.com/external/webrtc/+/db5bb404b0f42a7c0a43f882b34ba1325d8cbae2
> 
> TBR=kjellander@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:6828
> 
> Review-Url: https://codereview.webrtc.org/2920763002
> Cr-Commit-Position: refs/heads/master@{#18375}
> Committed: https://chromium.googlesource.com/external/webrtc/+/1a6f143d077aec85d919c516b4f74ada8b8c21a8

TBR=kjellander@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6828
Review-Url: https://codereview.webrtc.org/2918793002
Cr-Commit-Position: refs/heads/master@{#18376}
Committed: https://chromium.googlesource.com/external/webrtc/+/3edccb999cd134d6b10b6e1eead78f0ec58b0015

[mbonadei, 2017-06-01 11:28:37]

Created Reland of Enabling `gn check` on webrtc/test

[kjellander_webrtc, 2017-06-01 11:33:18]

lgtm

make sure you run a few trybots at least.

[mbonadei, 2017-06-01 11:47:12]

The CQ bit was checked by mbonadei@webrtc.org

[commit-bot: I haz the power, 2017-06-01 11:47:17]

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-06-01 11:47:21]

CQ is committing da patch.

Bot data: {"patchset_id": 160001, "attempt_start_ts": 1496317632573920,
"parent_rev": "1a6f143d077aec85d919c516b4f74ada8b8c21a8", "commit_rev":
"3edccb999cd134d6b10b6e1eead78f0ec58b0015"}

[commit-bot: I haz the power, 2017-06-01 11:47:24]

Description was changed from

==========
Reland of Enabling `gn check` on webrtc/test (patchset #1 id:1 of
https://codereview.webrtc.org/2920763002/ )

Reason for revert:
Fixing and re-landing.

Original issue's description:
> Revert of Enabling `gn check` on webrtc/test (patchset #9 id:160001 of
https://codereview.webrtc.org/2911203002/ )
> 
> Reason for revert:
> ERROR at //webrtc/test/testsupport/fileutils_unittest.cc:20:11: Can't include
this header from here.
> #include "webrtc/base/checks.h"
>           ^-------------------
> The target:
>   //webrtc/test:fileutils_unittests
> is including a file from the target:
>   //webrtc/base:rtc_base_approved
> 
> It's usually best to depend directly on the destination target.
> In some cases, the destination target is considered a subcomponent
> of an intermediate target. In this case, the intermediate target
> should depend publicly on the destination to forward the ability
> to include headers.
> 
> Dependency chain (there may also be others):
>   //webrtc/test:fileutils_unittests -->
>   //webrtc/test:fileutils --[private]-->
>   //webrtc/base:rtc_base_approved
> 
> 
> Original issue's description:
> > Enabling `gn check` on webrtc/test
> > 
> > BUG=webrtc:6828
> > NOTRY=True
> > 
> > Review-Url: https://codereview.webrtc.org/2911203002
> > Cr-Commit-Position: refs/heads/master@{#18372}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/db5bb404b0f42a7c0a43f882b...
> 
> TBR=kjellander@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:6828
> 
> Review-Url: https://codereview.webrtc.org/2920763002
> Cr-Commit-Position: refs/heads/master@{#18375}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/1a6f143d077aec85d919c516b...

TBR=kjellander@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6828
==========

to

==========
Reland of Enabling `gn check` on webrtc/test (patchset #1 id:1 of
https://codereview.webrtc.org/2920763002/ )

Reason for revert:
Fixing and re-landing.

Original issue's description:
> Revert of Enabling `gn check` on webrtc/test (patchset #9 id:160001 of
https://codereview.webrtc.org/2911203002/ )
> 
> Reason for revert:
> ERROR at //webrtc/test/testsupport/fileutils_unittest.cc:20:11: Can't include
this header from here.
> #include "webrtc/base/checks.h"
>           ^-------------------
> The target:
>   //webrtc/test:fileutils_unittests
> is including a file from the target:
>   //webrtc/base:rtc_base_approved
> 
> It's usually best to depend directly on the destination target.
> In some cases, the destination target is considered a subcomponent
> of an intermediate target. In this case, the intermediate target
> should depend publicly on the destination to forward the ability
> to include headers.
> 
> Dependency chain (there may also be others):
>   //webrtc/test:fileutils_unittests -->
>   //webrtc/test:fileutils --[private]-->
>   //webrtc/base:rtc_base_approved
> 
> 
> Original issue's description:
> > Enabling `gn check` on webrtc/test
> > 
> > BUG=webrtc:6828
> > NOTRY=True
> > 
> > Review-Url: https://codereview.webrtc.org/2911203002
> > Cr-Commit-Position: refs/heads/master@{#18372}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/db5bb404b0f42a7c0a43f882b...
> 
> TBR=kjellander@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:6828
> 
> Review-Url: https://codereview.webrtc.org/2920763002
> Cr-Commit-Position: refs/heads/master@{#18375}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/1a6f143d077aec85d919c516b...

TBR=kjellander@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6828

Review-Url: https://codereview.webrtc.org/2918793002
Cr-Commit-Position: refs/heads/master@{#18376}
Committed:
https://chromium.googlesource.com/external/webrtc/+/3edccb999cd134d6b10b6e1ee...
==========

[commit-bot: I haz the power, 2017-06-01 11:47:27]

Committed patchset #2 (id:160001) as
https://chromium.googlesource.com/external/webrtc/+/3edccb999cd134d6b10b6e1ee...