Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1818)

Unified Diff: content/renderer/render_frame_impl.cc

Issue 16032007: Create RenderFrame/RenderFrameHost for the main frame of a page. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixes based on initial feedback from Charlie. Created 7 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
new file mode 100644
index 0000000000000000000000000000000000000000..bd191e70d20c470e945959f5602d9bfaba40e893
--- /dev/null
+++ b/content/renderer/render_frame_impl.cc
@@ -0,0 +1,34 @@
+// 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.
+
+#include "content/renderer/render_frame_impl.h"
+
+#include "content/renderer/render_thread_impl.h"
+#include "content/renderer/render_view_impl.h"
+
+namespace content {
+
+RenderFrameImpl::RenderFrameImpl(RenderViewImpl* render_view, int routing_id)
+ : render_view_(render_view),
+ routing_id_(routing_id) {
+}
+
+RenderFrameImpl::~RenderFrameImpl() {
+}
+
+bool RenderFrameImpl::Send(IPC::Message* message) {
+ // If given a messsage without a routing ID, then assign our routing ID.
Charlie Reis 2013/05/30 18:26:59 Why aren't we going through the RenderView / Rende
nasko 2013/05/30 19:03:48 The reason I've done it is that a RenderFrame coul
+ if (message->routing_id() == MSG_ROUTING_NONE)
+ message->set_routing_id(routing_id_);
+
+ return RenderThread::Get()->Send(message);
+}
+
+bool RenderFrameImpl::OnMessageReceived(const IPC::Message &msg) {
+ // Pass the message up to the RenderView, until we have enough
+ // infrastructure to start processing messages in this object.
+ return render_view_->OnMessageReceived(msg);
+}
+
+} // namespace content

Powered by Google App Engine
This is Rietveld 408576698