Skip audio frames with unknown timestamp in MPEG2TS

media/formats/mp2t/es_parser_adts.cc

Index: media/formats/mp2t/es_parser_adts.cc
diff --git a/media/formats/mp2t/es_parser_adts.cc b/media/formats/mp2t/es_parser_adts.cc
index bb9433220cce02dd3838b419eb2674d88e3e8caf..b2f3ca0b505a99c2e36617ea5cb2c197408f22ab 100644
--- a/media/formats/mp2t/es_parser_adts.cc
+++ b/media/formats/mp2t/es_parser_adts.cc
@@ -140,8 +140,9 @@ bool EsParserAdts::ParseFromEsQueue() {
       audio_timestamp_helper_->SetBaseTimestamp(current_timing_desc.pts);
 
     if (audio_timestamp_helper_->base_timestamp() == kNoTimestamp()) {
-      DVLOG(1) << "Audio frame with unknown timestamp";
-      return false;
+      DVLOG(1) << "Skipped audio frame with unknown timestamp";

[damienv1, 2014/10/09 18:29:09]

nit: would prefer the present. "Skip audio frame...

[servolk, 2014/10/09 18:53:04]

Done.

+      SkipAdtsFrame(adts_frame);

[damienv1, 2014/10/09 18:29:09]

Could you update the unit test as well ?

[servolk, 2014/10/09 18:53:04]

Done.

+      continue;

[damienv1, 2014/10/09 18:29:09]

To be consistent, you should do the same for the MPEG1 audio ES parser.
At some point, we might want to have an EsAdapterAudio.

[servolk, 2014/10/09 18:53:03]

Done.

     }
     base::TimeDelta current_pts = audio_timestamp_helper_->GetTimestamp();
     base::TimeDelta frame_duration =