Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(97)

Side by Side Diff: media/blink/webmediaplayer_impl.cc

Issue 1418513016: Revert of webgl: optimize webgl.texSubImage2D(video) path. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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:
View unified diff | Download patch
« no previous file with comments | « media/blink/webmediaplayer_impl.h ('k') | media/renderers/skcanvas_video_renderer.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "media/blink/webmediaplayer_impl.h" 5 #include "media/blink/webmediaplayer_impl.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <cmath> 8 #include <cmath>
9 #include <limits> 9 #include <limits>
10 #include <string> 10 #include <string>
(...skipping 624 matching lines...) Expand 10 before | Expand all | Expand 10 after
635 635
636 unsigned WebMediaPlayerImpl::videoDecodedByteCount() const { 636 unsigned WebMediaPlayerImpl::videoDecodedByteCount() const {
637 DCHECK(main_task_runner_->BelongsToCurrentThread()); 637 DCHECK(main_task_runner_->BelongsToCurrentThread());
638 638
639 PipelineStatistics stats = pipeline_.GetStatistics(); 639 PipelineStatistics stats = pipeline_.GetStatistics();
640 return stats.video_bytes_decoded; 640 return stats.video_bytes_decoded;
641 } 641 }
642 642
643 bool WebMediaPlayerImpl::copyVideoTextureToPlatformTexture( 643 bool WebMediaPlayerImpl::copyVideoTextureToPlatformTexture(
644 blink::WebGraphicsContext3D* web_graphics_context, 644 blink::WebGraphicsContext3D* web_graphics_context,
645 const CopyVideoTextureParams& params) { 645 unsigned int texture,
646 unsigned int internal_format,
647 unsigned int type,
648 bool premultiply_alpha,
649 bool flip_y) {
646 TRACE_EVENT0("media", "WebMediaPlayerImpl:copyVideoTextureToPlatformTexture"); 650 TRACE_EVENT0("media", "WebMediaPlayerImpl:copyVideoTextureToPlatformTexture");
647 DCHECK((params.copyType == CopyVideoTextureParams::FullCopy &&
648 !params.xoffset && !params.yoffset) ||
649 (params.copyType == CopyVideoTextureParams::SubCopy &&
650 !params.internalFormat && !params.type));
651 651
652 scoped_refptr<VideoFrame> video_frame = GetCurrentFrameFromCompositor(); 652 scoped_refptr<VideoFrame> video_frame = GetCurrentFrameFromCompositor();
653 653
654 if (!video_frame.get() || !video_frame->HasTextures() || 654 if (!video_frame.get() || !video_frame->HasTextures() ||
655 media::VideoFrame::NumPlanes(video_frame->format()) != 1) { 655 media::VideoFrame::NumPlanes(video_frame->format()) != 1) {
656 return false; 656 return false;
657 } 657 }
658 658
659 // TODO(dshwang): need more elegant way to convert WebGraphicsContext3D to 659 // TODO(dshwang): need more elegant way to convert WebGraphicsContext3D to
660 // GLES2Interface. 660 // GLES2Interface.
661 gpu::gles2::GLES2Interface* gl = 661 gpu::gles2::GLES2Interface* gl =
662 static_cast<gpu_blink::WebGraphicsContext3DImpl*>(web_graphics_context) 662 static_cast<gpu_blink::WebGraphicsContext3DImpl*>(web_graphics_context)
663 ->GetGLInterface(); 663 ->GetGLInterface();
664 typedef SkCanvasVideoRenderer::CopyFrameSingleTextureParams CopyParams;
665 SkCanvasVideoRenderer::CopyVideoFrameSingleTextureToGLTexture( 664 SkCanvasVideoRenderer::CopyVideoFrameSingleTextureToGLTexture(
666 gl, video_frame.get(), 665 gl, video_frame.get(), texture, internal_format, type, premultiply_alpha,
667 CopyParams(params.copyType == CopyVideoTextureParams::FullCopy 666 flip_y);
668 ? CopyParams::FullCopy
669 : CopyParams::SubCopy,
670 params.target, params.texture, params.internalFormat,
671 params.type, params.level, params.xoffset, params.yoffset,
672 params.premultiplyAlpha, params.flipY));
673 return true; 667 return true;
674 } 668 }
675 669
676 WebMediaPlayer::MediaKeyException 670 WebMediaPlayer::MediaKeyException
677 WebMediaPlayerImpl::generateKeyRequest(const WebString& key_system, 671 WebMediaPlayerImpl::generateKeyRequest(const WebString& key_system,
678 const unsigned char* init_data, 672 const unsigned char* init_data,
679 unsigned init_data_length) { 673 unsigned init_data_length) {
680 DCHECK(main_task_runner_->BelongsToCurrentThread()); 674 DCHECK(main_task_runner_->BelongsToCurrentThread());
681 675
682 return encrypted_media_support_.GenerateKeyRequest( 676 return encrypted_media_support_.GenerateKeyRequest(
(...skipping 400 matching lines...) Expand 10 before | Expand all | Expand 10 after
1083 1077
1084 // pause() may be called after playback has ended and the HTMLMediaElement 1078 // pause() may be called after playback has ended and the HTMLMediaElement
1085 // requires that currentTime() == duration() after ending. We want to ensure 1079 // requires that currentTime() == duration() after ending. We want to ensure
1086 // |paused_time_| matches currentTime() in this case or a future seek() may 1080 // |paused_time_| matches currentTime() in this case or a future seek() may
1087 // incorrectly discard what it thinks is a seek to the existing time. 1081 // incorrectly discard what it thinks is a seek to the existing time.
1088 paused_time_ = 1082 paused_time_ =
1089 ended_ ? pipeline_.GetMediaDuration() : pipeline_.GetMediaTime(); 1083 ended_ ? pipeline_.GetMediaDuration() : pipeline_.GetMediaTime();
1090 } 1084 }
1091 1085
1092 } // namespace media 1086 } // namespace media
OLDNEW
« no previous file with comments | « media/blink/webmediaplayer_impl.h ('k') | media/renderers/skcanvas_video_renderer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698