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

Side by Side Diff: chrome/browser/predictors/resource_prefetch_predictor.h

Issue 10827103: Speculative resource prefetching - fixing missing navigations due to server side redirects. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Addressing Dominich's comment. 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
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 #ifndef CHROME_BROWSER_PREDICTORS_RESOURCE_PREFETCH_PREDICTOR_H_ 5 #ifndef CHROME_BROWSER_PREDICTORS_RESOURCE_PREFETCH_PREDICTOR_H_
6 #define CHROME_BROWSER_PREDICTORS_RESOURCE_PREFETCH_PREDICTOR_H_ 6 #define CHROME_BROWSER_PREDICTORS_RESOURCE_PREFETCH_PREDICTOR_H_
7 7
8 #include <map> 8 #include <map>
9 #include <string> 9 #include <string>
10 #include <vector> 10 #include <vector>
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
92 URLRequestSummary(const URLRequestSummary& other); 92 URLRequestSummary(const URLRequestSummary& other);
93 ~URLRequestSummary(); 93 ~URLRequestSummary();
94 94
95 NavigationID navigation_id; 95 NavigationID navigation_id;
96 GURL resource_url; 96 GURL resource_url;
97 ResourceType::Type resource_type; 97 ResourceType::Type resource_type;
98 98
99 // Only for responses. 99 // Only for responses.
100 std::string mime_type; 100 std::string mime_type;
101 bool was_cached; 101 bool was_cached;
102 GURL redirect_url; // Only when the request was redirected.
dominich 2012/07/31 20:04:06 nit: 'Empty unless the request was redirected.'
Shishir 2012/07/31 20:06:13 Done.
102 }; 103 };
103 104
104 ResourcePrefetchPredictor(const Config& config, Profile* profile); 105 ResourcePrefetchPredictor(const Config& config, Profile* profile);
105 virtual ~ResourcePrefetchPredictor(); 106 virtual ~ResourcePrefetchPredictor();
106 107
107 // Thread safe. 108 // Thread safe.
108 static bool IsEnabled(Profile* profile); 109 static bool IsEnabled(Profile* profile);
109 static bool ShouldRecordRequest(net::URLRequest* request, 110 static bool ShouldRecordRequest(net::URLRequest* request,
110 ResourceType::Type resource_type); 111 ResourceType::Type resource_type);
111 static bool ShouldRecordResponse(net::URLRequest* response); 112 static bool ShouldRecordResponse(net::URLRequest* response);
(...skipping 95 matching lines...) Expand 10 before | Expand all | Expand 10 after
207 208
208 NavigationMap inflight_navigations_; 209 NavigationMap inflight_navigations_;
209 UrlTableCacheMap url_table_cache_; 210 UrlTableCacheMap url_table_cache_;
210 211
211 DISALLOW_COPY_AND_ASSIGN(ResourcePrefetchPredictor); 212 DISALLOW_COPY_AND_ASSIGN(ResourcePrefetchPredictor);
212 }; 213 };
213 214
214 } // namespace predictors 215 } // namespace predictors
215 216
216 #endif // CHROME_BROWSER_PREDICTORS_RESOURCE_PREFETCH_PREDICTOR_H_ 217 #endif // CHROME_BROWSER_PREDICTORS_RESOURCE_PREFETCH_PREDICTOR_H_
OLDNEW
« no previous file with comments | « chrome/browser/net/resource_prefetch_predictor_observer.cc ('k') | chrome/browser/predictors/resource_prefetch_predictor.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698