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

remoting/host/desktop_session_agent.cc

 // Copyright (c) 2012 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 "remoting/host/desktop_session_agent.h"
 
 #include <utility>
 
 #include "base/files/file_util.h"
 #include "base/logging.h"
@@ skipping 439 matching lines @@
     // Stop the video capturer.
     video_capturer_.reset();
     last_frame_.reset();
     mouse_cursor_monitor_.reset();
   }
 }
 
 void DesktopSessionAgent::OnCaptureFrame() {
   DCHECK(caller_task_runner_->BelongsToCurrentThread());
 
-  mouse_cursor_monitor_->Capture();
-
   // webrtc::DesktopCapturer supports a very few (currently 2) outstanding
   // capture requests. The requests are serialized on
   // |video_capture_task_runner()| task runner. If the client issues more
   // requests, pixel data in captured frames will likely be corrupted but
   // stability of webrtc::DesktopCapturer will not be affected.
   video_capturer_->Capture(webrtc::DesktopRegion());
+
+  // Note, the client may switch between hardware and software renderers
+  // according to the first desktop image. So we need to make sure the first
+  // desktop image has been sent before the mouse cursor.
+  mouse_cursor_monitor_->Capture();

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

This doesn't guarantee that the cursor will be received after the first frame.
Cursor shape and video are sent on different channels, independent of each
other, so capture ordering here will have very little effect on how the messages
are received on the client.

But I don't see how it's related to this change. VideoRenderer is not
responsible for the cursor.

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

Sorry, my fault. I have double confirmed, the mouse cursor should be able to
render. There must be some other issues before.

 }
 
 void DesktopSessionAgent::OnInjectClipboardEvent(
     const std::string& serialized_event) {
   DCHECK(caller_task_runner_->BelongsToCurrentThread());
 
   protocol::ClipboardEvent event;
   if (!event.ParseFromString(serialized_event)) {
     LOG(ERROR) << "Failed to parse protocol::ClipboardEvent.";
     return;
@@ skipping 102 matching lines @@
   }
 }
 
 void DesktopSessionAgent::StopAudioCapturer() {
   DCHECK(audio_capture_task_runner_->BelongsToCurrentThread());
 
   audio_capturer_.reset();
 }
 
 }  // namespace remoting