Introduce a client minimum picture pool size

remoting/client/plugin/pepper_video_renderer_3d.h

 // Copyright 2014 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 REMOTING_CLIENT_PLUGIN_PEPPER_VIDEO_RENDERER_3D_H_
 #define REMOTING_CLIENT_PLUGIN_PEPPER_VIDEO_RENDERER_3D_H_
 
 #include <deque>
 #include <string>
 
@@ skipping 77 matching lines @@
 
   // Initializes |shader_program_| with the given shaders.
   void CreateProgram(const char* vertex_shader, const char* fragment_shader);
 
   // Creates a new shader and compiles |source| for it.
   void CreateShaderProgram(int type, const char* source);
 
   // CHECKs that the last OpenGL call has completed successfully.
   void CheckGLError();
 
+  // The implementation here requires this minimum number of pictures from the
+  // video decoder interface to work.
+  static const uint32_t kMinimumPictureCount = 3;

[Sergey Ulanov, 2015/08/11 23:10:20]

This doesn't need to be defined in the class. It can be just static const in the
.cc file.

+
   EventHandler* event_handler_;
 
   pp::Graphics3D graphics_;
   const PPB_OpenGLES2* gles2_if_;
   pp::VideoDecoder video_decoder_;
 
   webrtc::DesktopSize frame_size_;
   webrtc::DesktopVector frame_dpi_;
   webrtc::DesktopRegion desktop_shape_;
 
@@ skipping 46 matching lines @@
   bool debug_dirty_region_;
 
   pp::CompletionCallbackFactory<PepperVideoRenderer3D> callback_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(PepperVideoRenderer3D);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_CLIENT_PLUGIN_PEPPER_VIDEO_RENDERER_3D_H_