New issue
Advanced search Search tips

Issue 878070 link

Starred by 1 user
Status: Available
Owner: ----
Cc:
penghuang@chromium.org
gab@chromium.org
mlamouri@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

ClientDiscardableSharedMemoryManager creates waitable event when it shouldnt.

Project Member Reported by lethalantidote@chromium.org, Aug 27 
When compositing videos on a new thread, we ended up getting a warning about using ScopedAllowBaseSyncPrimitives. It turns out that we were creating a waitable event. We suspect that may be incorrect.

Comment 1 by mlamouri@chromium.org, Aug 27 

To give more colours, we shouldn't use WaitableEvent in the code base. Maybe this is required and there is no way to avoid it, in which case this bug should be closed and the TODO in the code should be removed. Though, given that the WaitableEvent seems to be simulating a sync IPC, it sounds that it's something we could avoid with an async API maybe?

Comment 2 by lethalantidote@chromium.org, Aug 28

Owner: gab@chromium.org

Comment 3 by gab@chromium.org, Aug 29

Cc: gab@chromium.org
Owner: ----
I can consult but not own this fix

Comment 4 by weiliangc@chromium.org, Aug 31

Cc: penghuang@chromium.org
Status: Available (was: Untriaged)
cc'ing owner from ClientDiscardableSharedMemoryManager.
►

Sign in to add a comment