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

Side by Side Diff: content/browser/loader/resource_request_info_impl.cc

Issue 125513002: Finally remove ResourceRequestInfo::GetAssociatedRenderView. Also remove the shared worker plumbing… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 6 years, 11 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 | Annotate | Revision Log
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/loader/resource_request_info_impl.h" 5 #include "content/browser/loader/resource_request_info_impl.h"
6 6
7 #include "content/browser/loader/global_routing_id.h" 7 #include "content/browser/loader/global_routing_id.h"
8 #include "content/browser/loader/resource_message_filter.h" 8 #include "content/browser/loader/resource_message_filter.h"
9 #include "content/browser/worker_host/worker_service_impl.h" 9 #include "content/browser/worker_host/worker_service_impl.h"
10 #include "content/common/net/url_request_user_data.h" 10 #include "content/common/net/url_request_user_data.h"
(...skipping 180 matching lines...) Expand 10 before | Expand all | Expand 10 after
191 } 191 }
192 192
193 bool ResourceRequestInfoImpl::HasUserGesture() const { 193 bool ResourceRequestInfoImpl::HasUserGesture() const {
194 return has_user_gesture_; 194 return has_user_gesture_;
195 } 195 }
196 196
197 bool ResourceRequestInfoImpl::WasIgnoredByHandler() const { 197 bool ResourceRequestInfoImpl::WasIgnoredByHandler() const {
198 return was_ignored_by_handler_; 198 return was_ignored_by_handler_;
199 } 199 }
200 200
201 bool ResourceRequestInfoImpl::GetAssociatedRenderView(
202 int* render_process_id,
203 int* render_view_id) const {
204 // If the request is from the worker process, find a content that owns the
205 // worker.
206 if (process_type_ == PROCESS_TYPE_WORKER) {
207 // Need to display some related UI for this network request - pick an
208 // arbitrary parent to do so.
209 int unused;
210 if (!WorkerServiceImpl::GetInstance()->GetRendererForWorker(
211 child_id_, render_process_id, render_view_id, &unused)) {
212 *render_process_id = -1;
213 *render_view_id = -1;
214 return false;
215 }
216 } else if (process_type_ == PROCESS_TYPE_PLUGIN) {
217 *render_process_id = origin_pid_;
218 *render_view_id = route_id_;
219 } else {
220 *render_process_id = child_id_;
221 *render_view_id = route_id_;
222 }
223 return true;
224 }
225
226 bool ResourceRequestInfoImpl::GetAssociatedRenderFrame( 201 bool ResourceRequestInfoImpl::GetAssociatedRenderFrame(
227 int* render_process_id, 202 int* render_process_id,
228 int* render_frame_id) const { 203 int* render_frame_id) const {
229 // If the request is from the worker process, find a content that owns the 204 // If the request is from the worker process, find a content that owns the
230 // worker. 205 // worker.
231 if (process_type_ == PROCESS_TYPE_WORKER) { 206 if (process_type_ == PROCESS_TYPE_WORKER) {
232 // Need to display some related UI for this network request - pick an 207 // Need to display some related UI for this network request - pick an
233 // arbitrary parent to do so. 208 // arbitrary parent to do so.
234 int unused;
235 if (!WorkerServiceImpl::GetInstance()->GetRendererForWorker( 209 if (!WorkerServiceImpl::GetInstance()->GetRendererForWorker(
236 child_id_, render_process_id, &unused, render_frame_id)) { 210 child_id_, render_process_id, render_frame_id)) {
237 *render_process_id = -1; 211 *render_process_id = -1;
238 *render_frame_id = -1; 212 *render_frame_id = -1;
239 return false; 213 return false;
240 } 214 }
241 } else if (process_type_ == PROCESS_TYPE_PLUGIN) { 215 } else if (process_type_ == PROCESS_TYPE_PLUGIN) {
242 *render_process_id = origin_pid_; 216 *render_process_id = origin_pid_;
243 *render_frame_id = render_frame_id_; 217 *render_frame_id = render_frame_id_;
244 } else { 218 } else {
245 *render_process_id = child_id_; 219 *render_process_id = child_id_;
246 *render_frame_id = render_frame_id_; 220 *render_frame_id = render_frame_id_;
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
285 base::WeakPtr<ResourceMessageFilter> filter) { 259 base::WeakPtr<ResourceMessageFilter> filter) {
286 child_id_ = child_id; 260 child_id_ = child_id;
287 route_id_ = route_id; 261 route_id_ = route_id;
288 origin_pid_ = origin_pid; 262 origin_pid_ = origin_pid;
289 request_id_ = request_id; 263 request_id_ = request_id;
290 frame_id_ = frame_id; 264 frame_id_ = frame_id;
291 filter_ = filter; 265 filter_ = filter;
292 } 266 }
293 267
294 } // namespace content 268 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/loader/resource_request_info_impl.h ('k') | content/browser/worker_host/worker_document_set.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698