Bug 214447

Summary: GenerateCertificateAsync should be called on signaling thread
Product: WebKit Reporter: Sihui Liu <sihui_liu>
Component: WebRTCAssignee: Sihui Liu <sihui_liu>
Severity: Normal CC: alecflett, beidson, eric.carlson, ews-watchlist, ggaren, glenn, hta, jer.noble, jsbell, philipj, sergio, tommyw, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=214481
Description Flags
Patch none

Description Sihui Liu 2020-07-17 00:25:23 PDT
Comment 1 Sihui Liu 2020-07-17 00:35:26 PDT
Created attachment 404545 [details]
Comment 2 Sihui Liu 2020-07-17 09:53:30 PDT
The added test would crash:
# Fatal error in: /Volumes/Data/worker/macOS-Mojave-Debug-Build-EWS/build/Source/ThirdParty/libwebrtc/Source/webrtc/rtc_base/rtc_certificate_generator.cc, line 147
# last system error: 2
# Check failed: signaling_thread_->IsCurrent()
Comment 3 Alexey Proskuryakov 2020-07-18 11:24:24 PDT
Is this a duplicate of bug 214478? This was filed earlier, but that one is more comprehensive, listing more affected tests.
Comment 4 youenn fablet 2020-07-20 02:15:25 PDT
Created attachment 404703 [details]
Comment 5 Geoffrey Garen 2020-07-20 09:26:29 PDT
Comment on attachment 404703 [details]

Comment 6 Geoffrey Garen 2020-07-20 09:27:26 PDT
Do we need to unskip storage/indexeddb/structured-clone-rtccertificate now?
Comment 7 youenn fablet 2020-07-20 09:52:59 PDT
(In reply to Geoffrey Garen from comment #6)
> Do we need to unskip storage/indexeddb/structured-clone-rtccertificate now?

LayoutTests/storage/indexeddb/structured-clone.html does not contain any RTCCertificate use (after https://bugs.webkit.org/show_bug.cgi?id=214481) so it should no longer crash. We probably forgot to reenable it.

We should probably readd RTCCertificate in the test and reenable it at the same time.
Comment 8 EWS 2020-07-20 09:58:18 PDT
Committed r264604: <https://trac.webkit.org/changeset/264604>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 404703 [details].
Comment 9 Radar WebKit Bug Importer 2020-07-20 09:59:25 PDT