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

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

Issue 1815363002: Add RetainedRef uses where needed. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
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 (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_widget_helper.h" 5 #include "content/browser/renderer_host/render_widget_helper.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/bind_helpers.h" 8 #include "base/bind_helpers.h"
9 #include "base/lazy_instance.h" 9 #include "base/lazy_instance.h"
10 #include "base/posix/eintr_wrapper.h" 10 #include "base/posix/eintr_wrapper.h"
(...skipping 105 matching lines...) Expand 10 before | Expand all | Expand 10 after
116 // single frame at this point. These requests will be resumed either in 116 // single frame at this point. These requests will be resumed either in
117 // WebContentsImpl::CreateNewWindow or RenderFrameHost::Init. 117 // WebContentsImpl::CreateNewWindow or RenderFrameHost::Init.
118 resource_dispatcher_host_->BlockRequestsForRoute( 118 resource_dispatcher_host_->BlockRequestsForRoute(
119 GlobalFrameRoutingId(render_process_id_, *main_frame_route_id)); 119 GlobalFrameRoutingId(render_process_id_, *main_frame_route_id));
120 } 120 }
121 121
122 BrowserThread::PostTask( 122 BrowserThread::PostTask(
123 BrowserThread::UI, FROM_HERE, 123 BrowserThread::UI, FROM_HERE,
124 base::Bind(&RenderWidgetHelper::OnCreateWindowOnUI, this, params, 124 base::Bind(&RenderWidgetHelper::OnCreateWindowOnUI, this, params,
125 *route_id, *main_frame_route_id, *main_frame_widget_route_id, 125 *route_id, *main_frame_route_id, *main_frame_widget_route_id,
126 make_scoped_refptr(session_storage_namespace))); 126 base::RetainedRef(session_storage_namespace)));
127 } 127 }
128 128
129 void RenderWidgetHelper::OnCreateWindowOnUI( 129 void RenderWidgetHelper::OnCreateWindowOnUI(
130 const ViewHostMsg_CreateWindow_Params& params, 130 const ViewHostMsg_CreateWindow_Params& params,
131 int32_t route_id, 131 int32_t route_id,
132 int32_t main_frame_route_id, 132 int32_t main_frame_route_id,
133 int32_t main_frame_widget_route_id, 133 int32_t main_frame_widget_route_id,
134 SessionStorageNamespace* session_storage_namespace) { 134 SessionStorageNamespace* session_storage_namespace) {
135 RenderViewHostImpl* host = 135 RenderViewHostImpl* host =
136 RenderViewHostImpl::FromID(render_process_id_, params.opener_id); 136 RenderViewHostImpl::FromID(render_process_id_, params.opener_id);
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
169 169
170 void RenderWidgetHelper::OnCreateFullscreenWidgetOnUI(int32_t opener_id, 170 void RenderWidgetHelper::OnCreateFullscreenWidgetOnUI(int32_t opener_id,
171 int32_t route_id) { 171 int32_t route_id) {
172 RenderViewHostImpl* host = RenderViewHostImpl::FromID( 172 RenderViewHostImpl* host = RenderViewHostImpl::FromID(
173 render_process_id_, opener_id); 173 render_process_id_, opener_id);
174 if (host) 174 if (host)
175 host->CreateNewFullscreenWidget(route_id); 175 host->CreateNewFullscreenWidget(route_id);
176 } 176 }
177 177
178 } // namespace content 178 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698