[Android WebView] Migrate the rendering code to a separate set of classes.

android_webview/browser/renderer_host/view_renderer_host.h

Index: android_webview/browser/renderer_host/view_renderer_host.h
diff --git a/android_webview/browser/renderer_host/view_renderer_host.h b/android_webview/browser/renderer_host/view_renderer_host.h
new file mode 100644
index 0000000000000000000000000000000000000000..b455ba21cb1a176b70c4093663f9ece16187e7f3
--- /dev/null
+++ b/android_webview/browser/renderer_host/view_renderer_host.h
@@ -0,0 +1,51 @@
+// Copyright (c) 2013 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 ANDROID_WEBVIEW_BROWSER_RENDER_HOST_VIEW_RENDERER_HOST_H_
+#define ANDROID_WEBVIEW_BROWSER_RENDER_HOST_VIEW_RENDERER_HOST_H_
+
+#include "base/threading/non_thread_safe.h"
+#include "content/public/browser/web_contents_observer.h"
+
+namespace android_webview {
+
+// Provides RenderViewHost wrapper functionality for sending WebView-specific
+// IPC messages to the renderer and from there to WebKit.
+class ViewRendererHost : public content::WebContentsObserver,
+                         public base::NonThreadSafe {
+ public:
+  class Client {
+   public:
+    virtual void OnPictureUpdated(int process_id, int render_view_id) = 0;
+
+   protected:
+    virtual ~Client() {}
+  };
+
+  ViewRendererHost(content::WebContents* contents, Client* client);
+  virtual ~ViewRendererHost();
+
+  // Captures the latest available picture pile synchronously.
+  void CapturePictureSync();
+
+  // Enables updating picture piles on every new frame.
+  // OnPictureUpdated is called when a new picture is available,
+  // stored by renderer id in RendererPictureMap.
+  void EnableCapturePictureCallback(bool enabled);
+
+  using content::WebContentsObserver::Observe;

[benm (inactive), 2013/01/21 20:34:49]

nit: Is this needed? Even if so, I don't think we are supposed to have 'using'
in headers (only .cc)

[Leandro Graciá Gil, 2013/01/22 08:18:46]

It's used to change the WebContents to observe in ViewRenderer::SetContents. We
had a similar issue in aw_render_view_ext.h (now in view_renderer_rv.h) to fix a
method visibility problem, and that was actually a solution proposed by joi.

+
+ private:
+  // content::WebContentsObserver implementation.
+  virtual void RenderViewGone(base::TerminationStatus status) OVERRIDE;
+  virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
+
+  void OnPictureUpdated();
+
+  Client* client_;
+};
+
+}  // namespace android_webview
+
+#endif  // ANDROID_WEBVIEW_BROWSER_RENDER_HOST_VIEW_RENDERER_HOST_H_