webkit/media/android/webmediaplayer_android.cc - Issue 11269017: Plumb through cropped output size for VideoFrame

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: webkit/media/android/webmediaplayer_android.cc

Issue 11269017: Plumb through cropped output size for VideoFrame (Closed) Base URL: https://git.chromium.org/git/chromium/src@git-svn

Patch Set: Found the windows failure, and fixed it. Thanks akalin@ Created 8 years, 1 month ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

Index: webkit/media/android/webmediaplayer_android.cc

diff --git a/webkit/media/android/webmediaplayer_android.cc b/webkit/media/android/webmediaplayer_android.cc

index 468e180490b80c8142a554c1af4f4a3bb3ae1d76..6c0722cee6fd16ab5f4f856aa55c805e21593d73 100644

--- a/webkit/media/android/webmediaplayer_android.cc

+++ b/webkit/media/android/webmediaplayer_android.cc

@@ -376,8 +376,8 @@ void WebMediaPlayerAndroid::WillDestroyCurrentMessageLoop() {

void WebMediaPlayerAndroid::ReallocateVideoFrame() {

if (texture_id_) {

video_frame_.reset(new WebVideoFrameImpl(VideoFrame::WrapNativeTexture(

- texture_id_, kGLTextureExternalOES, natural_size_, natural_size_,

- base::TimeDelta(),

+ texture_id_, kGLTextureExternalOES, natural_size_,

+ gfx::Rect(natural_size_), natural_size_, base::TimeDelta(),

VideoFrame::ReadPixelsCB(),

base::Closure())));

}