Revert "Reland "Add datachannel.maxPacketLifeTime""

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}

content/renderer/media/webrtc/mock_data_channel_impl.cc

@@ -34,6 +34,18 @@ bool MockDataChannel::reliable() const { return reliable_; }
 
 bool MockDataChannel::ordered() const { return config_.ordered; }
 
+uint16_t MockDataChannel::maxRetransmitTime() const {
+  // TODO(https://bugs.chromium.org/854385): Restore when change landed.
+  // return config_.maxRetransmitTime;
+  return -1;
+}
+
+uint16_t MockDataChannel::maxRetransmits() const {
+  // TODO(https://bugs.chromium.org/854385): Restore when change landed.
+  // return config_.maxRetransmits;
+  return -1;
+}
+
 std::string MockDataChannel::protocol() const { return config_.protocol; }
 
 bool MockDataChannel::negotiated() const { return config_.negotiated; }

content/renderer/media/webrtc/mock_data_channel_impl.h

@@ -24,6 +24,8 @@ class MockDataChannel : public webrtc::DataChannelInterface {
   std::string label() const override;
   bool reliable() const override;
   bool ordered() const override;
+  uint16_t maxRetransmitTime() const override;
+  uint16_t maxRetransmits() const override;
   std::string protocol() const override;
   bool negotiated() const override;
   int id() const override;

third_party/blink/renderer/modules/peerconnection/rtc_data_channel.cc

@@ -71,17 +71,12 @@ void IncrementCounters(const webrtc::DataChannelInterface& channel) {
   if (channel.negotiated())
     IncrementCounter(DataChannelCounters::kNegotiated);
 
-  // Only record max retransmits and max packet life time if set.
-  if (channel.maxRetransmitsOpt()) {
   UMA_HISTOGRAM_CUSTOM_COUNTS("WebRTC.DataChannelMaxRetransmits",
-                                *(channel.maxRetransmitsOpt()), 1,
+                              channel.maxRetransmits(), 1,
                               std::numeric_limits<uint16_t>::max(), 50);
-  }
-  if (channel.maxPacketLifeTime()) {
-    UMA_HISTOGRAM_CUSTOM_COUNTS("WebRTC.DataChannelMaxPacketLifeTime",
-                                *channel.maxPacketLifeTime(), 1,
+  UMA_HISTOGRAM_CUSTOM_COUNTS("WebRTC.DataChannelMaxRetransmitTime",
+                              channel.maxRetransmitTime(), 1,
                               std::numeric_limits<uint16_t>::max(), 50);
-  }
 }
 
 void RecordMessageSent(const webrtc::DataChannelInterface& channel,
@@ -262,24 +257,14 @@ bool RTCDataChannel::ordered() const {
   return channel()->ordered();
 }
 
-uint16_t RTCDataChannel::maxPacketLifeTime(bool& is_null) const {
+uint16_t RTCDataChannel::maxRetransmitTime() const {
   DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
-  if (channel()->maxPacketLifeTime()) {
-    is_null = false;
-    return *(channel()->maxPacketLifeTime());
-  }
-  is_null = true;
-  return -1;
+  return channel()->maxRetransmitTime();
 }
 
-uint16_t RTCDataChannel::maxRetransmits(bool& is_null) const {
+uint16_t RTCDataChannel::maxRetransmits() const {
   DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
-  if (channel()->maxRetransmitsOpt()) {
-    is_null = false;
-    return *(channel()->maxRetransmitsOpt());
-  }
-  is_null = true;
-  return -1;
+  return channel()->maxRetransmits();
 }
 
 String RTCDataChannel::protocol() const {

third_party/blink/renderer/modules/peerconnection/rtc_data_channel.h

@@ -66,8 +66,8 @@ class MODULES_EXPORT RTCDataChannel final
   bool reliable() const;
 
   bool ordered() const;
-  uint16_t maxPacketLifeTime(bool&) const;
-  uint16_t maxRetransmits(bool&) const;
+  uint16_t maxRetransmitTime() const;
+  uint16_t maxRetransmits() const;
   String protocol() const;
   bool negotiated() const;
   uint16_t id(bool& is_null) const;

third_party/blink/renderer/modules/peerconnection/rtc_data_channel.idl

@@ -40,8 +40,10 @@ enum RTCDataChannelState {
 ] interface RTCDataChannel : EventTarget {
     readonly attribute USVString label;
     readonly attribute boolean ordered;
-    readonly attribute unsigned short? maxPacketLifeTime;
-    readonly attribute unsigned short? maxRetransmits;
+    // TODO(foolip): |maxRetransmitTime| is called |maxPacketLifeTime| in the
+    // spec and both it and |maxRetransmits| are nullable.
+    [Measure] readonly attribute unsigned short maxRetransmitTime;
+    [Measure] readonly attribute unsigned short maxRetransmits;
     readonly attribute USVString protocol;
     readonly attribute boolean negotiated;
     readonly attribute unsigned short? id;

third_party/blink/renderer/modules/peerconnection/rtc_data_channel_test.cc

@@ -84,12 +84,8 @@ class MockDataChannel : public webrtc::DataChannelInterface {
   std::string label() const override { return std::string(); }
   bool reliable() const override { return false; }
   bool ordered() const override { return false; }
-  absl::optional<int> maxPacketLifeTime() const override {
-    return absl::nullopt;
-  }
-  absl::optional<int> maxRetransmitsOpt() const override {
-    return absl::nullopt;
-  }
+  uint16_t maxRetransmitTime() const override { return 0; }
+  uint16_t maxRetransmits() const override { return 0; }
   std::string protocol() const override { return std::string(); }
   bool negotiated() const override { return false; }
   int id() const override { return 0; }

third_party/blink/web_tests/external/wpt/webrtc/RTCPeerConnection-createDataChannel-expected.txt

 This is a testharness.js-based test.
 PASS createDataChannel with no argument should throw TypeError
 PASS createDataChannel with closed connection should throw InvalidStateError
-FAIL createDataChannel attribute default values assert_equals: expected (string) "low" but got (undefined) undefined
-FAIL createDataChannel with provided parameters should initialize attributes to provided values assert_equals: expected (string) "high" but got (undefined) undefined
+FAIL createDataChannel attribute default values assert_equals: expected (object) null but got (undefined) undefined
+FAIL createDataChannel with provided parameters should initialize attributes to provided values assert_equals: expected (object) null but got (undefined) undefined
 PASS createDataChannel with label "foo" should succeed
 PASS createDataChannel with label null should succeed
 PASS createDataChannel with label undefined should succeed
 PASS createDataChannel with label lone surrogate should succeed
 PASS createDataChannel with ordered false should succeed
 PASS createDataChannel with ordered null/undefined should succeed
-PASS createDataChannel with maxPacketLifeTime 0 should succeed
+FAIL createDataChannel with maxPacketLifeTime 0 should succeed assert_equals: expected (number) 0 but got (undefined) undefined
 PASS createDataChannel with maxRetransmits 0 should succeed
 PASS createDataChannel with both maxPacketLifeTime and maxRetransmits undefined should succeed
 PASS createDataChannel with both maxPacketLifeTime and maxRetransmits should throw TypeError

third_party/blink/web_tests/external/wpt/webrtc/RTCPeerConnection-ondatachannel-expected.txt

@@ -4,8 +4,8 @@ PASS Should be able to send data in a datachannel event handler
 FAIL Open event should not be raised when closing the channel in the datachannel event assert_unreached: Open event should not fire Reached unreachable code
 PASS Open event should be raised when closing the channel in the datachannel event after enqueuing a task
 FAIL Open event should not be raised when sending and immediately closing the channel in the datachannel event assert_unreached: Open event should not fire Reached unreachable code
-FAIL In-band negotiated channel created on remote peer should match the same configuration as local peer assert_equals: expected (string) "high" but got (undefined) undefined
-FAIL In-band negotiated channel created on remote peer should match the same (default) configuration as local peer assert_equals: expected (string) "low" but got (undefined) undefined
+FAIL In-band negotiated channel created on remote peer should match the same configuration as local peer assert_equals: expected (object) null but got (undefined) undefined
+FAIL In-band negotiated channel created on remote peer should match the same (default) configuration as local peer assert_equals: expected (object) null but got (undefined) undefined
 PASS Negotiated channel should not fire datachannel event on remote peer
 Harness: the test ran to completion.
 

third_party/blink/web_tests/external/wpt/webrtc/historical-expected.txt

 This is a testharness.js-based test.
 FAIL RTCDataChannel member reliable should not exist assert_false: expected false got true
-PASS RTCDataChannel member maxRetransmitTime should not exist
+FAIL RTCDataChannel member maxRetransmitTime should not exist assert_false: expected false got true
 FAIL RTCPeerConnection member addStream should not exist assert_false: expected false got true
 FAIL RTCPeerConnection member createDTMFSender should not exist assert_false: expected false got true
 FAIL RTCPeerConnection member getLocalStreams should not exist assert_false: expected false got true

third_party/blink/web_tests/external/wpt/webrtc/idlharness.https.window-expected.txt

@@ -398,7 +398,7 @@ PASS RTCDataChannel interface: existence and properties of interface prototype o
 PASS RTCDataChannel interface: existence and properties of interface prototype object's @@unscopables property
 PASS RTCDataChannel interface: attribute label
 PASS RTCDataChannel interface: attribute ordered
-PASS RTCDataChannel interface: attribute maxPacketLifeTime
+FAIL RTCDataChannel interface: attribute maxPacketLifeTime assert_true: The prototype object must have a property "maxPacketLifeTime" expected true got false
 PASS RTCDataChannel interface: attribute maxRetransmits
 PASS RTCDataChannel interface: attribute protocol
 PASS RTCDataChannel interface: attribute negotiated
@@ -423,7 +423,7 @@ PASS RTCDataChannel must be primary interface of new RTCPeerConnection().createD
 PASS Stringification of new RTCPeerConnection().createDataChannel('')
 PASS RTCDataChannel interface: new RTCPeerConnection().createDataChannel('') must inherit property "label" with the proper type
 PASS RTCDataChannel interface: new RTCPeerConnection().createDataChannel('') must inherit property "ordered" with the proper type
-PASS RTCDataChannel interface: new RTCPeerConnection().createDataChannel('') must inherit property "maxPacketLifeTime" with the proper type
+FAIL RTCDataChannel interface: new RTCPeerConnection().createDataChannel('') must inherit property "maxPacketLifeTime" with the proper type assert_inherits: property "maxPacketLifeTime" not found in prototype chain
 PASS RTCDataChannel interface: new RTCPeerConnection().createDataChannel('') must inherit property "maxRetransmits" with the proper type
 PASS RTCDataChannel interface: new RTCPeerConnection().createDataChannel('') must inherit property "protocol" with the proper type
 PASS RTCDataChannel interface: new RTCPeerConnection().createDataChannel('') must inherit property "negotiated" with the proper type

third_party/blink/web_tests/virtual/stable/webexposed/global-interface-listing-expected.txt

@@ -4989,7 +4989,7 @@ interface RTCDataChannel : EventTarget
     getter bufferedAmountLowThreshold
     getter id
     getter label
-    getter maxPacketLifeTime
+    getter maxRetransmitTime
     getter maxRetransmits
     getter negotiated
     getter onbufferedamountlow

third_party/blink/web_tests/virtual/webrtc-wpt-plan-b/external/wpt/webrtc/idlharness.https.window-expected.txt

@@ -398,7 +398,7 @@ PASS RTCDataChannel interface: existence and properties of interface prototype o
 PASS RTCDataChannel interface: existence and properties of interface prototype object's @@unscopables property
 PASS RTCDataChannel interface: attribute label
 PASS RTCDataChannel interface: attribute ordered
-PASS RTCDataChannel interface: attribute maxPacketLifeTime
+FAIL RTCDataChannel interface: attribute maxPacketLifeTime assert_true: The prototype object must have a property "maxPacketLifeTime" expected true got false
 PASS RTCDataChannel interface: attribute maxRetransmits
 PASS RTCDataChannel interface: attribute protocol
 PASS RTCDataChannel interface: attribute negotiated
@@ -423,7 +423,7 @@ PASS RTCDataChannel must be primary interface of new RTCPeerConnection().createD
 PASS Stringification of new RTCPeerConnection().createDataChannel('')
 PASS RTCDataChannel interface: new RTCPeerConnection().createDataChannel('') must inherit property "label" with the proper type
 PASS RTCDataChannel interface: new RTCPeerConnection().createDataChannel('') must inherit property "ordered" with the proper type
-PASS RTCDataChannel interface: new RTCPeerConnection().createDataChannel('') must inherit property "maxPacketLifeTime" with the proper type
+FAIL RTCDataChannel interface: new RTCPeerConnection().createDataChannel('') must inherit property "maxPacketLifeTime" with the proper type assert_inherits: property "maxPacketLifeTime" not found in prototype chain
 PASS RTCDataChannel interface: new RTCPeerConnection().createDataChannel('') must inherit property "maxRetransmits" with the proper type
 PASS RTCDataChannel interface: new RTCPeerConnection().createDataChannel('') must inherit property "protocol" with the proper type
 PASS RTCDataChannel interface: new RTCPeerConnection().createDataChannel('') must inherit property "negotiated" with the proper type

third_party/blink/web_tests/webexposed/global-interface-listing-expected.txt

@@ -5794,7 +5794,7 @@ interface RTCDataChannel : EventTarget
     getter bufferedAmountLowThreshold
     getter id
     getter label
-    getter maxPacketLifeTime
+    getter maxRetransmitTime
     getter maxRetransmits
     getter negotiated
     getter onbufferedamountlow