Clean up some HTTP/2 push related SpdyNetworkTransactionTests.

Remove SpdyNetworkTransactionTest.ServerPushBeforeHeaders, because this
case is covered by SpdyNetworkTransactionPushTest.*/0.

Remove SpdyNetworkTransactionTest.ServerPushInvalidCrossOrigin, because
this case is covered by
SpdyNetworkTransactionTest.ServerPushCrossOriginCorrectness.

Change ServerPushCrossOriginCorrectness into a parametrized subclass,
just like SpdyNetworkTransactionPushTest.  For clarity, rename
SpdyNetworkTransactionPushTest to SpdyNetworkTransactionPushHeaderTest,
ServerPushCrossOriginCorrectness to SpdyNetworkTransactionPushUrlTest,
and use PushHeaderTestParams and PushHeaderUrlParams as test data struct
name, respectively.

This will make it much easier to add tests for an upcoming histogram.

Bug: 831536
Change-Id: I298fbd9a2dd4b6567b8cb597fcb704b956819218
Reviewed-on: https://chromium-review.googlesource.com/1055751
Commit-Queue: Bence Béky <bnc@chromium.org>
Reviewed-by: Ryan Hamilton <rch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#558173}
1 file changed