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

Unified Diff: media/base/decoder_buffer_queue.cc

Issue 2158923004: Convert media constants to constexpr. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 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: media/base/decoder_buffer_queue.cc
diff --git a/media/base/decoder_buffer_queue.cc b/media/base/decoder_buffer_queue.cc
index 433343081eb7b6a081bae8a150e743fad24649ad..cd4a6846ad9b0a5e1d0114f893d9813ab48a86a0 100644
--- a/media/base/decoder_buffer_queue.cc
+++ b/media/base/decoder_buffer_queue.cc
@@ -11,9 +11,7 @@
namespace media {
DecoderBufferQueue::DecoderBufferQueue()
- : earliest_valid_timestamp_(kNoTimestamp()),
- data_size_(0) {
-}
+ : earliest_valid_timestamp_(kNoTimestamp), data_size_(0) {}
DecoderBufferQueue::~DecoderBufferQueue() {}
@@ -26,12 +24,12 @@ void DecoderBufferQueue::Push(const scoped_refptr<DecoderBuffer>& buffer) {
// TODO(scherkus): FFmpeg returns some packets with no timestamp after
// seeking. Fix and turn this into CHECK(). See http://crbug.com/162192
- if (buffer->timestamp() == kNoTimestamp()) {
+ if (buffer->timestamp() == kNoTimestamp) {
DVLOG(1) << "Buffer has no timestamp";
return;
}
- if (earliest_valid_timestamp_ == kNoTimestamp()) {
+ if (earliest_valid_timestamp_ == kNoTimestamp) {
earliest_valid_timestamp_ = buffer->timestamp();
}
@@ -68,7 +66,7 @@ void DecoderBufferQueue::Clear() {
queue_.clear();
data_size_ = 0;
in_order_queue_.clear();
- earliest_valid_timestamp_ = kNoTimestamp();
+ earliest_valid_timestamp_ = kNoTimestamp;
}
bool DecoderBufferQueue::IsEmpty() {

Powered by Google App Engine
This is Rietveld 408576698