Refactor BrowserViewRenderer-RenderThreadManager relationship.

android_webview/native/aw_gl_functor.h

Index: android_webview/native/aw_gl_functor.h
diff --git a/android_webview/native/aw_gl_functor.h b/android_webview/native/aw_gl_functor.h
index 949af1c1afacf96bdb6de61062245a4fc433cbd1..3278ddac9decc8797747a7673c077b8bd62f5765 100644
--- a/android_webview/native/aw_gl_functor.h
+++ b/android_webview/native/aw_gl_functor.h
@@ -7,25 +7,22 @@
 
 #include <jni.h>
 
+#include "android_webview/browser/compositor_frame_consumer.h"
+#include "android_webview/browser/compositor_frame_producer.h"

[boliu, 2016/04/21 17:05:40]

don't need producer

[Tobias Sargeant, 2016/04/21 17:48:27]

Done.

 #include "android_webview/browser/render_thread_manager.h"
 #include "android_webview/browser/render_thread_manager_client.h"
 #include "base/android/jni_weak_ref.h"
 
 namespace android_webview {
 
-class BrowserViewRenderer;
-
 class AwGLFunctor : public RenderThreadManagerClient {
  public:
-  void OnParentDrawConstraintsUpdated() override;
   bool RequestInvokeGL(bool wait_for_completion) override;
   void DetachFunctorFromView() override;
 
   AwGLFunctor(const JavaObjectWeakGlobalRef& java_ref);
   ~AwGLFunctor() override;
 
-  void SetBrowserViewRenderer(BrowserViewRenderer* browser_view_renderer);
-
   void Destroy(JNIEnv* env, const base::android::JavaParamRef<jobject>& obj);
   void DeleteHardwareRenderer(JNIEnv* env,
                               const base::android::JavaParamRef<jobject>& obj);
@@ -34,14 +31,13 @@ class AwGLFunctor : public RenderThreadManagerClient {
   jlong GetAwDrawGLFunction(JNIEnv* env,
                             const base::android::JavaParamRef<jobject>& obj);
 
-  RenderThreadManager* GetRenderThreadManager() {
-    return &render_thread_manager_;
+  CompositorFrameConsumer* GetCompositorFrameConsumer() {
+    return static_cast<CompositorFrameConsumer*>(&render_thread_manager_);

[boliu, 2016/04/21 17:05:40]

huh? why does this need to be a cast?

[Tobias Sargeant, 2016/04/21 17:48:27]

Doesn't. Removed.

   }
 
  private:
   JavaObjectWeakGlobalRef java_ref_;
   RenderThreadManager render_thread_manager_;
-  BrowserViewRenderer* browser_view_renderer_;
 };
 
 bool RegisterAwGLFunctor(JNIEnv* env);