New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 872065 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 8
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug

Blocking:
issue 866708



Sign in to add a comment

TextureVirtualDeviceMojoAdapterTest.BufferHandlesAreSharedWithReceiverConnectingLate fails with Mojo changes

Project Member Reported by roc...@chromium.org, Aug 7

Issue description

See blocked bug for details and motivation regarding what's changing in Mojo. Likely the failures are caused by incorrect task ordering assumptions.
 
Cc: chfremer@chromium.org
+chfremer in case you have any insight or cycles to help look into this
Labels: -Pri-3 Pri-1
I will look at it today.
And just to clarify, these failures appear when applying this CL[1]. The CL does not break any guarantees made by Mojo bindings, but it can affect the timing of arbitrary events. The failures therefore strongly imply that incorrect ordering assumptions being made somewhere, and this has been the case for all other blocking bugs which have been fixed so far.

[1] https://chromium-review.googlesource.com/c/chromium/src/+/1145692
Cc: -chfremer@chromium.org roc...@chromium.org
Owner: chfremer@chromium.org
Status: Fixed (was: Started)
Project Member

Comment 8 by bugdroid1@chromium.org, Aug 8

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7ffdde3135707d317126d4abad6d44ac81048681

commit 7ffdde3135707d317126d4abad6d44ac81048681
Author: Christian Fremerey <chfremer@chromium.org>
Date: Wed Aug 08 22:02:55 2018

Fix video capture unit test making wrong assumption about event ordering

Bug:  872065 
Change-Id: If0de2e5cb8470e604bc97f1c3689ae8972888e6c
Reviewed-on: https://chromium-review.googlesource.com/1167968
Reviewed-by: Ken Rockot <rockot@chromium.org>
Commit-Queue: Christian Fremerey <chfremer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581702}
[modify] https://crrev.com/7ffdde3135707d317126d4abad6d44ac81048681/services/video_capture/texture_virtual_device_mojo_adapter_unittest.cc

Sign in to add a comment