Revert of Server push cancellation: add a finch trial parameter

Revert of Server push cancellation: add a finch trial parameter (patchset #15 id:300001 of https://codereview.chromium.org/2692813002/ )

Reason for revert:
Broke net_unittests on Win7 (dbg) bot:

https://uberchromegw.corp.google.com/i/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29

https://uberchromegw.corp.google.com/i/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/57830

Original issue's description:
> Server push cancellation: add a finch trial parameter
> "enable_server_push_cancellation" to control the feature of server push
> cancellation. Create a HttpCacheLookupManager in HttpCache's constructor
> and set it to HttpNetworkSession which will set it thru reaching
> QuicChromiumClientSession and SpdySession.
> 
> BUG=232040
> 
> Review-Url: https://codereview.chromium.org/2692813002
> Cr-Original-Commit-Position: refs/heads/master@{#454457}
> Committed: https://chromium.googlesource.com/chromium/src/+/c83d602cac3c9a7a7ef554d172b7a8cd805f13e8
> Review-Url: https://codereview.chromium.org/2692813002
> Cr-Commit-Position: refs/heads/master@{#454679}
> Committed: https://chromium.googlesource.com/chromium/src/+/f04bd2c84154b59a3d870993051df4be0b13964f

TBR=rch@chromium.org,ckrasic@chromium.org,zhongyi@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=232040
Review-Url: https://codereview.chromium.org/2728183003
Cr-Commit-Position: refs/heads/master@{#454706}
Committed: https://chromium.googlesource.com/chromium/src/+/46a595cd9b30990f40400a81e9b99483f533cafa

[sclittle, 2017-03-03 23:22:52]

The CQ bit was checked by sclittle@chromium.org

[sclittle, 2017-03-03 23:22:53]

Created Revert of Server push cancellation: add a finch trial parameter

[commit-bot: I haz the power, 2017-03-03 23:23:07]

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

[commit-bot: I haz the power, 2017-03-03 23:24:05]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488583372856030, "parent_rev":
"9c3ae5fc7b7a3a9395b882f80661f209f3bfc4ef", "commit_rev":
"46a595cd9b30990f40400a81e9b99483f533cafa"}

[commit-bot: I haz the power, 2017-03-03 23:24:49]

Description was changed from

==========
Revert of Server push cancellation: add a finch trial parameter (patchset #15
id:300001 of https://codereview.chromium.org/2692813002/ )

Reason for revert:
Broke net_unittests on Win7 (dbg) bot:

https://uberchromegw.corp.google.com/i/chromium.win/builders/Win7%20Tests%20%...

https://uberchromegw.corp.google.com/i/chromium.win/builders/Win7%20Tests%20%...

Original issue's description:
> Server push cancellation: add a finch trial parameter
> "enable_server_push_cancellation" to control the feature of server push
> cancellation. Create a HttpCacheLookupManager in HttpCache's constructor
> and set it to HttpNetworkSession which will set it thru reaching
> QuicChromiumClientSession and SpdySession.
> 
> BUG=232040
> 
> Review-Url: https://codereview.chromium.org/2692813002
> Cr-Original-Commit-Position: refs/heads/master@{#454457}
> Committed:
https://chromium.googlesource.com/chromium/src/+/c83d602cac3c9a7a7ef554d172b7...
> Review-Url: https://codereview.chromium.org/2692813002
> Cr-Commit-Position: refs/heads/master@{#454679}
> Committed:
https://chromium.googlesource.com/chromium/src/+/f04bd2c84154b59a3d870993051d...

TBR=rch@chromium.org,ckrasic@chromium.org,zhongyi@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=232040
==========

to

==========
Revert of Server push cancellation: add a finch trial parameter (patchset #15
id:300001 of https://codereview.chromium.org/2692813002/ )

Reason for revert:
Broke net_unittests on Win7 (dbg) bot:

https://uberchromegw.corp.google.com/i/chromium.win/builders/Win7%20Tests%20%...

https://uberchromegw.corp.google.com/i/chromium.win/builders/Win7%20Tests%20%...

Original issue's description:
> Server push cancellation: add a finch trial parameter
> "enable_server_push_cancellation" to control the feature of server push
> cancellation. Create a HttpCacheLookupManager in HttpCache's constructor
> and set it to HttpNetworkSession which will set it thru reaching
> QuicChromiumClientSession and SpdySession.
> 
> BUG=232040
> 
> Review-Url: https://codereview.chromium.org/2692813002
> Cr-Original-Commit-Position: refs/heads/master@{#454457}
> Committed:
https://chromium.googlesource.com/chromium/src/+/c83d602cac3c9a7a7ef554d172b7...
> Review-Url: https://codereview.chromium.org/2692813002
> Cr-Commit-Position: refs/heads/master@{#454679}
> Committed:
https://chromium.googlesource.com/chromium/src/+/f04bd2c84154b59a3d870993051d...

TBR=rch@chromium.org,ckrasic@chromium.org,zhongyi@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=232040

Review-Url: https://codereview.chromium.org/2728183003
Cr-Commit-Position: refs/heads/master@{#454706}
Committed:
https://chromium.googlesource.com/chromium/src/+/46a595cd9b30990f40400a81e9b9...
==========

[commit-bot: I haz the power, 2017-03-03 23:24:50]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/46a595cd9b30990f40400a81e9b9...

[Ryan Hamilton, 2017-03-04 04:17:53]

On 2017/03/03 23:24:50, commit-bot: I haz the power wrote:
> Committed patchset #1 (id:1) as
>
https://chromium.googlesource.com/chromium/src/+/46a595cd9b30990f40400a81e9b9...

How can we target a win7 bot in order to verify this?

[sclittle, 2017-03-06 18:28:20]

On 2017/03/04 04:17:53, Ryan Hamilton wrote:
> On 2017/03/03 23:24:50, commit-bot: I haz the power wrote:
> > Committed patchset #1 (id:1) as
> >
>
https://chromium.googlesource.com/chromium/src/+/46a595cd9b30990f40400a81e9b9...
> 
> How can we target a win7 bot in order to verify this?

Looking at the answer from
https://groups.google.com/a/chromium.org/forum/#!topic/infra-dev/bIeyq_F0pHU,
and
https://cs.chromium.org/chromium/build/scripts/slave/recipe_modules/chromium_...:

It looks like the "win_chromium_dbg_ng" trybot corresponds to the "Win7 Tests
(dbg)(1)" builder that failed.