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

Side by Side Diff: media/base/android/media_codec_decoder.h

Issue 1963343002: Report media error if PlayOutputBuffer failed (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed Min's comment Created 4 years, 7 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
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 #ifndef MEDIA_BASE_ANDROID_MEDIA_CODEC_DECODER_H_ 5 #ifndef MEDIA_BASE_ANDROID_MEDIA_CODEC_DECODER_H_
6 #define MEDIA_BASE_ANDROID_MEDIA_CODEC_DECODER_H_ 6 #define MEDIA_BASE_ANDROID_MEDIA_CODEC_DECODER_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 9
10 #include <memory> 10 #include <memory>
(...skipping 267 matching lines...) Expand 10 before | Expand all | Expand 10 after
278 // We use delays for video decoder only. 278 // We use delays for video decoder only.
279 virtual void AssociateCurrentTimeWithPTS(base::TimeDelta current_time) {} 279 virtual void AssociateCurrentTimeWithPTS(base::TimeDelta current_time) {}
280 280
281 // Invalidate delay calculation. We use delays for video decoder only. 281 // Invalidate delay calculation. We use delays for video decoder only.
282 virtual void DissociatePTSFromTime() {} 282 virtual void DissociatePTSFromTime() {}
283 283
284 // Processes the change of the output format, varies by stream. 284 // Processes the change of the output format, varies by stream.
285 // Returns true if this processing succeeded. 285 // Returns true if this processing succeeded.
286 virtual bool OnOutputFormatChanged() = 0; 286 virtual bool OnOutputFormatChanged() = 0;
287 287
288 // Renders the decoded frame and releases output buffer, or posts 288 // Renders the decoded frame and releases output buffer, or posts a delayed
289 // a delayed task to do it at a later time, 289 // task to do it at a later time. Returnes false if an error occurred.
290 virtual void Render(int buffer_index, 290 virtual bool Render(int buffer_index,
291 size_t offset, 291 size_t offset,
292 size_t size, 292 size_t size,
293 RenderMode render_mode, 293 RenderMode render_mode,
294 base::TimeDelta pts, 294 base::TimeDelta pts,
295 bool eos_encountered) = 0; 295 bool eos_encountered) = 0;
296 296
297 // Returns the number of delayed task (we might have them for video). 297 // Returns the number of delayed task (we might have them for video).
298 virtual int NumDelayedRenderTasks() const; 298 virtual int NumDelayedRenderTasks() const;
299 299
300 // Releases output buffers that are dequeued and not released yet (video). 300 // Releases output buffers that are dequeued and not released yet (video).
(...skipping 160 matching lines...) Expand 10 before | Expand all | Expand 10 after
461 461
462 // NOTE: Weak pointers must be invalidated before all other member variables. 462 // NOTE: Weak pointers must be invalidated before all other member variables.
463 base::WeakPtrFactory<MediaCodecDecoder> weak_factory_; 463 base::WeakPtrFactory<MediaCodecDecoder> weak_factory_;
464 464
465 DISALLOW_COPY_AND_ASSIGN(MediaCodecDecoder); 465 DISALLOW_COPY_AND_ASSIGN(MediaCodecDecoder);
466 }; 466 };
467 467
468 } // namespace media 468 } // namespace media
469 469
470 #endif // MEDIA_BASE_ANDROID_MEDIA_CODEC_DECODER_H_ 470 #endif // MEDIA_BASE_ANDROID_MEDIA_CODEC_DECODER_H_
OLDNEW
« no previous file with comments | « media/base/android/audio_media_codec_decoder.cc ('k') | media/base/android/media_codec_decoder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698