Enable HTTP/2 to use a LIFO scheduler to schedule write.

Enable HTTP/2 to use a LIFO scheduler to schedule write.

This CL lands server change 152524616 by yasong.

BUG=488484
Review-Url: https://codereview.chromium.org/2844963005
Cr-Commit-Position: refs/heads/master@{#467971}
Committed: https://chromium.googlesource.com/chromium/src/+/c443c00c0c9e10b8ced1064d7da18c3a9668cac2

[yasong, 2017-04-27 21:51:14]

The CQ bit was checked by yasong@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-27 21:52:03]

Dry run: 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-04-28 01:36:21]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-28 01:36:21]

Dry run: This issue passed the CQ dry run.

[yasong, 2017-04-28 12:36:17]

Description was changed from

==========
Enable HTTP/2 to use a LIFO scheduler to schedule write.

This CL lands server change 152524616 by yasong.

BUG=488484
==========

to

==========
Enable HTTP/2 to use a LIFO scheduler to schedule write.

This CL lands server change 152524616 by yasong.

BUG=488484
==========

[yasong, 2017-04-28 12:36:17]

yasong@chromium.org changed reviewers:
+ bnc@chromium.org

[yasong, 2017-04-28 12:36:30]

On 2017/04/28 12:36:17, yasong wrote:
> mailto:yasong@chromium.org changed reviewers:
> + mailto:bnc@chromium.org

Ready for review. Thanks!

[Bence, 2017-04-28 12:49:02]

LGTM, thank you.

[Bence, 2017-04-28 12:49:07]

The CQ bit was checked by bnc@chromium.org

[commit-bot: I haz the power, 2017-04-28 12:49:39]

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-04-28 12:54:21]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493383747023860, "parent_rev":
"b62c659f2d62011b43e5930f07ac6bb57254331c", "commit_rev":
"c443c00c0c9e10b8ced1064d7da18c3a9668cac2"}

[commit-bot: I haz the power, 2017-04-28 12:54:32]

Description was changed from

==========
Enable HTTP/2 to use a LIFO scheduler to schedule write.

This CL lands server change 152524616 by yasong.

BUG=488484
==========

to

==========
Enable HTTP/2 to use a LIFO scheduler to schedule write.

This CL lands server change 152524616 by yasong.

BUG=488484

Review-Url: https://codereview.chromium.org/2844963005
Cr-Commit-Position: refs/heads/master@{#467971}
Committed:
https://chromium.googlesource.com/chromium/src/+/c443c00c0c9e10b8ced1064d7da1...
==========

[commit-bot: I haz the power, 2017-04-28 12:54:33]

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

[findit-for-me, 2017-04-28 16:07:42]

Findit(https://goo.gl/kROfz5) identified this CL at revision 467971 as the
culprit for
failures in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3Itb...

[sky, 2017-04-28 22:46:41]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2849983002/ by sky@chromium.org.

The reason for reverting is: Reverting in hopes of fixing net_unittests.
Specifically NSS/ClientCertStoreTest/0.CertAuthorityFiltering . See 716630..

[sky, 2017-04-29 15:16:30]

On 2017/04/28 22:46:41, sky wrote:
> A revert of this CL (patchset #1 id:1) has been created in
> https://codereview.chromium.org/2849983002/ by mailto:sky@chromium.org.
> 
> The reason for reverting is: Reverting in hopes of fixing net_unittests.
> Specifically NSS/ClientCertStoreTest/0.CertAuthorityFiltering . See 716630..

This appears to have been the culprit as the last 20 or so runs looks good on
the two linux bots that were failing.