Create RenderFrame/RenderFrameHost for the main frame of a page.

content/browser/renderer_host/render_frame_host_impl.h

Index: content/browser/renderer_host/render_frame_host_impl.h
diff --git a/content/browser/renderer_host/render_frame_host_impl.h b/content/browser/renderer_host/render_frame_host_impl.h
new file mode 100644
index 0000000000000000000000000000000000000000..8b6425493e4a40896db68408b7a4c6e5c9679c57
--- /dev/null
+++ b/content/browser/renderer_host/render_frame_host_impl.h
@@ -0,0 +1,48 @@
+// 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 CONTENT_BROWSER_RENDERER_HOST_RENDER_FRAME_HOST_IMPL_H_
+#define CONTENT_BROWSER_RENDERER_HOST_RENDER_FRAME_HOST_IMPL_H_
+
+#include "base/compiler_specific.h"
+#include "content/public/browser/render_frame_host.h"
+
+namespace content {
+
+class RenderViewHostImpl;
+
+

[Charlie Reis, 2013/05/30 00:41:53]

nit: Remove extra blank line.

[nasko, 2013/05/30 17:07:03]

Done.

+class CONTENT_EXPORT RenderFrameHostImpl : public RenderFrameHost {
+ public:
+  RenderFrameHostImpl(
+      RenderViewHostImpl* render_view_host,
+      int routing_id,
+      bool swapped_out);
+  virtual ~RenderFrameHostImpl();
+
+  // IPC::Sender
+  virtual bool Send(IPC::Message* msg) OVERRIDE;
+
+  // IPC::Listener
+  virtual bool OnMessageReceived(const IPC::Message& msg) OVERRIDE;
+
+  int routing_id() { return routing_id_; }
+
+ private:
+  friend class TestRenderViewHost;
+
+  RenderViewHostImpl* render_view_host_;
+
+  int routing_id_;
+
+  // Whether this RenderFrameHost is currently swapped out, such that the
+  // frame is being rendered by another process.
+  bool is_swapped_out_;

[Charlie Reis, 2013/05/30 00:41:53]

Do we need this yet?

[nasko, 2013/05/30 17:07:03]

Not yet, but it made the pattern of constructing the object very intuitive. If
you feel strongly this shouldn't be there yet, I'll remove it.

[Charlie Reis, 2013/05/30 18:26:59]

I just don't want people (myself included) to think we can start using this
value, since we're not updating it.  It's fine to pass it into the constructor
and do nothing with it, but let's not keep it as a member until we're ready to
keep it up to date.

[nasko, 2013/05/30 19:03:48]

Done.

+
+  DISALLOW_COPY_AND_ASSIGN(RenderFrameHostImpl);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_RENDERER_HOST_RENDER_FRAME_HOST_IMPL_H_