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

Unified 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/loader/resource_request_info_impl.cc
===================================================================
--- content/browser/loader/resource_request_info_impl.cc (revision 243184)
+++ content/browser/loader/resource_request_info_impl.cc (working copy)
@@ -198,31 +198,6 @@
return was_ignored_by_handler_;
}
-bool ResourceRequestInfoImpl::GetAssociatedRenderView(
- int* render_process_id,
- int* render_view_id) const {
- // If the request is from the worker process, find a content that owns the
- // worker.
- if (process_type_ == PROCESS_TYPE_WORKER) {
- // Need to display some related UI for this network request - pick an
- // arbitrary parent to do so.
- int unused;
- if (!WorkerServiceImpl::GetInstance()->GetRendererForWorker(
- child_id_, render_process_id, render_view_id, &unused)) {
- *render_process_id = -1;
- *render_view_id = -1;
- return false;
- }
- } else if (process_type_ == PROCESS_TYPE_PLUGIN) {
- *render_process_id = origin_pid_;
- *render_view_id = route_id_;
- } else {
- *render_process_id = child_id_;
- *render_view_id = route_id_;
- }
- return true;
-}
-
bool ResourceRequestInfoImpl::GetAssociatedRenderFrame(
int* render_process_id,
int* render_frame_id) const {
@@ -231,9 +206,8 @@
if (process_type_ == PROCESS_TYPE_WORKER) {
// Need to display some related UI for this network request - pick an
// arbitrary parent to do so.
- int unused;
if (!WorkerServiceImpl::GetInstance()->GetRendererForWorker(
- child_id_, render_process_id, &unused, render_frame_id)) {
+ child_id_, render_process_id, render_frame_id)) {
*render_process_id = -1;
*render_frame_id = -1;
return false;
« 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