Revert of Enabling `gn check` on webrtc/test

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

[mbonadei, 2017-06-01 11:25:33]

The CQ bit was checked by mbonadei@webrtc.org

[mbonadei, 2017-06-01 11:25:33]

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

[commit-bot: I haz the power, 2017-06-01 11:25:37]

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:25:40]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496316333154700, "parent_rev":
"11c7b703d782f7339bfe127390b4986e8667d7d2", "commit_rev":
"1a6f143d077aec85d919c516b4f74ada8b8c21a8"}

[commit-bot: I haz the power, 2017-06-01 11:25:43]

Description was changed from

==========
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
==========

to

==========
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...
==========

[commit-bot: I haz the power, 2017-06-01 11:25:44]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/1a6f143d077aec85d919c516b...

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

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.webrtc.org/2918793002/ by mbonadei@webrtc.org.

The reason for reverting is: Fixing and re-landing..

[kjellander_webrtc, 2017-06-01 11:28:53]

so close, but then a trybot race condition on turning this on hits us :P
lgtm