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

Side by Side Diff: content/child/resource_dispatcher.h

Issue 268423002: Fix WebURLLoaderImpl::Context leak if a pending request is canceled. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rephrase the comment Created 6 years, 7 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 // See http://dev.chromium.org/developers/design-documents/multi-process-resourc e-loading 5 // See http://dev.chromium.org/developers/design-documents/multi-process-resourc e-loading
6 6
7 #ifndef CONTENT_CHILD_RESOURCE_DISPATCHER_H_ 7 #ifndef CONTENT_CHILD_RESOURCE_DISPATCHER_H_
8 #define CONTENT_CHILD_RESOURCE_DISPATCHER_H_ 8 #define CONTENT_CHILD_RESOURCE_DISPATCHER_H_
9 9
10 #include <deque> 10 #include <deque>
(...skipping 95 matching lines...) Expand 10 before | Expand all | Expand 10 after
106 ~PendingRequestInfo(); 106 ~PendingRequestInfo();
107 107
108 RequestPeer* peer; 108 RequestPeer* peer;
109 ResourceType::Type resource_type; 109 ResourceType::Type resource_type;
110 // The PID of the original process which issued this request. This gets 110 // The PID of the original process which issued this request. This gets
111 // non-zero only for a request proxied by another renderer, particularly 111 // non-zero only for a request proxied by another renderer, particularly
112 // requests from plugins. 112 // requests from plugins.
113 int origin_pid; 113 int origin_pid;
114 MessageQueue deferred_message_queue; 114 MessageQueue deferred_message_queue;
115 bool is_deferred; 115 bool is_deferred;
116 bool is_canceled;
116 // Original requested url. 117 // Original requested url.
117 GURL url; 118 GURL url;
118 // The security origin of the frame that initiates this request. 119 // The security origin of the frame that initiates this request.
119 GURL frame_origin; 120 GURL frame_origin;
120 // The url of the latest response even in case of redirection. 121 // The url of the latest response even in case of redirection.
121 GURL response_url; 122 GURL response_url;
122 linked_ptr<IPC::Message> pending_redirect_message; 123 linked_ptr<IPC::Message> pending_redirect_message;
123 base::TimeTicks request_start; 124 base::TimeTicks request_start;
124 base::TimeTicks response_start; 125 base::TimeTicks response_start;
125 base::TimeTicks completion_time; 126 base::TimeTicks completion_time;
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
212 213
213 // IO thread timestamp for ongoing IPC message. 214 // IO thread timestamp for ongoing IPC message.
214 base::TimeTicks io_timestamp_; 215 base::TimeTicks io_timestamp_;
215 216
216 DISALLOW_COPY_AND_ASSIGN(ResourceDispatcher); 217 DISALLOW_COPY_AND_ASSIGN(ResourceDispatcher);
217 }; 218 };
218 219
219 } // namespace content 220 } // namespace content
220 221
221 #endif // CONTENT_CHILD_RESOURCE_DISPATCHER_H_ 222 #endif // CONTENT_CHILD_RESOURCE_DISPATCHER_H_
OLDNEW
« no previous file with comments | « no previous file | content/child/resource_dispatcher.cc » ('j') | content/child/resource_dispatcher.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698