Be able to build cast_unittest and related targets in Chrome tree

media/cast/audio_receiver/audio_receiver.cc

 // Copyright 2013 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/cast/audio_receiver/audio_receiver.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/message_loop/message_loop.h"
 #include "media/cast/audio_receiver/audio_decoder.h"
@@ skipping 53 matching lines @@
 class LocalRtpAudioFeedback : public RtpPayloadFeedback {
  public:
   explicit LocalRtpAudioFeedback(AudioReceiver* audio_receiver)
       : audio_receiver_(audio_receiver) {
   }
 
   virtual void CastFeedback(const RtcpCastMessage& cast_message) OVERRIDE {
     audio_receiver_->CastFeedback(cast_message);
   }
 
-  virtual void RequestKeyFrame() OVERRIDE {
+  virtual void RequestKeyFrame() {

[hubbe, 2013/09/26 23:03:20]

I guess I don't understand OVERRIDE, why is it not appropriate here?

[Alpha Left Google, 2013/09/27 00:45:13]

Because base class doesn't have this method any more. I think this method should
be deleted now.

[hubbe, 2013/09/27 19:22:53]

Then delete it?
Or at least add a TODO

[Alpha Left Google, 2013/09/27 21:13:53]

Already removed.

     DCHECK(false) << "Invalid callback";
   }
 
  private:
   AudioReceiver* audio_receiver_;
 };
 
 class LocalRtpReceiverStatistics : public RtpReceiverStatistics {
  public:
   explicit LocalRtpReceiverStatistics(RtpReceiver* rtp_receiver)
@@ skipping 210 matching lines @@
       weak_factory_.GetWeakPtr()), time_to_send);
 }
 
 void AudioReceiver::SendNextRtcpReport() {
   rtcp_->SendRtcpReport(incoming_ssrc_);
   ScheduleNextRtcpReport();
 }
 
 }  // namespace cast
 }  // namespace media