[Remoting Android] New touch feedback texture

remoting/client/gl_renderer.cc

 // Copyright 2016 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/client/gl_renderer.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "remoting/client/gl_canvas.h"
@@ skipping 115 matching lines @@
 void GlRenderer::OnRender() {
   DCHECK(thread_checker_.CalledOnValidThread());
   render_scheduled_ = false;
   if (!delegate_ || !delegate_->CanRenderFrame()) {
     return;
   }
 
   if (canvas_) {
     glClear(GL_COLOR_BUFFER_BIT);
     desktop_.Draw();
-    cursor_.Draw();
     if (cursor_feedback_.Draw()) {
       RequestRender();
     }
+    cursor_.Draw();

[joedow, 2016/08/24 18:53:28]

I think it would be good to add some comments (either a block or inline) about
the layers.  At the moment we only have desktop, feedback, and cursor, but
perhaps we will want to add another layer in the future or someone reading the
code might wonder why/whether this order matters.

[Yuwei, 2016/08/24 19:06:44]

Done.

   }
 
   delegate_->OnFrameRendered();
 
   while (!pending_done_callbacks_.empty()) {
     pending_done_callbacks_.front().Run();
     pending_done_callbacks_.pop();
   }
 }
 
 }  // namespace remoting