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

Side by Side Diff: webkit/media/webmediaplayer_impl.h

Issue 12412007: Add a new API in WebMediaPlayer to do a GPU-GPU textures copy if possible (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Add new API in WebMediaPlayer to do GPU-GPU textures copy Created 7 years, 9 months 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 | « no previous file | webkit/media/webmediaplayer_impl.cc » ('j') | webkit/media/webmediaplayer_impl.cc » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 // Delegate calls from WebCore::MediaPlayerPrivate to Chrome's video player. 5 // Delegate calls from WebCore::MediaPlayerPrivate to Chrome's video player.
6 // It contains Pipeline which is the actual media player pipeline, it glues 6 // It contains Pipeline which is the actual media player pipeline, it glues
7 // the media player pipeline, data source, audio renderer and renderer. 7 // the media player pipeline, data source, audio renderer and renderer.
8 // Pipeline would creates multiple threads and access some public methods 8 // Pipeline would creates multiple threads and access some public methods
9 // of this class, so we need to be extra careful about concurrent access of 9 // of this class, so we need to be extra careful about concurrent access of
10 // methods and members. 10 // methods and members.
(...skipping 14 matching lines...) Expand all
25 #include "base/memory/ref_counted.h" 25 #include "base/memory/ref_counted.h"
26 #include "base/memory/scoped_ptr.h" 26 #include "base/memory/scoped_ptr.h"
27 #include "base/memory/weak_ptr.h" 27 #include "base/memory/weak_ptr.h"
28 #include "base/threading/thread.h" 28 #include "base/threading/thread.h"
29 #include "googleurl/src/gurl.h" 29 #include "googleurl/src/gurl.h"
30 #include "media/base/audio_renderer_sink.h" 30 #include "media/base/audio_renderer_sink.h"
31 #include "media/base/decryptor.h" 31 #include "media/base/decryptor.h"
32 #include "media/base/pipeline.h" 32 #include "media/base/pipeline.h"
33 #include "media/filters/skcanvas_video_renderer.h" 33 #include "media/filters/skcanvas_video_renderer.h"
34 #include "skia/ext/platform_canvas.h" 34 #include "skia/ext/platform_canvas.h"
35 #include "third_party/WebKit/Source/Platform/chromium/public/WebGraphicsContext3 D.h"
35 #include "third_party/WebKit/Source/WebKit/chromium/public/WebAudioSourceProvide r.h" 36 #include "third_party/WebKit/Source/WebKit/chromium/public/WebAudioSourceProvide r.h"
36 #include "third_party/WebKit/Source/WebKit/chromium/public/WebMediaPlayer.h" 37 #include "third_party/WebKit/Source/WebKit/chromium/public/WebMediaPlayer.h"
37 #include "third_party/WebKit/Source/WebKit/chromium/public/WebMediaPlayerClient. h" 38 #include "third_party/WebKit/Source/WebKit/chromium/public/WebMediaPlayerClient. h"
38 #include "webkit/media/crypto/key_systems.h" 39 #include "webkit/media/crypto/key_systems.h"
39 #include "webkit/media/crypto/proxy_decryptor.h" 40 #include "webkit/media/crypto/proxy_decryptor.h"
40 41
41 class RenderAudioSourceProvider; 42 class RenderAudioSourceProvider;
42 43
43 namespace WebKit { 44 namespace WebKit {
44 class WebFrame; 45 class WebFrame;
(...skipping 91 matching lines...) Expand 10 before | Expand all | Expand 10 after
136 virtual float mediaTimeForTimeValue(float timeValue) const; 137 virtual float mediaTimeForTimeValue(float timeValue) const;
137 138
138 virtual unsigned decodedFrameCount() const; 139 virtual unsigned decodedFrameCount() const;
139 virtual unsigned droppedFrameCount() const; 140 virtual unsigned droppedFrameCount() const;
140 virtual unsigned audioDecodedByteCount() const; 141 virtual unsigned audioDecodedByteCount() const;
141 virtual unsigned videoDecodedByteCount() const; 142 virtual unsigned videoDecodedByteCount() const;
142 143
143 virtual WebKit::WebVideoFrame* getCurrentFrame(); 144 virtual WebKit::WebVideoFrame* getCurrentFrame();
144 virtual void putCurrentFrame(WebKit::WebVideoFrame* web_video_frame); 145 virtual void putCurrentFrame(WebKit::WebVideoFrame* web_video_frame);
145 146
147 // Do the GPU-GPU texture copy using CHROMIUM_copy_texture extension
scherkus (not reviewing) 2013/03/08 20:17:37 technically this comment belongs on the interface
148 virtual bool copyVideoTextureToPlatformTexture(
149 WebKit::WebGraphicsContext3D* webGraphicsContext,
scherkus (not reviewing) 2013/03/08 20:17:37 chromium uses unix_hacker style for variable names
150 unsigned int texture,
151 unsigned int level,
152 unsigned int internalFormat);
scherkus (not reviewing) 2013/03/08 20:17:37 internal_format
153
146 virtual WebKit::WebAudioSourceProvider* audioSourceProvider(); 154 virtual WebKit::WebAudioSourceProvider* audioSourceProvider();
147 155
148 virtual MediaKeyException generateKeyRequest( 156 virtual MediaKeyException generateKeyRequest(
149 const WebKit::WebString& key_system, 157 const WebKit::WebString& key_system,
150 const unsigned char* init_data, 158 const unsigned char* init_data,
151 unsigned init_data_length); 159 unsigned init_data_length);
152 160
153 virtual MediaKeyException addKey(const WebKit::WebString& key_system, 161 virtual MediaKeyException addKey(const WebKit::WebString& key_system,
154 const unsigned char* key, 162 const unsigned char* key,
155 unsigned key_length, 163 unsigned key_length,
(...skipping 175 matching lines...) Expand 10 before | Expand all | Expand 10 after
331 media::SkCanvasVideoRenderer skcanvas_video_renderer_; 339 media::SkCanvasVideoRenderer skcanvas_video_renderer_;
332 scoped_refptr<media::VideoFrame> current_frame_; 340 scoped_refptr<media::VideoFrame> current_frame_;
333 bool pending_repaint_; 341 bool pending_repaint_;
334 342
335 DISALLOW_COPY_AND_ASSIGN(WebMediaPlayerImpl); 343 DISALLOW_COPY_AND_ASSIGN(WebMediaPlayerImpl);
336 }; 344 };
337 345
338 } // namespace webkit_media 346 } // namespace webkit_media
339 347
340 #endif // WEBKIT_MEDIA_WEBMEDIAPLAYER_IMPL_H_ 348 #endif // WEBKIT_MEDIA_WEBMEDIAPLAYER_IMPL_H_
OLDNEW
« no previous file with comments | « no previous file | webkit/media/webmediaplayer_impl.cc » ('j') | webkit/media/webmediaplayer_impl.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698