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

android_webview/renderer/view_renderer_rv.h

+// 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_RENDERER_VIEW_RENDERER_RV_H_
+#define ANDROID_WEBVIEW_RENDERER_VIEW_RENDERER_RV_H_
+
+#include "content/public/renderer/render_view_observer.h"
+
+namespace android_webview {
+
+// Render-process side of ViewRendererHost.
+// Implements required interaction with content::RenderView.
+class ViewRendererRV : public content::RenderViewObserver {

[joth, 2013/01/21 22:54:24]

The RV in the name is non-obvious.
But BrowserViewRenderer and RendererViewRenderer are even more clumsy :)

ViewRendererHost and ViewRenderer seem OK to me.

We can bikeshed over whether the Aw prefix would help pigeon hole it better
too... 

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

Using ViewRenderer & ViewRendererHost + BrowserViewRenderer.

+ public:
+  static void RenderViewCreated(content::RenderView* render_view);
+
+  // Required to be public by IPC_MESSAGE_HANDLER_DELAY_REPLY.
+  using content::RenderViewObserver::Send;
+
+ private:
+  ViewRendererRV(content::RenderView* render_view);
+  virtual ~ViewRendererRV();
+
+  // content::RenderViewObserver implementation.
+  virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
+  virtual void DidCommit() OVERRIDE;
+
+  void OnEnableCapturePictureCallback(bool enable);
+  void OnCapturePictureSync(IPC::Message* reply_msg);
+
+  bool capture_picture_enabled_;
+
+  DISALLOW_COPY_AND_ASSIGN(ViewRendererRV);
+};
+
+}  // namespace android_webview
+
+#endif  // ANDROID_WEBVIEW_RENDERER_VIEW_RENDERER_RV_H_