VideoRenderer: Fires all pending callbacks in dtor.

media/base/video_renderer.h

 // 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.
 
 #ifndef MEDIA_BASE_VIDEO_RENDERER_H_
 #define MEDIA_BASE_VIDEO_RENDERER_H_
 
 #include "base/callback.h"
 #include "base/memory/ref_counted.h"
 #include "base/time/time.h"
@@ skipping 10 matching lines @@
  public:
   // Used to update the pipeline's clock time. The parameter is the time that
   // the clock should not exceed.
   typedef base::Callback<void(base::TimeDelta)> TimeCB;
 
   // Used to query the current time or duration of the media.
   typedef base::Callback<base::TimeDelta()> TimeDeltaCB;
 
   VideoRenderer();
 
-  // Stops all operations and drops all pending callbacks.
-  // TODO(xhwang): Fires all pending callbacks to be consistent with the rest of
-  // media pipeline.
+  // Stops all operations and fires all pending callbacks.
   virtual ~VideoRenderer();
 
   // Initializes a VideoRenderer with |stream|, executing |init_cb| upon
   // completion.
   //
   // |statistics_cb| is executed periodically with video rendering stats, such
   // as dropped frames.
   //
   // |time_cb| is executed whenever time has advanced by way of video rendering.
   //
@@ skipping 30 matching lines @@
   // Only valid to call after a successful Initialize() or Flush().
   virtual void StartPlaying() = 0;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(VideoRenderer);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_VIDEO_RENDERER_H_