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

Side by Side Diff: content/public/browser/resource_request_details.cc

Issue 184873004: Update resource requests to remove redundant frame ID. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Update comments Created 6 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 | 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/public/browser/resource_request_details.h" 5 #include "content/public/browser/resource_request_details.h"
6 6
7 #include "content/browser/worker_host/worker_service_impl.h" 7 #include "content/browser/worker_host/worker_service_impl.h"
8 #include "content/public/browser/resource_request_info.h" 8 #include "content/public/browser/resource_request_info.h"
9 #include "net/url_request/url_request.h" 9 #include "net/url_request/url_request.h"
10 10
11 namespace content { 11 namespace content {
12 12
13 ResourceRequestDetails::ResourceRequestDetails(const net::URLRequest* request, 13 ResourceRequestDetails::ResourceRequestDetails(const net::URLRequest* request,
14 int cert_id) 14 int cert_id)
15 : url(request->url()), 15 : url(request->url()),
16 original_url(request->original_url()), 16 original_url(request->original_url()),
17 method(request->method()), 17 method(request->method()),
18 referrer(request->referrer()), 18 referrer(request->referrer()),
19 has_upload(request->has_upload()), 19 has_upload(request->has_upload()),
20 load_flags(request->load_flags()), 20 load_flags(request->load_flags()),
21 status(request->status()), 21 status(request->status()),
22 ssl_cert_id(cert_id), 22 ssl_cert_id(cert_id),
23 ssl_cert_status(request->ssl_info().cert_status), 23 ssl_cert_status(request->ssl_info().cert_status),
24 socket_address(request->GetSocketAddress()) { 24 socket_address(request->GetSocketAddress()) {
25 const ResourceRequestInfo* info = ResourceRequestInfo::ForRequest(request); 25 const ResourceRequestInfo* info = ResourceRequestInfo::ForRequest(request);
26 resource_type = info->GetResourceType(); 26 resource_type = info->GetResourceType();
27 frame_id = info->GetFrameID(); 27 render_frame_id = info->GetRenderFrameID();
28 28
29 // If request is from the worker process on behalf of a renderer, use 29 // If request is from the worker process on behalf of a renderer, use
30 // the renderer process id, since it consumes the notification response 30 // the renderer process id, since it consumes the notification response
31 // such as ssl state etc. 31 // such as ssl state etc.
32 // TODO(atwilson): need to notify all associated renderers in the case 32 // TODO(atwilson): need to notify all associated renderers in the case
33 // 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
34 // the first one (works for dedicated workers and shared workers with 34 // the first one (works for dedicated workers and shared workers with
35 // a single process). 35 // a single process).
36 int render_frame_id; 36 int worker_render_frame_id;
37 if (!WorkerServiceImpl::GetInstance()->GetRendererForWorker( 37 if (!WorkerServiceImpl::GetInstance()->GetRendererForWorker(
38 info->GetChildID(), &origin_child_id, &render_frame_id)) { 38 info->GetChildID(), &origin_child_id, &worker_render_frame_id)) {
39 origin_child_id = info->GetChildID(); 39 origin_child_id = info->GetChildID();
40 } 40 }
41 } 41 }
42 42
43 ResourceRequestDetails::~ResourceRequestDetails() {} 43 ResourceRequestDetails::~ResourceRequestDetails() {}
44 44
45 ResourceRedirectDetails::ResourceRedirectDetails(const net::URLRequest* request, 45 ResourceRedirectDetails::ResourceRedirectDetails(const net::URLRequest* request,
46 int cert_id, 46 int cert_id,
47 const GURL& new_url) 47 const GURL& new_url)
48 : ResourceRequestDetails(request, cert_id), 48 : ResourceRequestDetails(request, cert_id),
49 new_url(new_url) { 49 new_url(new_url) {
50 } 50 }
51 51
52 ResourceRedirectDetails::~ResourceRedirectDetails() {} 52 ResourceRedirectDetails::~ResourceRedirectDetails() {}
53 53
54 } // namespace content 54 } // namespace content
OLDNEW
« no previous file with comments | « content/public/browser/resource_request_details.h ('k') | content/public/browser/resource_request_info.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698