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

Side by Side Diff: content/renderer/render_frame_proxy.cc

Issue 1750863002: Revert of Separate RenderView from RenderWidget, part 1: inheritance. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@renderwidgettest
Patch Set: Created 4 years, 9 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/renderer/render_frame_proxy.h" 5 #include "content/renderer/render_frame_proxy.h"
6 6
7 #include <stdint.h> 7 #include <stdint.h>
8 #include <map> 8 #include <map>
9 #include <utility> 9 #include <utility>
10 10
(...skipping 83 matching lines...) Expand 10 before | Expand all | Expand 10 after
94 RenderViewImpl* render_view = nullptr; 94 RenderViewImpl* render_view = nullptr;
95 RenderWidget* render_widget = nullptr; 95 RenderWidget* render_widget = nullptr;
96 blink::WebRemoteFrame* web_frame = nullptr; 96 blink::WebRemoteFrame* web_frame = nullptr;
97 97
98 if (!parent) { 98 if (!parent) {
99 // Create a top level WebRemoteFrame. 99 // Create a top level WebRemoteFrame.
100 render_view = RenderViewImpl::FromRoutingID(render_view_routing_id); 100 render_view = RenderViewImpl::FromRoutingID(render_view_routing_id);
101 web_frame = 101 web_frame =
102 blink::WebRemoteFrame::create(replicated_state.scope, proxy.get()); 102 blink::WebRemoteFrame::create(replicated_state.scope, proxy.get());
103 render_view->webview()->setMainFrame(web_frame); 103 render_view->webview()->setMainFrame(web_frame);
104 render_widget = render_view->GetWidget(); 104 render_widget = render_view;
105 } else { 105 } else {
106 // Create a frame under an existing parent. The parent is always expected 106 // Create a frame under an existing parent. The parent is always expected
107 // to be a RenderFrameProxy, because navigations initiated by local frames 107 // to be a RenderFrameProxy, because navigations initiated by local frames
108 // should not wind up here. 108 // should not wind up here.
109 109
110 web_frame = parent->web_frame()->createRemoteChild( 110 web_frame = parent->web_frame()->createRemoteChild(
111 replicated_state.scope, 111 replicated_state.scope,
112 blink::WebString::fromUTF8(replicated_state.name), 112 blink::WebString::fromUTF8(replicated_state.name),
113 replicated_state.sandbox_flags, proxy.get()); 113 replicated_state.sandbox_flags, proxy.get());
114 render_view = parent->render_view(); 114 render_view = parent->render_view();
(...skipping 361 matching lines...) Expand 10 before | Expand all | Expand 10 after
476 blink::WebLocalFrame* source) { 476 blink::WebLocalFrame* source) {
477 int source_routing_id = RenderFrameImpl::FromWebFrame(source)->GetRoutingID(); 477 int source_routing_id = RenderFrameImpl::FromWebFrame(source)->GetRoutingID();
478 Send(new FrameHostMsg_AdvanceFocus(routing_id_, type, source_routing_id)); 478 Send(new FrameHostMsg_AdvanceFocus(routing_id_, type, source_routing_id));
479 } 479 }
480 480
481 void RenderFrameProxy::frameFocused() { 481 void RenderFrameProxy::frameFocused() {
482 Send(new FrameHostMsg_FrameFocused(routing_id_)); 482 Send(new FrameHostMsg_FrameFocused(routing_id_));
483 } 483 }
484 484
485 } // namespace 485 } // namespace
OLDNEW
« no previous file with comments | « content/renderer/render_frame_impl_browsertest.cc ('k') | content/renderer/render_view_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698