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

Side by Side Diff: webkit/blob/blob_url_request_job_factory.h

Issue 10855209: Refactoring: ProtocolHandler::MaybeCreateJob takes NetworkDelegate as argument (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Latest merge Created 8 years, 3 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
« no previous file with comments | « webkit/blob/blob_url_request_job.cc ('k') | webkit/blob/blob_url_request_job_factory.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) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 #ifndef WEBKIT_BLOB_BLOB_URL_REQUEST_JOB_FACTORY_H_ 5 #ifndef WEBKIT_BLOB_BLOB_URL_REQUEST_JOB_FACTORY_H_
6 #define WEBKIT_BLOB_BLOB_URL_REQUEST_JOB_FACTORY_H_ 6 #define WEBKIT_BLOB_BLOB_URL_REQUEST_JOB_FACTORY_H_
7 7
8 #include "base/compiler_specific.h" 8 #include "base/compiler_specific.h"
9 #include "base/memory/ref_counted.h" 9 #include "base/memory/ref_counted.h"
10 #include "net/url_request/url_request_job_factory.h" 10 #include "net/url_request/url_request_job_factory.h"
(...skipping 15 matching lines...) Expand all
26 class BLOB_EXPORT BlobProtocolHandler 26 class BLOB_EXPORT BlobProtocolHandler
27 : public net::URLRequestJobFactory::ProtocolHandler { 27 : public net::URLRequestJobFactory::ProtocolHandler {
28 public: 28 public:
29 // |controller|'s lifetime should exceed the lifetime of the ProtocolHandler. 29 // |controller|'s lifetime should exceed the lifetime of the ProtocolHandler.
30 explicit BlobProtocolHandler( 30 explicit BlobProtocolHandler(
31 BlobStorageController* blob_storage_controller, 31 BlobStorageController* blob_storage_controller,
32 base::MessageLoopProxy* file_loop_proxy); 32 base::MessageLoopProxy* file_loop_proxy);
33 virtual ~BlobProtocolHandler(); 33 virtual ~BlobProtocolHandler();
34 34
35 virtual net::URLRequestJob* MaybeCreateJob( 35 virtual net::URLRequestJob* MaybeCreateJob(
36 net::URLRequest* request) const OVERRIDE; 36 net::URLRequest* request,
37 net::NetworkDelegate* network_delegate) const OVERRIDE;
37 38
38 private: 39 private:
39 virtual scoped_refptr<BlobData> LookupBlobData( 40 virtual scoped_refptr<BlobData> LookupBlobData(
40 net::URLRequest* request) const; 41 net::URLRequest* request) const;
41 42
42 // No scoped_refptr because |blob_storage_controller_| is owned by the 43 // No scoped_refptr because |blob_storage_controller_| is owned by the
43 // ProfileIOData, which also owns this ProtocolHandler. 44 // ProfileIOData, which also owns this ProtocolHandler.
44 BlobStorageController* const blob_storage_controller_; 45 BlobStorageController* const blob_storage_controller_;
45 const scoped_refptr<base::MessageLoopProxy> file_loop_proxy_; 46 const scoped_refptr<base::MessageLoopProxy> file_loop_proxy_;
46 47
47 DISALLOW_COPY_AND_ASSIGN(BlobProtocolHandler); 48 DISALLOW_COPY_AND_ASSIGN(BlobProtocolHandler);
48 }; 49 };
49 50
50 } // namespace webkit_blob 51 } // namespace webkit_blob
51 52
52 #endif // WEBKIT_BLOB_BLOB_URL_REQUEST_JOB_FACTORY_H_ 53 #endif // WEBKIT_BLOB_BLOB_URL_REQUEST_JOB_FACTORY_H_
OLDNEW
« no previous file with comments | « webkit/blob/blob_url_request_job.cc ('k') | webkit/blob/blob_url_request_job_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698