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

Issues Search
    My Issues | Starred     Open | Closed | All

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

Issue 2831233004: predictors: Add resource type to manifest. (Closed)
Patch Set: Created 3 years, 8 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 side-by-side diff with in-line comments
Download patch
« chrome/browser/predictors/resource_prefetch_predictor_tables.cc ('K') | « chrome/browser/predictors/resource_prefetch_predictor_test_util.cc ('k') | components/precache/core/proto/precache.proto » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/predictors/resource_prefetch_predictor_unittest.cc
diff --git a/chrome/browser/predictors/resource_prefetch_predictor_unittest.cc b/chrome/browser/predictors/resource_prefetch_predictor_unittest.cc
index 763076926d1ef95a785eaee4ae3a0df0f0bec6cf..57a89f5a482596a729d865e4724688c7473eaf5f 100644
--- a/chrome/browser/predictors/resource_prefetch_predictor_unittest.cc
+++ b/chrome/browser/predictors/resource_prefetch_predictor_unittest.cc
@@ -432,14 +432,17 @@ void ResourcePrefetchPredictorTest::InitializeSampleData() {
{ // Manifest data.
precache::PrecacheManifest google = CreateManifestData(11);
- InitializePrecacheResource(google.add_resource(),
- "http://google.com/script.js", 0.5);
- InitializePrecacheResource(google.add_resource(),
- "http://static.google.com/style.css", 0.333);
+ InitializePrecacheResource(
+ google.add_resource(), "http://google.com/script.js", 0.5,
+ precache::PrecacheResource::RESOURCE_TYPE_SCRIPT);
+ InitializePrecacheResource(
+ google.add_resource(), "http://static.google.com/style.css", 0.333,
+ precache::PrecacheResource::RESOURCE_TYPE_STYLESHEET);
precache::PrecacheManifest facebook = CreateManifestData(12);
- InitializePrecacheResource(facebook.add_resource(),
- "http://fb.com/static.css", 0.99);
+ InitializePrecacheResource(
+ facebook.add_resource(), "http://fb.com/static.css", 0.99,
+ precache::PrecacheResource::RESOURCE_TYPE_STYLESHEET);
test_manifest_data_.insert(std::make_pair("google.com", google));
test_manifest_data_.insert(std::make_pair("facebook.com", facebook));
@@ -1097,9 +1100,11 @@ TEST_F(ResourcePrefetchPredictorTest, ManifestHostNotInDB) {
precache::PrecacheManifest manifest =
CreateManifestData(base::Time::Now().ToDoubleT());
InitializePrecacheResource(manifest.add_resource(),
- "http://cdn.google.com/script.js", 0.9);
- InitializePrecacheResource(manifest.add_resource(),
- "http://cdn.google.com/style.css", 0.75);
+ "http://cdn.google.com/script.js", 0.9,
+ precache::PrecacheResource::RESOURCE_TYPE_SCRIPT);
+ InitializePrecacheResource(
+ manifest.add_resource(), "http://cdn.google.com/style.css", 0.75,
+ precache::PrecacheResource::RESOURCE_TYPE_STYLESHEET);
EXPECT_CALL(*mock_tables_.get(), UpdateManifestData("google.com", manifest));
@@ -1122,7 +1127,8 @@ TEST_F(ResourcePrefetchPredictorTest, ManifestHostInDB) {
precache::PrecacheManifest manifest =
CreateManifestData(base::Time::Now().ToDoubleT());
InitializePrecacheResource(manifest.add_resource(),
- "http://google.com/image.jpg", 0.1);
+ "http://google.com/image.jpg", 0.1,
+ precache::PrecacheResource::RESOURCE_TYPE_IMAGE);
EXPECT_CALL(*mock_tables_.get(), UpdateManifestData("google.com", manifest));
@@ -1145,7 +1151,8 @@ TEST_F(ResourcePrefetchPredictorTest, ManifestHostNotInDBAndDBFull) {
precache::PrecacheManifest manifest =
CreateManifestData(base::Time::Now().ToDoubleT());
InitializePrecacheResource(manifest.add_resource(),
- "http://en.wikipedia.org/logo.png", 1.0);
+ "http://en.wikipedia.org/logo.png", 1.0,
+ precache::PrecacheResource::RESOURCE_TYPE_IMAGE);
EXPECT_CALL(*mock_tables_.get(),
DeleteManifestData(std::vector<std::string>({"google.com"})));
@@ -1160,9 +1167,11 @@ TEST_F(ResourcePrefetchPredictorTest, ManifestUnknownFieldsRemoved) {
precache::PrecacheManifest manifest =
CreateManifestData(base::Time::Now().ToDoubleT());
InitializePrecacheResource(manifest.add_resource(),
- "http://cdn.google.com/script.js", 0.9);
- InitializePrecacheResource(manifest.add_resource(),
- "http://cdn.google.com/style.css", 0.75);
+ "http://cdn.google.com/script.js", 0.9,
+ precache::PrecacheResource::RESOURCE_TYPE_SCRIPT);
+ InitializePrecacheResource(
+ manifest.add_resource(), "http://cdn.google.com/style.css", 0.75,
+ precache::PrecacheResource::RESOURCE_TYPE_STYLESHEET);
precache::PrecacheManifest manifest_with_unknown_fields(manifest);
manifest_with_unknown_fields.mutable_id()->mutable_unknown_fields()->append(
@@ -1189,9 +1198,11 @@ TEST_F(ResourcePrefetchPredictorTest, ManifestTooOld) {
precache::PrecacheManifest manifest =
CreateManifestData(old_time.ToDoubleT());
InitializePrecacheResource(manifest.add_resource(),
- "http://cdn.google.com/script.js", 0.9);
- InitializePrecacheResource(manifest.add_resource(),
- "http://cdn.google.com/style.css", 0.75);
+ "http://cdn.google.com/script.js", 0.9,
+ precache::PrecacheResource::RESOURCE_TYPE_SCRIPT);
+ InitializePrecacheResource(
+ manifest.add_resource(), "http://cdn.google.com/style.css", 0.75,
+ precache::PrecacheResource::RESOURCE_TYPE_STYLESHEET);
// No calls to DB should happen.
predictor_->OnManifestFetched("google.com", manifest);
@@ -1211,8 +1222,11 @@ TEST_F(ResourcePrefetchPredictorTest, ManifestUnusedRemoved) {
precache::PrecacheManifest manifest =
CreateManifestData(base::Time::Now().ToDoubleT());
- InitializePrecacheResource(manifest.add_resource(), script_url, 0.9);
- InitializePrecacheResource(manifest.add_resource(), style_url, 0.75);
+ InitializePrecacheResource(manifest.add_resource(), script_url, 0.9,
+ precache::PrecacheResource::RESOURCE_TYPE_SCRIPT);
+ InitializePrecacheResource(
+ manifest.add_resource(), style_url, 0.75,
+ precache::PrecacheResource::RESOURCE_TYPE_STYLESHEET);
InitializeExperiment(&manifest, internal::kUnusedRemovedExperiment,
{true, false});
@@ -1796,37 +1810,46 @@ TEST_F(ResourcePrefetchPredictorTest, PopulateFromManifest) {
precache::PrecacheManifest google =
CreateManifestData(base::Time::Now().ToDoubleT());
InitializePrecacheResource(google.add_resource(),
- "https://static.google.com/good", 0.9);
+ "https://static.google.com/good.js", 0.9,
+ precache::PrecacheResource::RESOURCE_TYPE_SCRIPT);
InitializePrecacheResource(google.add_resource(),
- "https://static.google.com/low_confidence", 0.6);
+ "https://static.google.com/versioned_removed", 0.8,
+ precache::PrecacheResource::RESOURCE_TYPE_SCRIPT);
InitializePrecacheResource(google.add_resource(),
- "https://static.google.com/versionned_removed",
- 0.8);
+ "https://static.google.com/unused_removed", 0.8,
+ precache::PrecacheResource::RESOURCE_TYPE_SCRIPT);
InitializePrecacheResource(google.add_resource(),
- "https://static.google.com/unused_removed", 0.8);
+ "https://static.google.com/no_store", 0.8,
+ precache::PrecacheResource::RESOURCE_TYPE_SCRIPT);
+ InitializePrecacheResource(
+ google.add_resource(), "https://static.google.com/good.css", 0.75,
+ precache::PrecacheResource::RESOURCE_TYPE_STYLESHEET);
InitializePrecacheResource(google.add_resource(),
- "https://static.google.com/no_store", 0.8);
+ "https://static.google.com/low_confidence", 0.6,
+ precache::PrecacheResource::RESOURCE_TYPE_SCRIPT);
InitializeExperiment(&google, internal::kVersionedRemovedExperiment,
- {true, true, false, true, true});
+ {true, false, true, true, true});
InitializeExperiment(&google, internal::kUnusedRemovedExperiment,
- {true, true, true, false, true});
+ {true, true, false, true, true});
InitializeExperiment(&google, internal::kNoStoreRemovedExperiment,
- {true, true, true, true, false});
+ {true, true, true, false, true});
// The data that's too old.
base::Time old_time = base::Time::Now() - base::TimeDelta::FromDays(7);
precache::PrecacheManifest facebook =
CreateManifestData(old_time.ToDoubleT());
InitializePrecacheResource(facebook.add_resource(),
- "https://static.facebook.com/good", 0.9);
+ "https://static.facebook.com/good", 0.9,
+ precache::PrecacheResource::RESOURCE_TYPE_SCRIPT);
predictor_->manifest_table_cache_->insert({"google.com", google});
predictor_->manifest_table_cache_->insert({"facebook.com", facebook});
std::vector<GURL> urls;
EXPECT_TRUE(predictor_->PopulateFromManifest("google.com", &urls));
- EXPECT_THAT(urls,
- UnorderedElementsAre(GURL("https://static.google.com/good")));
+ EXPECT_EQ(urls,
+ std::vector<GURL>({GURL("https://static.google.com/good.css"),
+ GURL("https://static.google.com/good.js")}));
urls.clear();
EXPECT_FALSE(predictor_->PopulateFromManifest("facebook.com", &urls));
@@ -1895,7 +1918,8 @@ TEST_F(ResourcePrefetchPredictorTest, GetPrefetchData) {
const std::string& resource_url = "https://static.google.com/resource";
precache::PrecacheManifest manifest =
CreateManifestData(base::Time::Now().ToDoubleT());
- InitializePrecacheResource(manifest.add_resource(), resource_url, 0.9);
+ InitializePrecacheResource(manifest.add_resource(), resource_url, 0.9,
+ precache::PrecacheResource::RESOURCE_TYPE_SCRIPT);
predictor_->manifest_table_cache_->insert({"google.com", manifest});
urls.clear();
« chrome/browser/predictors/resource_prefetch_predictor_tables.cc ('K') | « chrome/browser/predictors/resource_prefetch_predictor_test_util.cc ('k') | components/precache/core/proto/precache.proto » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698