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

Side by Side Diff: content/browser/loader/resource_handler.cc

Issue 2526983002: Refactor ResourceHandler API. (Closed)
Patch Set: Fix Created 4 years 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
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/browser/loader/resource_handler.h" 5 #include "content/browser/loader/resource_handler.h"
6 6
7 #include "content/browser/loader/resource_request_info_impl.h" 7 #include "content/browser/loader/resource_request_info_impl.h"
8 8
9 namespace content { 9 namespace content {
10 10
11 ResourceHandler::Delegate::Delegate() {}
12
13 ResourceHandler::Delegate::~Delegate() {}
14
15 void ResourceHandler::SetDelegate(Delegate* delegate) {
16 delegate_ = delegate;
17 }
18
11 ResourceHandler::~ResourceHandler() {} 19 ResourceHandler::~ResourceHandler() {}
12 20
13 ResourceHandler::ResourceHandler(net::URLRequest* request) 21 ResourceHandler::ResourceHandler(net::URLRequest* request)
14 : controller_(NULL), 22 : request_(request) {}
15 request_(request) { 23
24 std::unique_ptr<ResourceController> ResourceHandler::TakeController() {
25 DCHECK(controller_);
26
27 std::unique_ptr<ResourceController> controller;
28 std::swap(controller, controller_);
29 return controller;
16 } 30 }
17 31
18 void ResourceHandler::SetController(ResourceController* controller) { 32 void ResourceHandler::Resume() {
19 controller_ = controller; 33 TakeController()->Resume();
34 }
35
36 void ResourceHandler::Cancel() {
37 TakeController()->Cancel();
38 }
39
40 void ResourceHandler::CancelAndIgnore() {
41 TakeController()->CancelAndIgnore();
42 }
43
44 void ResourceHandler::CancelWithError(int error_code) {
45 TakeController()->CancelWithError(error_code);
46 }
47
48 void ResourceHandler::OutOfBandCancel(int error_code, bool tell_renderer) {
49 delegate_->OutOfBandCancel(error_code, tell_renderer);
20 } 50 }
21 51
22 ResourceRequestInfoImpl* ResourceHandler::GetRequestInfo() const { 52 ResourceRequestInfoImpl* ResourceHandler::GetRequestInfo() const {
23 return ResourceRequestInfoImpl::ForRequest(request_); 53 return ResourceRequestInfoImpl::ForRequest(request_);
24 } 54 }
25 55
26 int ResourceHandler::GetRequestID() const { 56 int ResourceHandler::GetRequestID() const {
27 return GetRequestInfo()->GetRequestID(); 57 return GetRequestInfo()->GetRequestID();
28 } 58 }
29 59
30 ResourceMessageFilter* ResourceHandler::GetFilter() const { 60 ResourceMessageFilter* ResourceHandler::GetFilter() const {
31 return GetRequestInfo()->requester_info()->filter(); 61 return GetRequestInfo()->requester_info()->filter();
32 } 62 }
33 63
34 } // namespace content 64 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698