[TO 54] AudioRendererImpl: Don't advance time when rendering stops.

media/renderers/audio_renderer_impl_unittest.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/renderers/audio_renderer_impl.h"
 
 #include <memory>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 737 matching lines @@
   EXPECT_TRUE(ConsumeBufferedData(frames_to_consume, 1));
 
   // Current time should never go backwards even for irregular OS callbacks and
   // those with odd / wrong delay values.
   EXPECT_EQ(last_media_time, CurrentMediaTime());
 
   // Stop ticking, the media time should be clamped to what's been rendered.
   StopTicking();
   EXPECT_EQ(timestamp_helper.GetTimestamp(), CurrentMediaTime());
   tick_clock_->Advance(kConsumptionDuration * 2);
-  timestamp_helper.AddFrames(frames_to_consume.value);
+
+  // TODO(chcunningham): Uncomment the AddFrames() call below. AudioClock should
+  // be expected to advance time through the last rendered buffer's samples, but
+  // we've currently capped it to not advance time after ticking stops as a
+  // short term workaround for messy blink code. See longterm solution at
+  // http://crrev.com/2425463002.
+  // timestamp_helper.AddFrames(frames_to_consume.value);
   EXPECT_EQ(timestamp_helper.GetTimestamp(), CurrentMediaTime());
 }
 
 TEST_F(AudioRendererImplTest, RenderingDelayedForEarlyStartTime) {
   Initialize();
 
   // Choose a first timestamp a few buffers into the future, which ends halfway
   // through the desired output buffer; this allows for maximum test coverage.
   const double kBuffers = 4.5;
   const base::TimeDelta first_timestamp =
@@ skipping 258 matching lines @@
   // Advance far enough that we shouldn't be clamped to current time (tested
   // already above).
   tick_clock_->Advance(kOneSecond);
   EXPECT_EQ(
       current_time + timestamp_helper.GetFrameDuration(frames_to_consume.value),
       CurrentMediaWallClockTime(&is_time_moving));
   EXPECT_TRUE(is_time_moving);
 }
 
 }  // namespace media