Index: cc/resources/video_resource_updater.cc |
diff --git a/cc/resources/video_resource_updater.cc b/cc/resources/video_resource_updater.cc |
index dee285a28506c651c282418be98a432d2cf88ed5..e6bf21b9cd3dc3b88a4da201b8532e9e9cc10826 100644 |
--- a/cc/resources/video_resource_updater.cc |
+++ b/cc/resources/video_resource_updater.cc |
@@ -311,7 +311,7 @@ VideoFrameExternalResources VideoResourceUpdater::CreateForSoftwarePlanes( |
static void ReturnTexture(const scoped_refptr<media::VideoFrame>& frame, |
uint32 sync_point, |
bool lost_resource) { |
- frame->mailbox_holder()->sync_point = sync_point; |
+ frame->AppendReleaseSyncPoint(sync_point); |
} |
VideoFrameExternalResources VideoResourceUpdater::CreateForHardwarePlanes( |
@@ -343,6 +343,8 @@ VideoFrameExternalResources VideoResourceUpdater::CreateForHardwarePlanes( |
return VideoFrameExternalResources(); |
} |
+ // Don't share the mailbox of the video frame with the compositor, because |
danakj
2014/03/06 19:31:31
What does this comment mean?
dshwang
2014/03/06 19:51:37
Sorry for broken explanation.
ResourceProvider act
|
+ // the mailbox of the video frame can be read by another thread. i.e. webgl |
external_resources.mailboxes.push_back( |
TextureMailbox(mailbox_holder->mailbox, |
mailbox_holder->texture_target, |