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

Unified Diff: chrome/browser/predictors/resource_prefetch_predictor_test_util.cc

Issue 2470143007: Revert of predictors: Basic browsertest checks predictor learning. (Closed)
Patch Set: Created 4 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/predictors/resource_prefetch_predictor_test_util.cc
diff --git a/chrome/browser/predictors/resource_prefetch_predictor_test_util.cc b/chrome/browser/predictors/resource_prefetch_predictor_test_util.cc
index 8048f66d0100e58e74e76403bef6a3d1aa3e3681..c07e16bf30255a8e1ec9723e3b4b2ee2f05170bb 100644
--- a/chrome/browser/predictors/resource_prefetch_predictor_test_util.cc
+++ b/chrome/browser/predictors/resource_prefetch_predictor_test_util.cc
@@ -58,52 +58,6 @@
data.set_primary_key(primary_key);
data.set_last_visit_time(last_visit_time);
return data;
-}
-
-NavigationID CreateNavigationID(int process_id,
- int render_frame_id,
- const std::string& main_frame_url) {
- NavigationID navigation_id(process_id, render_frame_id, GURL(main_frame_url));
- navigation_id.creation_time = base::TimeTicks::Now();
- return navigation_id;
-}
-
-PageRequestSummary CreatePageRequestSummary(
- const std::string& main_frame_url,
- const std::string& initial_url,
- const std::vector<URLRequestSummary>& subresource_requests) {
- GURL main_frame_gurl(main_frame_url);
- PageRequestSummary summary(main_frame_gurl);
- summary.initial_url = GURL(initial_url);
- summary.subresource_requests = subresource_requests;
- return summary;
-}
-
-URLRequestSummary CreateURLRequestSummary(int process_id,
- int render_frame_id,
- const std::string& main_frame_url,
- const std::string& resource_url,
- content::ResourceType resource_type,
- net::RequestPriority priority,
- const std::string& mime_type,
- bool was_cached,
- const std::string& redirect_url,
- bool has_validators,
- bool always_revalidate) {
- URLRequestSummary summary;
- summary.navigation_id =
- CreateNavigationID(process_id, render_frame_id, main_frame_url);
- summary.resource_url =
- resource_url.empty() ? GURL(main_frame_url) : GURL(resource_url);
- summary.resource_type = resource_type;
- summary.priority = priority;
- summary.mime_type = mime_type;
- summary.was_cached = was_cached;
- if (!redirect_url.empty())
- summary.redirect_url = GURL(redirect_url);
- summary.has_validators = has_validators;
- summary.always_revalidate = always_revalidate;
- return summary;
}
std::ostream& operator<<(std::ostream& os, const PrefetchData& data) {

Powered by Google App Engine
This is Rietveld 408576698