Changeset 278851 in webkit

Timestamp:

Jun 14, 2021, 4:39:32 PM (5 years ago)

Author:

Truitt Savell

Message:

Unreviewed, reverting r278842.

introduced a failing test

Reverted changeset:

"Fix RTCDataChannelInit::decode"
​https://bugs.webkit.org/show_bug.cgi?id=226968
​https://commits.webkit.org/r278842

Location:

trunk

Files:

• LayoutTests/ChangeLog (modified) (1 diff)
• LayoutTests/imported/w3c/ChangeLog (modified) (1 diff)
• LayoutTests/imported/w3c/web-platform-tests/webrtc-extensions/transfer-datachannel-service-worker.https-expected.txt (deleted)
• LayoutTests/imported/w3c/web-platform-tests/webrtc-extensions/transfer-datachannel-service-worker.https.html (deleted)
• LayoutTests/imported/w3c/web-platform-tests/webrtc-extensions/transfer-datachannel-service-worker.js (deleted)
• LayoutTests/platform/mac-wk1/TestExpectations (modified) (1 diff)
• Source/WebCore/ChangeLog (modified) (1 diff)
• Source/WebCore/platform/mediastream/RTCDataChannelHandler.h (modified) (2 diffs)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2021-06-14  Truitt Savell  <tsavell@apple.com>
+
+        Unreviewed, reverting r278842.
+
+        introduced a failing test
+
+        Reverted changeset:
+
+        "Fix RTCDataChannelInit::decode"
+        https://bugs.webkit.org/show_bug.cgi?id=226968
+        https://commits.webkit.org/r278842
+
 2021-06-14  Patrick Angle  <pangle@apple.com>
 

trunk/LayoutTests/imported/w3c/ChangeLog

@@ +1 @@
+2021-06-14  Truitt Savell  <tsavell@apple.com>
+
+        Unreviewed, reverting r278842.
+
+        introduced a failing test
+
+        Reverted changeset:
+
+        "Fix RTCDataChannelInit::decode"
+        https://bugs.webkit.org/show_bug.cgi?id=226968
+        https://commits.webkit.org/r278842
+
 2021-06-14  Youenn Fablet  <youenn@apple.com>
 

trunk/LayoutTests/platform/mac-wk1/TestExpectations

@@ -377 +377 @@
 imported/w3c/web-platform-tests/worklets/layout-worklet-service-worker-interception.https.html [ Skip ]
 imported/w3c/web-platform-tests/worklets/paint-worklet-service-worker-interception.https.html [ Skip ]
-imported/w3c/web-platform-tests/webrtc-extensions/transfer-datachannel-service-worker.https.html [ Skip ]
 
 # Quota check missing in WK1

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2021-06-14  Truitt Savell  <tsavell@apple.com>
+
+        Unreviewed, reverting r278842.
+
+        introduced a failing test
+
+        Reverted changeset:
+
+        "Fix RTCDataChannelInit::decode"
+        https://bugs.webkit.org/show_bug.cgi?id=226968
+        https://commits.webkit.org/r278842
+
 2021-06-14  Devin Rousso  <drousso@apple.com>
 

trunk/Source/WebCore/platform/mediastream/RTCDataChannelHandler.h

@@ -63 +63 @@
 template<class Decoder> std::optional<RTCDataChannelInit> RTCDataChannelInit::decode(Decoder& decoder)
 {
-    std::optional<std::optional<bool>> ordered;
+    std::optional<bool> ordered;
     decoder >> ordered;
     if (!ordered)
         return { };
 
-    std::optional<std::optional<unsigned short>> maxPacketLifeTime;
+    std::optional<unsigned short> maxPacketLifeTime;
     decoder >> maxPacketLifeTime;
     if (!maxPacketLifeTime)
         return { };
 
-    std::optional<std::optional<unsigned short>> maxRetransmits;
+    std::optional<unsigned short> maxRetransmits;
     decoder >> maxRetransmits;
     if (!maxRetransmits)
@@ -82 +82 @@
         return { };
 
-    std::optional<std::optional<bool>> negotiated;
+    std::optional<bool> negotiated;
     decoder >> negotiated;
     if (!negotiated)
         return { };
 
-    std::optional<std::optional<unsigned short>> id;
+    std::optional<unsigned short> id;
     decoder >> id;
     if (!id)