Bug 236767 - Multiple concurrency violations in LibWebRTCCodecsProxy
Summary: Multiple concurrency violations in LibWebRTCCodecsProxy
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebRTC (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Kimmo Kinnunen
URL:
Keywords: InRadar
Depends on: 236832
Blocks:
  Show dependency treegraph
 
Reported: 2022-02-17 02:45 PST by Kimmo Kinnunen
Modified: 2022-03-03 01:53 PST (History)
4 users (show)

See Also:


Attachments
Patch (23.79 KB, patch)
2022-02-17 03:02 PST, Kimmo Kinnunen
ews-feeder: commit-queue-
Details | Formatted Diff | Diff
Patch (23.64 KB, patch)
2022-02-18 05:21 PST, Kimmo Kinnunen
no flags Details | Formatted Diff | Diff
Patch for landing (23.79 KB, patch)
2022-02-24 03:24 PST, Kimmo Kinnunen
no flags Details | Formatted Diff | Diff
Patch for landing (23.71 KB, patch)
2022-02-25 04:40 PST, Kimmo Kinnunen
no flags Details | Formatted Diff | Diff
Patch for landing (23.74 KB, patch)
2022-02-26 01:36 PST, Kimmo Kinnunen
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kimmo Kinnunen 2022-02-17 02:45:44 PST
Multiple concurrency violations in LibWebRTCCodecsProxy
Comment 1 Kimmo Kinnunen 2022-02-17 02:59:06 PST
<rdar://88904160>
Comment 2 Kimmo Kinnunen 2022-02-17 03:02:23 PST
Created attachment 452342 [details]
Patch
Comment 3 Kimmo Kinnunen 2022-02-18 05:21:53 PST
Created attachment 452512 [details]
Patch
Comment 4 Dean Jackson 2022-02-19 08:24:39 PST
Comment on attachment 452512 [details]
Patch

Removing r? flag - it's already been reviewed by Antti. It looks like this is an EWS/landing test.
Comment 5 Kimmo Kinnunen 2022-02-24 03:24:30 PST
Created attachment 453090 [details]
Patch for landing
Comment 6 Kimmo Kinnunen 2022-02-25 04:40:45 PST
Created attachment 453197 [details]
Patch for landing
Comment 7 EWS 2022-02-26 00:14:06 PST
Tools/Scripts/svn-apply failed to apply attachment 453197 [details] to trunk.
Please resolve the conflicts and upload a new patch.
Comment 8 Kimmo Kinnunen 2022-02-26 01:36:42 PST
Created attachment 453289 [details]
Patch for landing
Comment 9 EWS 2022-02-26 04:09:21 PST
Committed r290547 (?): <https://commits.webkit.org/r290547>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 453289 [details].
Comment 10 youenn fablet 2022-03-01 07:25:24 PST
Comment on attachment 453289 [details]
Patch for landing

View in context: https://bugs.webkit.org/attachment.cgi?id=453289&action=review

> Source/WebKit/GPUProcess/webrtc/LibWebRTCCodecsProxy.mm:281
> +        ASSERT_IS_TESTING_IPC();

This is a new ASSERT that is currently hit in bots, I'll fix it.
Comment 11 Kimmo Kinnunen 2022-03-03 01:53:14 PST
(In reply to youenn fablet from comment #10)
> Comment on attachment 453289 [details]
> Patch for landing
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=453289&action=review
> 
> > Source/WebKit/GPUProcess/webrtc/LibWebRTCCodecsProxy.mm:281
> > +        ASSERT_IS_TESTING_IPC();
> 
> This is a new ASSERT that is currently hit in bots, I'll fix it.

Thanks, sorry for the trouble..