Dynamic error tracking in RTCVideoDecoder

content/renderer/media/rtc_video_decoder.h

Index: content/renderer/media/rtc_video_decoder.h
diff --git a/content/renderer/media/rtc_video_decoder.h b/content/renderer/media/rtc_video_decoder.h
index fedb941f257c185e9f2a5312a423805300fb6e44..7ed95bb8fb1c4a4af9979f92f3c3c178e0094e86 100644
--- a/content/renderer/media/rtc_video_decoder.h
+++ b/content/renderer/media/rtc_video_decoder.h
@@ -194,6 +194,12 @@ class CONTENT_EXPORT RTCVideoDecoder
   // Clear the pending_buffers_ queue, freeing memory.
   void ClearPendingBuffers();
 
+  // Keep track of VDA errors.

[mcasas, 2016/04/01 16:39:09]

s/Keep/Keeps/ ,  from [1]
"These comments should be descriptive ("Opens the file") 
rather than imperative ("Open the file"); the comment 
describes the function, it does not tell the function what to do."

Same elsewhere.

[1] https://google.github.io/styleguide/cppguide.html#Function_Comments

[emircan, 2016/04/05 18:02:18]

Done.

+  void IncrementVDAErrorCounter();
+
+  // Reset |vda_error_counter_| after a successfull run of decode.
+  void TryResetVDAErrorCounter();
+
   enum State {
     UNINITIALIZED,  // The decoder has not initialized.
     INITIALIZED,    // The decoder has initialized.
@@ -211,7 +217,10 @@ class CONTENT_EXPORT RTCVideoDecoder
   media::VideoCodecProfile vda_codec_profile_;
 
   // Number of times that |vda_| notified of an error.
-  uint32_t num_vda_errors_;
+  uint32_t vda_error_counter_;
+
+  // Number of decode completed consecutively since the last vda error.
+  uint32_t num_consecutive_vda_success_since_error_;
 
   // The video codec type, as reported by WebRTC.
   const webrtc::VideoCodecType video_codec_type_;