| commit | 6d37d3361d5acfa1aeed812fca290c6b67148379 | |
| author | dalecurtis <dalecurtis@chromium.org> | |
| Tue, 16 Jun 2015 20:26:37 +0000 (16 13:26 -0700) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Tue, 16 Jun 2015 20:27:04 +0000 (16 20:27 +0000) | ||
| tree | db791bb5aa1325ac7c6b38e7c1f9a601c97bb80c | treesnapshot (tar.gz zip) |
| parent | fcd63268f1b8c9e2725ac2aac69c0a8de128d21a | commitdiff |
Don't hold provider lock when calling out of VFPCI.
DidReceiveFrame() can lead back into a HasCurrentFrame() call
which will cause a deadlock. Also avoids one more potential
hotspot in StopUsingProvider() where RemoveVideoFrameController()
was called while the provider lock was held.
BUG=496362
TEST=none
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1174193003
Cr-Commit-Position: refs/heads/master@{#334679}
DidReceiveFrame() can lead back into a HasCurrentFrame() call
which will cause a deadlock. Also avoids one more potential
hotspot in StopUsingProvider() where RemoveVideoFrameController()
was called while the provider lock was held.
BUG=496362
TEST=none
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1174193003
Cr-Commit-Position: refs/heads/master@{#334679}
| cc/layers/video_frame_provider_client_impl.cc | diffblobblamehistory |