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

Side by Side Diff: content/browser/loader/resource_dispatcher_host_impl.h

Issue 1285863003: ResourceScheduler: remove dependency on ResourceRequestInfo and request_id (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@lkgr
Patch Set: DCHECK that there isn't an existing ScheduledResourceRequest attached to the URLRequest. Created 5 years, 4 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
« no previous file with comments | « no previous file | content/browser/loader/resource_dispatcher_host_impl.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 // This is the browser side of the resource dispatcher, it receives requests 5 // This is the browser side of the resource dispatcher, it receives requests
6 // from the child process (i.e. [Renderer, Plugin, Worker]ProcessHost), and 6 // from the child process (i.e. [Renderer, Plugin, Worker]ProcessHost), and
7 // dispatches them to URLRequests. It then forwards the messages from the 7 // dispatches them to URLRequests. It then forwards the messages from the
8 // URLRequests back to the correct process for handling. 8 // URLRequests back to the correct process for handling.
9 // 9 //
10 // See http://dev.chromium.org/developers/design-documents/multi-process-resourc e-loading 10 // See http://dev.chromium.org/developers/design-documents/multi-process-resourc e-loading
(...skipping 21 matching lines...) Expand all
32 #include "content/common/content_export.h" 32 #include "content/common/content_export.h"
33 #include "content/common/resource_request_body.h" 33 #include "content/common/resource_request_body.h"
34 #include "content/public/browser/child_process_data.h" 34 #include "content/public/browser/child_process_data.h"
35 #include "content/public/browser/download_item.h" 35 #include "content/public/browser/download_item.h"
36 #include "content/public/browser/download_url_parameters.h" 36 #include "content/public/browser/download_url_parameters.h"
37 #include "content/public/browser/global_request_id.h" 37 #include "content/public/browser/global_request_id.h"
38 #include "content/public/browser/notification_types.h" 38 #include "content/public/browser/notification_types.h"
39 #include "content/public/browser/resource_dispatcher_host.h" 39 #include "content/public/browser/resource_dispatcher_host.h"
40 #include "content/public/common/resource_type.h" 40 #include "content/public/common/resource_type.h"
41 #include "ipc/ipc_message.h" 41 #include "ipc/ipc_message.h"
42 #include "net/base/request_priority.h"
42 #include "net/cookies/canonical_cookie.h" 43 #include "net/cookies/canonical_cookie.h"
43 #include "net/url_request/url_request.h" 44 #include "net/url_request/url_request.h"
44 45
45 class ResourceHandler; 46 class ResourceHandler;
46 struct ResourceHostMsg_Request; 47 struct ResourceHostMsg_Request;
47 48
48 namespace base { 49 namespace base {
49 class FilePath; 50 class FilePath;
50 } 51 }
51 52
(...skipping 402 matching lines...) Expand 10 before | Expand all | Expand 10 after
454 ResourceContext* resource_context, 455 ResourceContext* resource_context,
455 AppCacheService* appcache_service, 456 AppCacheService* appcache_service,
456 int child_id, 457 int child_id,
457 int route_id, 458 int route_id,
458 scoped_ptr<ResourceHandler> handler); 459 scoped_ptr<ResourceHandler> handler);
459 460
460 void OnDataDownloadedACK(int request_id); 461 void OnDataDownloadedACK(int request_id);
461 void OnUploadProgressACK(int request_id); 462 void OnUploadProgressACK(int request_id);
462 void OnCancelRequest(int request_id); 463 void OnCancelRequest(int request_id);
463 void OnReleaseDownloadedFile(int request_id); 464 void OnReleaseDownloadedFile(int request_id);
465 void OnDidChangePriority(int request_id,
466 net::RequestPriority new_priority,
467 int intra_priority_value);
464 468
465 // Creates ResourceRequestInfoImpl for a download or page save. 469 // Creates ResourceRequestInfoImpl for a download or page save.
466 // |download| should be true if the request is a file download. 470 // |download| should be true if the request is a file download.
467 ResourceRequestInfoImpl* CreateRequestInfo( 471 ResourceRequestInfoImpl* CreateRequestInfo(
468 int child_id, 472 int child_id,
469 int route_id, 473 int route_id,
470 bool download, 474 bool download,
471 ResourceContext* context); 475 ResourceContext* context);
472 476
473 // Relationship of resource being authenticated with the top level page. 477 // Relationship of resource being authenticated with the top level page.
(...skipping 116 matching lines...) Expand 10 before | Expand all | Expand 10 after
590 DelegateMap delegate_map_; 594 DelegateMap delegate_map_;
591 595
592 scoped_ptr<ResourceScheduler> scheduler_; 596 scoped_ptr<ResourceScheduler> scheduler_;
593 597
594 DISALLOW_COPY_AND_ASSIGN(ResourceDispatcherHostImpl); 598 DISALLOW_COPY_AND_ASSIGN(ResourceDispatcherHostImpl);
595 }; 599 };
596 600
597 } // namespace content 601 } // namespace content
598 602
599 #endif // CONTENT_BROWSER_LOADER_RESOURCE_DISPATCHER_HOST_IMPL_H_ 603 #endif // CONTENT_BROWSER_LOADER_RESOURCE_DISPATCHER_HOST_IMPL_H_
OLDNEW
« no previous file with comments | « no previous file | content/browser/loader/resource_dispatcher_host_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698