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

Side by Side Diff: content/browser/renderer_host/render_view_host_impl.cc

Issue 1986643002: Track pending WebContents and widgets by (process_id, routing_id) pair. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@fullscreen-flash
Patch Set: Disable popup menu test for Mac/Android Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/browser/renderer_host/render_view_host_impl.h" 5 #include "content/browser/renderer_host/render_view_host_impl.h"
6 6
7 #include <set> 7 #include <set>
8 #include <string> 8 #include <string>
9 #include <utility> 9 #include <utility>
10 #include <vector> 10 #include <vector>
(...skipping 950 matching lines...) Expand 10 before | Expand all | Expand 10 after
961 } 961 }
962 962
963 void RenderViewHostImpl::CreateNewFullscreenWidget(int32_t route_id) { 963 void RenderViewHostImpl::CreateNewFullscreenWidget(int32_t route_id) {
964 delegate_->CreateNewFullscreenWidget(GetProcess()->GetID(), route_id); 964 delegate_->CreateNewFullscreenWidget(GetProcess()->GetID(), route_id);
965 } 965 }
966 966
967 void RenderViewHostImpl::OnShowView(int route_id, 967 void RenderViewHostImpl::OnShowView(int route_id,
968 WindowOpenDisposition disposition, 968 WindowOpenDisposition disposition,
969 const gfx::Rect& initial_rect, 969 const gfx::Rect& initial_rect,
970 bool user_gesture) { 970 bool user_gesture) {
971 delegate_->ShowCreatedWindow(route_id, disposition, initial_rect, 971 delegate_->ShowCreatedWindow(GetProcess()->GetID(), route_id, disposition,
972 user_gesture); 972 initial_rect, user_gesture);
973 Send(new ViewMsg_Move_ACK(route_id)); 973 Send(new ViewMsg_Move_ACK(route_id));
974 } 974 }
975 975
976 void RenderViewHostImpl::OnShowWidget(int route_id, 976 void RenderViewHostImpl::OnShowWidget(int route_id,
977 const gfx::Rect& initial_rect) { 977 const gfx::Rect& initial_rect) {
978 delegate_->ShowCreatedWidget(route_id, initial_rect); 978 delegate_->ShowCreatedWidget(GetProcess()->GetID(), route_id, initial_rect);
979 Send(new ViewMsg_Move_ACK(route_id)); 979 Send(new ViewMsg_Move_ACK(route_id));
980 } 980 }
981 981
982 void RenderViewHostImpl::OnShowFullscreenWidget(int route_id) { 982 void RenderViewHostImpl::OnShowFullscreenWidget(int route_id) {
983 delegate_->ShowCreatedFullscreenWidget(route_id); 983 delegate_->ShowCreatedFullscreenWidget(GetProcess()->GetID(), route_id);
984 Send(new ViewMsg_Move_ACK(route_id)); 984 Send(new ViewMsg_Move_ACK(route_id));
985 } 985 }
986 986
987 void RenderViewHostImpl::OnRenderProcessGone(int status, int exit_code) { 987 void RenderViewHostImpl::OnRenderProcessGone(int status, int exit_code) {
988 // Do nothing, otherwise RenderWidgetHostImpl will assume it is not a 988 // Do nothing, otherwise RenderWidgetHostImpl will assume it is not a
989 // RenderViewHostImpl and destroy itself. 989 // RenderViewHostImpl and destroy itself.
990 // TODO(nasko): Remove this hack once RenderViewHost and RenderWidgetHost are 990 // TODO(nasko): Remove this hack once RenderViewHost and RenderWidgetHost are
991 // decoupled. 991 // decoupled.
992 } 992 }
993 993
(...skipping 354 matching lines...) Expand 10 before | Expand all | Expand 10 after
1348 } else { 1348 } else {
1349 render_view_ready_on_process_launch_ = true; 1349 render_view_ready_on_process_launch_ = true;
1350 } 1350 }
1351 } 1351 }
1352 1352
1353 void RenderViewHostImpl::RenderViewReady() { 1353 void RenderViewHostImpl::RenderViewReady() {
1354 delegate_->RenderViewReady(this); 1354 delegate_->RenderViewReady(this);
1355 } 1355 }
1356 1356
1357 } // namespace content 1357 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/renderer_host/render_view_host_delegate.h ('k') | content/browser/site_per_process_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698