Fix accumulation error with front_timestamp, move test only code.

media/filters/audio_clock_unittest.cc

Index: media/filters/audio_clock_unittest.cc
diff --git a/media/filters/audio_clock_unittest.cc b/media/filters/audio_clock_unittest.cc
index 2144bb0d67037504d93ee59d04a0b85bcd936b55..00743242862db8d7e44fefad5eecd79d7ed858a4 100644
--- a/media/filters/audio_clock_unittest.cc
+++ b/media/filters/audio_clock_unittest.cc
@@ -40,20 +40,27 @@ class AudioClockTest : public testing::Test {
   }
 
   int ContiguousAudioDataBufferedInDays() {
-    return clock_.contiguous_audio_data_buffered().InDays();
+    base::TimeDelta total, same_rate_total;
+    clock_.ContiguousAudioDataBufferedForTesting(&total, &same_rate_total);
+    return total.InDays();
   }
 
   int ContiguousAudioDataBufferedInMilliseconds() {
-    return clock_.contiguous_audio_data_buffered().InMilliseconds();
+    base::TimeDelta total, same_rate_total;
+    clock_.ContiguousAudioDataBufferedForTesting(&total, &same_rate_total);
+    return total.InMilliseconds();
   }
 
   int ContiguousAudioDataBufferedAtSameRateInMilliseconds() {
-    return clock_.contiguous_audio_data_buffered_at_same_rate()
-        .InMilliseconds();
+    base::TimeDelta total, same_rate_total;
+    clock_.ContiguousAudioDataBufferedForTesting(&total, &same_rate_total);
+    return same_rate_total.InMilliseconds();
   }
 
   const int sample_rate_;
   AudioClock clock_;
+  base::TimeDelta contiguous_audio_data_buffered_;
+  base::TimeDelta contiguous_audio_data_buffered_at_same_rate_;

[jrummell, 2015/07/14 17:32:22]

nit: I don't see these 2 values used anywhere in the changes. Moved from
audio_clock.h and then not used?

[DaleCurtis, 2015/07/14 17:33:59]

Whoops, yeah moved and unused. Removed. 

 
  private:
   DISALLOW_COPY_AND_ASSIGN(AudioClockTest);
@@ -73,15 +80,6 @@ TEST_F(AudioClockTest, BackTimestampStartsAtStartTimestamp) {
   EXPECT_EQ(expected, clock.back_timestamp());
 }
 
-TEST_F(AudioClockTest, ContiguousAudioDataBufferedStartsAtZero) {
-  EXPECT_EQ(base::TimeDelta(), clock_.contiguous_audio_data_buffered());
-}
-
-TEST_F(AudioClockTest, ContiguousAudioDataBufferedAtSameRateStartsAtZero) {
-  EXPECT_EQ(base::TimeDelta(),
-            clock_.contiguous_audio_data_buffered_at_same_rate());
-}
-
 TEST_F(AudioClockTest, Playback) {
   // The first time we write data we should still expect our start timestamp
   // due to delay.