GitLab

This reverts commit c9d067e0.

Reason for revert: appears to be responsible for failures in external/wpt/webrtc/idlharness.https.window.html on: 
 - https://ci.chromium.org/p/chromium/builders/ci/Linux%20Tests starting with https://ci.chromium.org/p/chromium/builders/ci/Linux%20Tests/80197
 - https://ci.chromium.org/p/chromium/builders/ci/Win10%20Tests%20x64 starting with https://ci.chromium.org/p/chromium/builders/ci/Win10%20Tests%20x64/38821

Original change's description:
> Reland "Add datachannel.maxPacketLifeTime"
> 
> This reverts commit 49d46905.
> 
> Reason for revert: Blame not credible - listed failures seem to be all MSAN timeouts.
> 
> All they had in common with this CL is that the tests used the "idlharness" tool.
> 
> Original change's description:
> > Revert "Add datachannel.maxPacketLifeTime"
> >
> > This reverts commit e557c49c.
> >
> > Reason for revert: I suspect this CL causes the following test failures:
> > external/wpt/xhr/idlharness.any.html
> > external/wpt/storage/idlharness.https.any.html
> > external/wpt/background-fetch/idlharness.https.any.serviceworker.html
> >
> > The first failed build: https://ci.chromium.org/p/chromium/builders/ci/WebKit%20Linux%20MSAN/938
> >
> > Original change's description:
> > > Add datachannel.maxPacketLifeTime
> > >
> > > Delete nonstandard attribute maxRetransmitTime, and make both
> > > maxPacketLifeTime and maxRetransmits nullable.
> > > (Usage of both attributes has been measured to be negligible.)
> > >
> > > Bug: 696681
> > > Change-Id: If37e38114d4718b868413ea8ced9527c88510c15
> > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1710557
> > > Commit-Queue: Harald Alvestrand <hta@chromium.org>
> > > Reviewed-by: Florent Castelli <orphis@chromium.org>
> > > Reviewed-by: Philip Jägenstedt <foolip@chromium.org>
> > > Reviewed-by: Steve Anton <steveanton@chromium.org>
> > > Cr-Commit-Position: refs/heads/master@{#679965}
> >
> Bug: 696681
> 
> Change-Id: Ic6264584cb4184f1c5acb1049fa5eb2312b41bdb
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1715508
> Reviewed-by: Philip Jägenstedt <foolip@chromium.org>
> Reviewed-by: Florent Castelli <orphis@chromium.org>
> Reviewed-by: Steve Anton <steveanton@chromium.org>
> Commit-Queue: Harald Alvestrand <hta@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#680916}

TBR=hta@chromium.org,foolip@chromium.org,orphis@chromium.org,steveanton@chromium.org

Change-Id: I5e5cb6425eeba84cef20313672042c8a0a7893c2
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 696681
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1719574Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: John Budorick <jbudorick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#681051}