Consolidate and improve audio decoding test for all decoders.

media/filters/audio_file_reader.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/filters/audio_file_reader.h"
 
 #include <cmath>
 
 #include "base/logging.h"
 #include "base/time/time.h"
@@ skipping 239 matching lines @@
     // Skip packets from other streams.
     if (output_packet->stream_index != stream_index_) {
       av_free_packet(output_packet);
       continue;
     }
     return true;
   }
   return false;
 }
 
+bool AudioFileReader::SeekForTesting(base::TimeDelta seek_time) {

[wolenetz, 2014/06/06 21:00:05]

How ingrained is embedding alternate timebase times in a base::TimeDelta? It
seems strange / confusing versus just int64. Does it give us any advantage like
non-finite or invalid value representation that is meaningful for wherever we
put a non-microsecond time value as a microseconds in a TimeDelta? Especially
strange to me if this is ingrained in our non-test code.

[DaleCurtis, 2014/06/06 21:17:00]

See my comment about skipping timebase conversion when calling this function.

+  return av_seek_frame(glue_->format_context(),
+                       stream_index_,
+                       seek_time.InMicroseconds(),
+                       AVSEEK_FLAG_BACKWARD) >= 0;
+}
+
 }  // namespace media