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

Unified Diff: media/formats/mp2t/es_parser_mpeg1audio.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/formats/mp2t/es_parser_mpeg1audio.cc
diff --git a/media/formats/mp2t/es_parser_mpeg1audio.cc b/media/formats/mp2t/es_parser_mpeg1audio.cc
index 96ed8d62c4820fd27928519fafa64560489523fc..7cde8f80e1aafbc3eacab82da5488d70bf254c67 100644
--- a/media/formats/mp2t/es_parser_mpeg1audio.cc
+++ b/media/formats/mp2t/es_parser_mpeg1audio.cc
@@ -60,10 +60,10 @@ bool EsParserMpeg1Audio::ParseFromEsQueue() {
// Get the PTS & the duration of this access unit.
TimingDesc current_timing_desc =
GetTimingDescriptor(mpeg1audio_frame.queue_offset);
- if (current_timing_desc.pts != kNoTimestamp())
+ if (current_timing_desc.pts != kNoTimestamp)
audio_timestamp_helper_->SetBaseTimestamp(current_timing_desc.pts);
- if (audio_timestamp_helper_->base_timestamp() == kNoTimestamp()) {
+ if (audio_timestamp_helper_->base_timestamp() == kNoTimestamp) {
DVLOG(1) << "Skipping audio frame with unknown timestamp";
SkipMpeg1AudioFrame(mpeg1audio_frame);
continue;
@@ -179,7 +179,7 @@ bool EsParserMpeg1Audio::UpdateAudioConfiguration(
DVLOG(1) << "Channel layout: " << header.channel_layout;
// Reset the timestamp helper to use a new time scale.
if (audio_timestamp_helper_ &&
- audio_timestamp_helper_->base_timestamp() != kNoTimestamp()) {
+ audio_timestamp_helper_->base_timestamp() != kNoTimestamp) {
base::TimeDelta base_timestamp = audio_timestamp_helper_->GetTimestamp();
audio_timestamp_helper_.reset(
new AudioTimestampHelper(header.sample_rate));

Powered by Google App Engine
This is Rietveld 408576698