GitLab

This reverts commit a78d9786.

Reason for revert:

Findit (https://goo.gl/kROfz5) identified CL at revision 645869 as the
culprit for flakes in the build cycles as shown on:
https://analysis.chromium.org/p/chromium/flake-portal/analysis/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyQwsSDEZsYWtlQ3VscHJpdCIxY2hyb21pdW0vYTc4ZDk3ODZhMWE5OTI1NmRiNmY5YzA2NTZlMWY0MWZhOGY1N2Y5NQw

Sample Failed Build: https://ci.chromium.org/buildbot/chromium.mac/Mac10.10%20Tests/41071

Sample Failed Step: webkit_layout_tests on (none) GPU on Mac on Mac-10.10.5

Sample Flaky Test: virtual/nobinary-for-inspector-protocol/inspector-protocol/heap-profiler/heap-snapshot-with-multiple-retainers.js

Original change's description:
> [DevTools] Internally use binary protocol by default.
> 
> There's now a switch --disable-internal-devtools-binary-protocol.
> 
> Also:
> - Remove the old virtual test suites since the enable flag is gone.
> - Introduce new virtual test suites to test the diable flag.
> 
> There's one test for which behavior changes just a tad (more precision),
> so I'm updating the expectations and also keeping the old state
> for the virtual test suite.
> 
> Change-Id: Iab705aa2b7cfb996a05ce1a7d0804ca20adad009
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544735
> Commit-Queue: Johannes Henkel <johannes@chromium.org>
> Reviewed-by: Pavel Feldman <pfeldman@chromium.org>
> Reviewed-by: Andrey Kosyakov <caseq@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#645869}

Change-Id: I99dc80b0774884819d11d6611c787b9bc485eec6
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546634
Cr-Commit-Position: refs/heads/master@{#646071}