Adding the iOS app and integration example with GlRenderer.

remoting/client/gl_renderer.h

Index: remoting/client/gl_renderer.h
diff --git a/remoting/client/gl_renderer.h b/remoting/client/gl_renderer.h
index 832bd636667a43a2f8915bf45feed3ed8782b762..17b7b3bdb931f193b3d62d221d117bd531ef0fd5 100644
--- a/remoting/client/gl_renderer.h
+++ b/remoting/client/gl_renderer.h
@@ -6,6 +6,7 @@
 #define REMOTING_CLIENT_GL_RENDERER_H_
 
 #include <queue>
+#include <vector>
 
 #include "base/callback.h"
 #include "base/macros.h"
@@ -91,6 +92,8 @@ class GlRenderer {
   // Called when the surface is destroyed.
   void OnSurfaceDestroyed();
 
+  void AddDrawable(GlDrawable* drawable);

[Yuwei, 2016/12/19 23:26:09]

Could we just make the class own the drawable? Holding raw pointer may cause
burden to maintain the order of destruction. What about passing and storing
std::unique_ptr?

Currently the way this class works is the caller sets the position of the cursor
and the desktop image, etc., then the class will just render out the result, so
how the canvas is drawn/the GlDrawable stuff seem to be an implementation detail
to me. I feel that it's clearer to provide a function like DrawDemoScreen() and
make the demo screen a part of the renderer.

Anyway it's still up to you to decide whether the user is allowed to add extra
drawables to the renderer O_o

[Yuwei, 2016/12/19 23:54:19]

I meant something like ShouldDrawDemoScreen(boolean). Not a function similar to
Draw()...

+
   // Returns the weak pointer to be used on the display thread.
   base::WeakPtr<GlRenderer> GetWeakPtr();
 
@@ -125,6 +128,8 @@ class GlRenderer {
   GlCursorFeedback cursor_feedback_;
   GlDesktop desktop_;
 
+  std::vector<GlDrawable*> drawables;

[joedow, 2016/12/19 23:56:26]

Does order of the drawing matter?  Maybe add a comment on whether it does or
not.

[joedow, 2016/12/19 23:56:26]

Is a raw* the right thing?  Can ownership be transferred or perhaps use a
WeakPtr here?

[joedow, 2016/12/19 23:56:26]

s/drawables/drawables_

+
   base::ThreadChecker thread_checker_;
   base::WeakPtr<GlRenderer> weak_ptr_;
   base::WeakPtrFactory<GlRenderer> weak_factory_;