Adding drawable to CRD andorid and iOS gl rendering pipeline.

remoting/client/gl_renderer.h

 // 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.
 
 #ifndef REMOTING_CLIENT_GL_RENDERER_H_
 #define REMOTING_CLIENT_GL_RENDERER_H_
 
 #include <queue>
+#include <vector>
 
 #include "base/callback.h"
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
 #include "base/threading/thread_checker.h"
 #include "remoting/client/gl_cursor.h"
 #include "remoting/client/gl_cursor_feedback.h"
 #include "remoting/client/gl_desktop.h"
 #include "remoting/proto/control.pb.h"
 
@@ skipping 65 matching lines @@
   // surface is recreated.
   void OnSurfaceCreated(int gl_version);
 
   // Sets the size of the view. Called right after OnSurfaceCreated() or
   // whenever the view size is changed.
   void OnSurfaceChanged(int view_width, int view_height);
 
   // Called when the surface is destroyed.
   void OnSurfaceDestroyed();
 
+  void AddDrawable(base::WeakPtr<GlDrawable> drawable);
+
   // Returns the weak pointer to be used on the display thread.
   base::WeakPtr<GlRenderer> GetWeakPtr();
 
+  static GlRenderer* CreateGlRendererWithDesktop();

[joedow, 2016/12/22 00:29:03]

Add comment block please.  Who owns the renderer* passed out?  Details about
expected ownership would be good.

[joedow, 2016/12/22 19:18:10]

Actually this would also be more clear if it returned a std::unique_ptr.  That
way it is clear that ownership of the object is transferred to the caller.

[nicholss, 2017/01/09 18:50:24]

Done.

+
  private:
   friend class GlRendererTest;
 
   // Post a rendering task to the task runner of current thread.
   // Do nothing if render_callback_ is not set yet or an existing rendering task
   // in the queue will cover changes before this function is called.
   void RequestRender();
 
   // Draws out everything on current OpenGL buffer and runs closures in
   // |pending_done_callbacks_|.
@@ skipping 11 matching lines @@
 
   int canvas_width_ = 0;
   int canvas_height_ = 0;
 
   std::unique_ptr<GlCanvas> canvas_;
 
   GlCursor cursor_;
   GlCursorFeedback cursor_feedback_;
   GlDesktop desktop_;
 
+  std::vector<base::WeakPtr<GlDrawable>> drawables_;
+
   base::ThreadChecker thread_checker_;
-  base::WeakPtr<GlRenderer> weak_ptr_;
   base::WeakPtrFactory<GlRenderer> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(GlRenderer);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_CLIENT_GL_RENDERER_H_