Attachment 357064 Details for Bug 192599 – Patch

[patch] Patch

bug-192599-20181211112037.patch (text/plain), 4.49 KB, created by youenn fablet on 2018-12-11 11:20:38 PST

(hide)

Description:

Filename:

MIME Type:

Creator: youenn fablet

Created: 2018-12-11 11:20:38 PST

Size: 4.49 KB

patch

obsolete

>Subversion Revision: 239028
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 3aa0e8cea035144cceafc95e81412646bf5b92e9..2943d7300956221efb0a38b55dca1ce7f62b50d3 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,17 @@
>+2018-12-11  Youenn Fablet  <youenn@apple.com>
>+
>+        IDB should store RTCCertificate
>+        https://bugs.webkit.org/show_bug.cgi?id=192599
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        In case there is no script execution context, do not create a JS wrapper for RTCCertificate.
>+
>+        Test: webrtc/certificates-indexeddb.html
>+
>+        * bindings/js/SerializedScriptValue.cpp:
>+        (WebCore::CloneDeserializer::readTerminal):
>+
> 2018-12-10  Youenn Fablet  <youenn@apple.com>
> 
>         Remove derived classes of RealtimeMediaSourceCenter
>diff --git a/Source/WebCore/bindings/js/SerializedScriptValue.cpp b/Source/WebCore/bindings/js/SerializedScriptValue.cpp
>index ab8120fa34883caadf7b2067c826f2ccffd23ca6..53e8d760634884cbb23402576ec648eb46bc843a 100644
>--- a/Source/WebCore/bindings/js/SerializedScriptValue.cpp
>+++ b/Source/WebCore/bindings/js/SerializedScriptValue.cpp
>@@ -3014,6 +3014,10 @@ private:
>             return readImageBitmap();
> #if ENABLE(WEB_RTC)
>         case RTCCertificateTag:
>+            if (!scriptExecutionContextFromExecState(m_exec)) {
>+                fail();
>+                return JSValue();
>+            }
>             return readRTCCertificate();
> 
> #endif
>diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
>index 202893917e53d9aaa0af273fc573667ad8a8ea56..da1ac9a2facbd66bd05ba54e5434bb3f8c76defb 100644
>--- a/LayoutTests/ChangeLog
>+++ b/LayoutTests/ChangeLog
>@@ -1,3 +1,13 @@
>+2018-12-11  Youenn Fablet  <youenn@apple.com>
>+
>+        IDB should store RTCCertificate
>+        https://bugs.webkit.org/show_bug.cgi?id=192599
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * webrtc/certificates-indexeddb-expected.txt: Added.
>+        * webrtc/certificates-indexeddb.html: Added.
>+
> 2018-12-09  Commit Queue  <commit-queue@webkit.org>
> 
>         Unreviewed, rolling out r239010.
>diff --git a/LayoutTests/webrtc/certificates-indexeddb-expected.txt b/LayoutTests/webrtc/certificates-indexeddb-expected.txt
>new file mode 100644
>index 0000000000000000000000000000000000000000..cdcf735d091b471b5722f774cb273c0deae0b395
>--- /dev/null
>+++ b/LayoutTests/webrtc/certificates-indexeddb-expected.txt
>@@ -0,0 +1,3 @@
>+
>+PASS Testing certificates IndexedDB storage 
>+
>diff --git a/LayoutTests/webrtc/certificates-indexeddb.html b/LayoutTests/webrtc/certificates-indexeddb.html
>new file mode 100644
>index 0000000000000000000000000000000000000000..3aa8bacac116d1f77324dbedccad153fcf2cfd47
>--- /dev/null
>+++ b/LayoutTests/webrtc/certificates-indexeddb.html
>@@ -0,0 +1,54 @@
>+<!doctype html>
>+<html>
>+    <head>
>+        <meta charset="utf-8">
>+        <title>Testing certificates IndexedDB storage</title>
>+        <script src="../resources/testharness.js"></script>
>+        <script src="../resources/testharnessreport.js"></script>
>+    </head>
>+    <body>
>+        <script>
>+let certificate;
>+
>+function storeKey() {
>+    var objectStore = db.transaction("certificate", "readwrite").objectStore("certificate");
>+    var req = objectStore.put(certificate, "mycertificate");
>+    req.onerror = function(event) {
>+        assert_unreached("Could not put a key into database: " + event.target.error.name);
>+    }
>+    req.onsuccess = function(event) { readKey(); }
>+}
>+
>+function readKey() {
>+    var objectStore = db.transaction("certificate").objectStore("certificate");
>+    var req = objectStore.get("mycertificate");
>+    req.onerror = function(event) {
>+        assert_not_reached("Could not get a key from database: " + event.target.error.name);
>+    }
>+    req.onsuccess = function(event) {
>+        assert_not_equals(event.target.result, undefined);
>+        assert_equals(event.target.result.expires, certificate.expires);
>+        done();
>+    }
>+}
>+
>+async function doTest() {
>+    certificate = await  RTCPeerConnection.generateCertificate({ name: 'ECDSA', namedCurve: 'P-256' });
>+
>+    var openRequest = indexedDB.open("rtc");
>+    openRequest.onupgradeneeded = function(event) {
>+        var objectStore = event.target.result.createObjectStore("certificate");
>+    }
>+    openRequest.onerror = function(event) {
>+        assert_not_reached("Could not open database: " + event.target.error.name);
>+    }
>+    openRequest.onsuccess = function(event) {
>+        db = event.target.result;
>+        storeKey();
>+    }
>+}
>+doTest();
>+
>+        </script>
>+    </body>
>+</html>

Actions: View | Formatted Diff | Diff

Attachments on bug 192599: 357064 | 357065 | 357073