[Chromoting] Fallback to use software renderer if desktop image size is larger than GPU limitaiton

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 <stdint.h>
 
 #include <list>
 #include <memory>
 #include <string>
 
 #include "base/callback.h"
 #include "base/macros.h"
+#include "ppapi/c/ppb_opengles2.h"

[Sergey Ulanov, 2016/07/14 17:45:28]

Don't need this. Use int instead of GLint below.

[Hzj_jie, 2016/07/15 02:30:50]

Done.

 #include "ppapi/cpp/graphics_3d.h"
 #include "ppapi/cpp/instance_handle.h"
 #include "ppapi/cpp/video_decoder.h"
 #include "ppapi/utility/completion_callback_factory.h"
 #include "remoting/client/plugin/pepper_video_renderer.h"
+#include "remoting/client/plugin/pepper_video_renderer_2d.h"
 #include "remoting/protocol/video_stub.h"
 #include "third_party/webrtc/modules/desktop_capture/desktop_geometry.h"
 
 struct PPB_OpenGLES2;
 
 namespace remoting {
 
 namespace protocol {
 class FrameStatsConsumer;
 }  // namespace protocol
@@ skipping 98 matching lines @@
   std::list<std::unique_ptr<FrameTracker>> next_picture_frames_;
 
   // The current picture shown on the screen or being rendered. Must be deleted
   // before |video_decoder_|.
   std::unique_ptr<Picture> current_picture_;
 
   // FrameTrackers for frames in |current_picture_|. The queue is emptied once
   // the |current_picture_| is rendered.
   std::list<std::unique_ptr<FrameTracker>> current_picture_frames_;
 
+  // The fallback software renderer, if input video packet size is larger than
+  // hardware limitation.
+  PepperVideoRenderer2D fallback_renderer_;

[Sergey Ulanov, 2016/07/14 17:45:28]

I don't think we want to allocate fallback renderer until we actually need it.
Make this a pointer. Then you wouldn't need use_fallback_renderer_.

[Hzj_jie, 2016/07/15 02:30:50]

I believe SetPepperContext / OnViewChanged / EnableDebugDirtyRegion / Initialize
/ OnSessionConfig are called before processing the first frame. So we always
need to initialize fallback_renderer_ even we do not use it.

[Sergey Ulanov, 2016/07/15 23:33:18]

You do need to call these methods, but they can be called only when
fallback_renderer_ is actually needed.

[Hzj_jie, 2016/07/16 01:19:21]

Oh, that seems unachievable. We do not store most of the input parameters, and
some of them are even const reference. We even cannot guarantee whether they are
available later.

+  bool fallback_renderer_usable_;

[Sergey Ulanov, 2016/07/14 17:45:28]

The 2D renderer is not expected to fail. Just add a CHECK() to verify it's
initialized correctly.

[Hzj_jie, 2016/07/15 02:30:50]

Done.

+  bool use_fallback_renderer_;
+
   // Set to true if the screen has been resized and needs to be repainted.
   bool force_repaint_ = false;
 
   // The texture type for which |shader_program| was initialized. Can be either
   // 0, GL_TEXTURE_2D, GL_TEXTURE_RECTANGLE_ARB or GL_TEXTURE_EXTERNAL_OES. 0
   // indicates that |shader_program_| hasn't been intialized.
   uint32_t current_shader_program_texture_target_ = 0;
 
   // Shader program ID.
   unsigned int shader_program_ = 0;
 
   // Location of the scale value to be passed to the |shader_program_|.
   int shader_texcoord_scale_location_ = 0;
 
   // True if the renderer has received frame from the host.
   bool frame_received_ = false;
 
   // True if dirty regions are to be sent to |event_handler_| for debugging.
   bool debug_dirty_region_ = false;
 
+  GLint gl_max_texture_size_;

[Sergey Ulanov, 2016/07/14 17:45:28]

int

[Hzj_jie, 2016/07/15 02:30:50]

Done.

+
   pp::CompletionCallbackFactory<PepperVideoRenderer3D> callback_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(PepperVideoRenderer3D);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_CLIENT_PLUGIN_PEPPER_VIDEO_RENDERER_3D_H_