Add QP sum stats for received streams.

webrtc/modules/video_coding/test/test_util.h

 /*
  *  Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 39 matching lines @@
 
   virtual webrtc::EventWrapper* CreateEvent() { return new NullEvent; }
 };
 
 class FileOutputFrameReceiver : public webrtc::VCMReceiveCallback {
  public:
   FileOutputFrameReceiver(const std::string& base_out_filename, uint32_t ssrc);
   virtual ~FileOutputFrameReceiver();
 
   // VCMReceiveCallback
-  virtual int32_t FrameToRender(webrtc::VideoFrame& video_frame);  // NOLINT
+  virtual int32_t FrameToRender(webrtc::VideoFrame& video_frame,
+                                int qp);  // NOLINT

[hbos, 2017/01/25 11:31:12]

Can you change virtual to override? If so do it everywhere in this file.

[sakal, 2017/01/25 12:39:34]

Done for this method.

I don't want to touch other classes in order to not clutter this already pretty
hard to review CL that touches a lot of files.

I also removed the NOLINTs because they didn't seem necessary and were possibly
added because of the missing override?

[hbos, 2017/01/26 15:04:45]

There might be waterfall bots that complain if a file mixes using virtual in
override cases and using override. That happened to me a couple of times and I
had to revert, not sure if that is still the case.

I'm OK with virtual or override in all places in the file. Or this way if it
doesn't break waterfall.

[sakal, 2017/01/26 15:46:52]

I changed all methods to override and removed a few unneeded ones.

 
  private:
   std::string out_filename_;
   FILE* out_file_;
   FILE* timing_file_;
   int width_;
   int height_;
   int count_;
 
   RTC_DISALLOW_IMPLICIT_CONSTRUCTORS(FileOutputFrameReceiver);
 };
 
 class CmdArgs {
  public:
   CmdArgs();
 
   std::string codecName;
   webrtc::VideoCodecType codecType;
   int width;
   int height;
   int rtt;
   std::string inputFile;
   std::string outputFile;
 };
 
 #endif  // WEBRTC_MODULES_VIDEO_CODING_TEST_TEST_UTIL_H_