Commit 9941e71c authored by Dale Curtis's avatar Dale Curtis Committed by Commit Bot

Fix invalid bool/int conversion on PrepareVideoFrameForWebGL.

This prevents the "skip already uploaded frame" logic from actually
working. Though it seems to be broken in some other way currently.

Bug: 776222
Change-Id: I58dc0490c976588d1c0d9a37210330fcfa399474
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2031720Reviewed-by: default avatarMounir Lamouri <mlamouri@chromium.org>
Reviewed-by: default avatarKai Ninomiya <kainino@chromium.org>
Commit-Queue: Mounir Lamouri <mlamouri@chromium.org>
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Auto-Submit: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#737400}
parent 9555c031
...@@ -506,7 +506,7 @@ bool HTMLVideoElement::PrepareVideoFrameForWebGL( ...@@ -506,7 +506,7 @@ bool HTMLVideoElement::PrepareVideoFrameForWebGL(
gpu::gles2::GLES2Interface* gl, gpu::gles2::GLES2Interface* gl,
GLenum target, GLenum target,
GLuint texture, GLuint texture,
bool already_uploaded_id, int already_uploaded_id,
WebMediaPlayer::VideoFrameUploadMetadata* out_metadata) { WebMediaPlayer::VideoFrameUploadMetadata* out_metadata) {
if (!GetWebMediaPlayer()) if (!GetWebMediaPlayer())
return false; return false;
......
...@@ -147,7 +147,7 @@ class CORE_EXPORT HTMLVideoElement final ...@@ -147,7 +147,7 @@ class CORE_EXPORT HTMLVideoElement final
gpu::gles2::GLES2Interface*, gpu::gles2::GLES2Interface*,
GLenum target, GLenum target,
GLuint texture, GLuint texture,
bool already_uploaded_id, int already_uploaded_id,
WebMediaPlayer::VideoFrameUploadMetadata* out_metadata); WebMediaPlayer::VideoFrameUploadMetadata* out_metadata);
bool ShouldDisplayPosterImage() const { return GetDisplayMode() == kPoster; } bool ShouldDisplayPosterImage() const { return GetDisplayMode() == kPoster; }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment