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

Side by Side Diff: chrome/browser/renderer_host/resource_request_details.cc

Issue 6025009: Get rid of RenderViewHostDelegate::Resource and dispatch the IPC messages dir... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 12 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) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 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 "chrome/browser/renderer_host/resource_request_details.h" 5 #include "chrome/browser/renderer_host/resource_request_details.h"
6 6
7 #include "chrome/browser/renderer_host/resource_dispatcher_host.h"
8 #include "chrome/browser/renderer_host/resource_dispatcher_host_request_info.h"
7 #include "chrome/browser/worker_host/worker_service.h" 9 #include "chrome/browser/worker_host/worker_service.h"
8 10
9 ResourceRequestDetails::ResourceRequestDetails(const net::URLRequest* request, 11 ResourceRequestDetails::ResourceRequestDetails(const net::URLRequest* request,
10 int cert_id) 12 int cert_id)
11 : url_(request->url()), 13 : url_(request->url()),
12 original_url_(request->original_url()), 14 original_url_(request->original_url()),
13 method_(request->method()), 15 method_(request->method()),
14 referrer_(request->referrer()), 16 referrer_(request->referrer()),
15 has_upload_(request->has_upload()), 17 has_upload_(request->has_upload()),
16 load_flags_(request->load_flags()), 18 load_flags_(request->load_flags()),
(...skipping 12 matching lines...) Expand all
29 // such as ssl state etc. 31 // such as ssl state etc.
30 // TODO(atwilson): need to notify all associated renderers in the case 32 // TODO(atwilson): need to notify all associated renderers in the case
31 // of ssl state change (http://crbug.com/25357). For now, just notify 33 // of ssl state change (http://crbug.com/25357). For now, just notify
32 // the first one (works for dedicated workers and shared workers with 34 // the first one (works for dedicated workers and shared workers with
33 // a single process). 35 // a single process).
34 int temp; 36 int temp;
35 if (!WorkerService::GetInstance()->GetRendererForWorker( 37 if (!WorkerService::GetInstance()->GetRendererForWorker(
36 info->child_id(), &origin_child_id_, &temp)) { 38 info->child_id(), &origin_child_id_, &temp)) {
37 origin_child_id_ = info->child_id(); 39 origin_child_id_ = info->child_id();
38 } 40 }
41
42 ResourceDispatcherHost::RenderViewForRequest(
43 request, &render_process_id_, &render_view_id_);
39 } 44 }
40 45
41 ResourceRequestDetails::~ResourceRequestDetails() {} 46 ResourceRequestDetails::~ResourceRequestDetails() {}
42 47
43 ResourceRedirectDetails::ResourceRedirectDetails(const net::URLRequest* request, 48 ResourceRedirectDetails::ResourceRedirectDetails(const net::URLRequest* request,
44 int cert_id, 49 int cert_id,
45 const GURL& new_url) 50 const GURL& new_url)
46 : ResourceRequestDetails(request, cert_id), 51 : ResourceRequestDetails(request, cert_id),
47 new_url_(new_url) { 52 new_url_(new_url) {
48 } 53 }
49 54
50 ResourceRedirectDetails::~ResourceRedirectDetails() {} 55 ResourceRedirectDetails::~ResourceRedirectDetails() {}
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698