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

Unified Diff: ppapi/shared_impl/media_stream_frame_buffer.h

Issue 140783004: [PPAPI] Pepper MediaStream API audio track implementation and example. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update Created 6 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: ppapi/shared_impl/media_stream_frame_buffer.h
diff --git a/ppapi/shared_impl/media_stream_frame_buffer.h b/ppapi/shared_impl/media_stream_frame_buffer.h
index e95084642c03bc20a2ae32b452df4f0a88ca8c4d..46cfee402e01fdada265ca9f6a8d6d7540f4b51b 100644
--- a/ppapi/shared_impl/media_stream_frame_buffer.h
+++ b/ppapi/shared_impl/media_stream_frame_buffer.h
@@ -38,8 +38,10 @@ class PPAPI_SHARED_EXPORT MediaStreamFrameBuffer {
class PPAPI_SHARED_EXPORT Delegate {
public:
virtual ~Delegate();
- // It is called when a new frame is enqueued.
- virtual void OnNewFrameEnqueued();
+ // It is called before a new frame being enqueued. Return true if the frame
+ // is consumed by |Delegate|, and then the frame will not be added into the
+ // |frame_queue_|.
+ virtual bool OnNewFramePreEnqueued(int32_t index);
yzshen1 2014/01/29 21:28:06 BeforeNewFrameEnqueued() may be more clearer. Tha
Peng 2014/01/31 18:54:43 I though the performance is slightly better by avo
};
// MediaStreamFrameBuffer doesn't own |delegate|, the caller should keep

Powered by Google App Engine
This is Rietveld 408576698