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

Side by Side Diff: chrome/browser/predictors/resource_prefetcher.cc

Issue 10817004: Adds speculative prefetching of resources. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Addressing Dominich's comments. Created 8 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
OLDNEW
(Empty)
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
3 // found in the LICENSE file.
4
5 #include "chrome/browser/predictors/resource_prefetcher.h"
6
7 #include "base/stl_util.h"
8 #include "chrome/browser/predictors/resource_prefetcher_manager.h"
9 #include "content/public/browser/browser_thread.h"
10 #include "net/base/io_buffer.h"
11
12 namespace {
13
14 // The size of the buffer used to read the resource.
15 static const size_t kResourceBufferSizeBytes = 50000;
16
17 } // namespace
18
19 namespace predictors {
20
21 ResourcePrefetcher::Request::Request(const GURL& i_resource_url)
22 : resource_url(i_resource_url),
23 prefetch_status(PREFETCH_STATUS_NOT_STARTED),
24 usage_status(USAGE_STATUS_NOT_REQUESTED) {
25 }
26
27 ResourcePrefetcher::Request::Request(const Request& other)
28 : resource_url(other.resource_url),
29 prefetch_status(other.prefetch_status),
30 usage_status(other.usage_status) {
31 }
32
33 ResourcePrefetcher::ResourcePrefetcher(
34 ResourcePrefetcherManager* manager,
35 const ResourcePrefetchPredictorConfig& config,
36 const NavigationID& navigation_id,
37 scoped_ptr<RequestVector> requests)
38 : state_(INITIALIZED),
39 manager_(manager),
40 config_(config),
41 navigation_id_(navigation_id),
42 request_vector_(requests.Pass()) {
43 CHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::IO));
44 DCHECK(request_vector_.get());
45
46 std::copy(request_vector_->begin(), request_vector_->end(),
47 std::back_inserter(request_queue_));
48 }
49
50 ResourcePrefetcher::~ResourcePrefetcher() {
51 // Delete any pending net::URLRequests.
52 STLDeleteContainerPairFirstPointers(inflight_requests_.begin(),
53 inflight_requests_.end());
54 }
55
56 void ResourcePrefetcher::Start() {
57 CHECK(CalledOnValidThread());
58
59 CHECK_EQ(state_, INITIALIZED);
60 state_ = RUNNING;
61
62 TryToLaunchPrefetchRequests();
63 }
64
65 void ResourcePrefetcher::Stop() {
66 CHECK(CalledOnValidThread());
67
68 if (state_ == FINISHED)
69 return;
70
71 state_ = STOPPED;
72 }
73
74 void ResourcePrefetcher::TryToLaunchPrefetchRequests() {
75 CHECK(state_ == RUNNING || state_ == STOPPED);
76
77 // Try to launch new requests if the state is RUNNING.
78 if (state_ == RUNNING) {
79 bool request_available = true;
80
81 // Loop through the requests while we are under the
82 // max_prefetches_inflight_per_host_per_navigation limit, looking for a URL
83 // for which the max_prefetches_inflight_per_host_per_navigation limit has
84 // not been reached. Try to launch as many requests as possible.
85 while ((static_cast<int>(inflight_requests_.size()) <
86 config_.max_prefetches_inflight_per_navigation) &&
dominich 2012/08/02 15:00:50 nit - one too many spaces indented here, i think.
Shishir 2012/08/02 22:06:54 Looks right. 4 spaces from 's'
dominich 2012/08/02 22:10:15 But the start of the line, for this, is the ( befo
87 request_available) {
88 std::list<Request*>::iterator request_it = request_queue_.begin();
89 for (; request_it != request_queue_.end(); ++request_it) {
90 const std::string& host = (*request_it)->resource_url.host();
91
92 std::map<std::string, int>::iterator host_it =
93 host_inflight_counts_.find(host);
94 if (host_it == host_inflight_counts_.end() ||
95 host_it->second <
96 config_.max_prefetches_inflight_per_host_per_navigation)
97 break;
98 }
99 request_available = request_it != request_queue_.end();
100
101 if (request_available) {
102 SendRequest(*request_it);
103 request_queue_.erase(request_it);
104 }
105 }
106 }
107
108 // If the inflight_requests_ is empty, we cant launch any more. Finish.
109 if (inflight_requests_.empty()) {
110 CHECK(host_inflight_counts_.empty());
111 CHECK(request_queue_.empty() || state_ == STOPPED);
112
113 state_ = FINISHED;
114 manager_->ResourcePrefetcherFinished(this, request_vector_.release());
115 }
116 }
117
118 void ResourcePrefetcher::SendRequest(Request* request) {
119 request->prefetch_status = Request::PREFETCH_STATUS_STARTED;
120
121 net::URLRequest* url_request =
122 new net::URLRequest(request->resource_url,
123 this,
124 manager_->GetURLRequestContext());
125 inflight_requests_[url_request] = request;
126 host_inflight_counts_[url_request->original_url().host()] += 1;
127
128 url_request->set_method("GET");
129 url_request->set_first_party_for_cookies(navigation_id_.main_frame_url);
130 url_request->set_referrer(navigation_id_.main_frame_url.spec());
131 url_request->set_priority(net::LOW);
132 url_request->Start();
133 }
134
135 void ResourcePrefetcher::FinishRequest(net::URLRequest* request,
136 Request::PrefetchStatus status) {
137 std::map<net::URLRequest*, Request*>::iterator request_it =
138 inflight_requests_.find(request);
139 CHECK(request_it != inflight_requests_.end());
140
141 const std::string host = request->original_url().host();
142 std::map<std::string, int>::iterator host_it = host_inflight_counts_.find(
143 host);
144 CHECK_GT(host_it->second, 0);
145 host_it->second -= 1;
146 if (host_it->second == 0)
147 host_inflight_counts_.erase(host);
148
149 request_it->second->prefetch_status = status;
150 inflight_requests_.erase(request_it);
151
152 delete request;
153
154 TryToLaunchPrefetchRequests();
155 }
156
157 void ResourcePrefetcher::ReadFullResponse(net::URLRequest* request) {
158 bool status = true;
159 while (status) {
160 int bytes_read = 0;
161 scoped_refptr<net::IOBuffer> buffer(new net::IOBuffer(
162 kResourceBufferSizeBytes));
163 status = request->Read(buffer, kResourceBufferSizeBytes, &bytes_read);
164
165 if (status) {
166 if (request->status().error()) {
167 FinishRequest(request, Request::PREFETCH_STATUS_FAILED);
168 return;
169 } else if (bytes_read == 0) {
170 if (request->was_cached())
171 FinishRequest(request, Request::PREFETCH_STATUS_FROM_CACHE);
172 else
173 FinishRequest(request, Request::PREFETCH_STATUS_FROM_NETWORK);
174 return;
175 }
176 }
177 }
178 }
179
180 void ResourcePrefetcher::OnReceivedRedirect(net::URLRequest* request,
181 const GURL& new_url,
182 bool* defer_redirect) {
183 FinishRequest(request, Request::PREFETCH_STATUS_REDIRECTED);
184 }
185
186 void ResourcePrefetcher::OnAuthRequired(net::URLRequest* request,
187 net::AuthChallengeInfo* auth_info) {
188 FinishRequest(request, Request::PREFETCH_STATUS_AUTH_REQUIRED);
189 }
190
191 void ResourcePrefetcher::OnCertificateRequested(
192 net::URLRequest* request,
193 net::SSLCertRequestInfo* cert_request_info) {
194 FinishRequest(request, Request::PREFETCH_STATUS_CERT_REQUIRED);
195 }
196
197 void ResourcePrefetcher::OnSSLCertificateError(net::URLRequest* request,
198 const net::SSLInfo& ssl_info,
199 bool fatal) {
200 FinishRequest(request, Request::PREFETCH_STATUS_CERT_ERROR);
201 }
202
203 void ResourcePrefetcher::OnResponseStarted(net::URLRequest* request) {
204 if (request->status().error()) {
205 FinishRequest(request, Request::PREFETCH_STATUS_FAILED);
206 return;
207 }
208
209 ReadFullResponse(request);
210 }
211
212 void ResourcePrefetcher::OnReadCompleted(net::URLRequest* request,
213 int bytes_read) {
214 if (request->status().error()) {
215 FinishRequest(request, Request::PREFETCH_STATUS_FAILED);
216 return;
217 }
218
219 ReadFullResponse(request);
220 }
221
222 } // namespace predictors
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698