Part 1: Refactor Android WebView graphics code

android_webview/browser/browser_view_renderer.h

Index: android_webview/browser/browser_view_renderer.h
diff --git a/android_webview/browser/browser_view_renderer.h b/android_webview/browser/browser_view_renderer.h
index 1fc5650a459c670f963282c545efd7111682d119..59a231fdaf571fa1532fe6fb01a020c148f17b56 100644
--- a/android_webview/browser/browser_view_renderer.h
+++ b/android_webview/browser/browser_view_renderer.h
@@ -61,12 +61,14 @@ class BrowserViewRenderer : public content::SynchronousCompositorClient,
 
   BrowserViewRenderer(
       BrowserViewRendererClient* client,
-      SharedRendererState* shared_renderer_state,
       content::WebContents* web_contents,
       const scoped_refptr<base::SingleThreadTaskRunner>& ui_task_runner);
 
   virtual ~BrowserViewRenderer();
 
+  SharedRendererState* GetSharedRendererState();
+  bool RequestDrawGL(jobject canvas, bool wait_for_completion);
+
   // Main handler for view drawing: performs a SW draw immediately, or sets up
   // a subsequent GL Draw (via BrowserViewRendererClient::RequestDrawGL) and
   // returns true. A false return value indicates nothing was or will be drawn.
@@ -188,7 +190,8 @@ class BrowserViewRenderer : public content::SynchronousCompositorClient,
   std::string ToString() const;
 
   BrowserViewRendererClient* client_;
-  SharedRendererState* shared_renderer_state_;
+  // BrowserViewRenderer owns SharedRendererState.

[boliu, 2014/10/24 22:12:01]

Don't need this, that's obvious.

[hush (inactive), 2014/10/24 22:30:15]

Done.

+  SharedRendererState shared_renderer_state_;
   content::WebContents* web_contents_;
   scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner_;