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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: content/browser/service_worker/service_worker_database_unittest.cc

Issue 647953003: Service Worker script sizes in database. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git/+/master
Patch Set: Created 6 years, 2 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
« content/browser/service_worker/service_worker_database.cc ('K') | « content/browser/service_worker/service_worker_database.proto ('k') | content/browser/service_worker/service_worker_job_unittest.cc » ('j') | content/browser/service_worker/service_worker_script_cache_map.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/service_worker/service_worker_database_unittest.cc
diff --git a/content/browser/service_worker/service_worker_database_unittest.cc b/content/browser/service_worker/service_worker_database_unittest.cc
index e8fd82cb200450fd54bb58b1779f4e8e168b37c6..030ff5e023dba5a5cc6da5c4c19cb85b4dc02b53 100644
--- a/content/browser/service_worker/service_worker_database_unittest.cc
+++ b/content/browser/service_worker/service_worker_database_unittest.cc
@@ -37,12 +37,9 @@ GURL URL(const GURL& origin, const std::string& path) {
return out;
}
-Resource CreateResource(int64 resource_id, const GURL& url) {
+Resource CreateResource(int64 resource_id, const GURL& url, uint64 size_bytes) {
EXPECT_TRUE(url.is_valid());
- Resource resource;
- resource.resource_id = resource_id;
- resource.url = url;
- return resource;
+ return Resource(resource_id, url, size_bytes);
}
ServiceWorkerDatabase* CreateDatabase(const base::FilePath& path) {
@@ -62,6 +59,8 @@ void VerifyRegistrationData(const RegistrationData& expected,
EXPECT_EQ(expected.is_active, actual.is_active);
EXPECT_EQ(expected.has_fetch_handler, actual.has_fetch_handler);
EXPECT_EQ(expected.last_update_check, actual.last_update_check);
+ EXPECT_EQ(expected.resources_total_size_bytes,
+ actual.resources_total_size_bytes);
}
void VerifyResourceRecords(const std::vector<Resource>& expected,
@@ -70,6 +69,7 @@ void VerifyResourceRecords(const std::vector<Resource>& expected,
for (size_t i = 0; i < expected.size(); ++i) {
EXPECT_EQ(expected[i].resource_id, actual[i].resource_id);
EXPECT_EQ(expected[i].url, actual[i].url);
+ EXPECT_EQ(expected[i].size_bytes, actual[i].size_bytes);
}
}
@@ -465,10 +465,11 @@ TEST(ServiceWorkerDatabaseTest, Registration_Basic) {
data.scope = URL(origin, "/foo");
data.script = URL(origin, "/script.js");
data.version_id = 200;
+ data.resources_total_size_bytes = 10939 + 200;
std::vector<Resource> resources;
- resources.push_back(CreateResource(1, URL(origin, "/resource1")));
- resources.push_back(CreateResource(2, URL(origin, "/resource2")));
+ resources.push_back(CreateResource(1, URL(origin, "/resource1"), 10939));
+ resources.push_back(CreateResource(2, URL(origin, "/resource2"), 200));
// Write a resource to the uncommitted list to make sure that writing
// registration removes resource ids associated with the registration from
@@ -544,8 +545,8 @@ TEST(ServiceWorkerDatabaseTest, DeleteNonExistentRegistration) {
data.version_id = 200;
std::vector<Resource> resources;
- resources.push_back(CreateResource(1, URL(origin, "/resource1")));
- resources.push_back(CreateResource(2, URL(origin, "/resource2")));
+ resources.push_back(CreateResource(1, URL(origin, "/resource1"), 19));
+ resources.push_back(CreateResource(2, URL(origin, "/resource2"), 29129));
const int64 kNonExistentRegistrationId = 999;
const int64 kArbitraryVersionId = 222; // Used as a dummy initial value
@@ -593,8 +594,8 @@ TEST(ServiceWorkerDatabaseTest, Registration_Overwrite) {
data.version_id = 200;
std::vector<Resource> resources1;
- resources1.push_back(CreateResource(1, URL(origin, "/resource1")));
- resources1.push_back(CreateResource(2, URL(origin, "/resource2")));
+ resources1.push_back(CreateResource(1, URL(origin, "/resource1"), 10));
+ resources1.push_back(CreateResource(2, URL(origin, "/resource2"), 11));
int64 deleted_version_id = 222; // Dummy inital value
std::vector<int64> newly_purgeable_resources;
@@ -618,8 +619,8 @@ TEST(ServiceWorkerDatabaseTest, Registration_Overwrite) {
RegistrationData updated_data = data;
updated_data.version_id = data.version_id + 1;
std::vector<Resource> resources2;
- resources2.push_back(CreateResource(3, URL(origin, "/resource3")));
- resources2.push_back(CreateResource(4, URL(origin, "/resource4")));
+ resources2.push_back(CreateResource(3, URL(origin, "/resource3"), 12));
+ resources2.push_back(CreateResource(4, URL(origin, "/resource4"), 13));
EXPECT_EQ(ServiceWorkerDatabase::STATUS_OK,
database->WriteRegistration(updated_data,
@@ -662,8 +663,8 @@ TEST(ServiceWorkerDatabaseTest, Registration_Multiple) {
data1.version_id = 200;
std::vector<Resource> resources1;
- resources1.push_back(CreateResource(1, URL(origin, "/resource1")));
- resources1.push_back(CreateResource(2, URL(origin, "/resource2")));
+ resources1.push_back(CreateResource(1, URL(origin, "/resource1"), 1451));
+ resources1.push_back(CreateResource(2, URL(origin, "/resource2"), 15234));
EXPECT_EQ(
ServiceWorkerDatabase::STATUS_OK,
database->WriteRegistration(
@@ -677,8 +678,8 @@ TEST(ServiceWorkerDatabaseTest, Registration_Multiple) {
data2.version_id = 201;
std::vector<Resource> resources2;
- resources2.push_back(CreateResource(3, URL(origin, "/resource3")));
- resources2.push_back(CreateResource(4, URL(origin, "/resource4")));
+ resources2.push_back(CreateResource(3, URL(origin, "/resource3"), 5));
+ resources2.push_back(CreateResource(4, URL(origin, "/resource4"), 6));
EXPECT_EQ(
ServiceWorkerDatabase::STATUS_OK,
database->WriteRegistration(
@@ -959,8 +960,8 @@ TEST(ServiceWorkerDatabaseTest, DeleteAllDataForOrigin) {
data1.version_id = 100;
std::vector<Resource> resources1;
- resources1.push_back(CreateResource(1, URL(origin1, "/resource1")));
- resources1.push_back(CreateResource(2, URL(origin1, "/resource2")));
+ resources1.push_back(CreateResource(1, URL(origin1, "/resource1"), 2013));
+ resources1.push_back(CreateResource(2, URL(origin1, "/resource2"), 512));
ASSERT_EQ(
ServiceWorkerDatabase::STATUS_OK,
database->WriteRegistration(
@@ -973,8 +974,8 @@ TEST(ServiceWorkerDatabaseTest, DeleteAllDataForOrigin) {
data2.version_id = 101;
std::vector<Resource> resources2;
- resources2.push_back(CreateResource(3, URL(origin1, "/resource3")));
- resources2.push_back(CreateResource(4, URL(origin1, "/resource4")));
+ resources2.push_back(CreateResource(3, URL(origin1, "/resource3"), 4));
+ resources2.push_back(CreateResource(4, URL(origin1, "/resource4"), 5));
ASSERT_EQ(
ServiceWorkerDatabase::STATUS_OK,
database->WriteRegistration(
@@ -988,8 +989,8 @@ TEST(ServiceWorkerDatabaseTest, DeleteAllDataForOrigin) {
data3.version_id = 102;
std::vector<Resource> resources3;
- resources3.push_back(CreateResource(5, URL(origin2, "/resource5")));
- resources3.push_back(CreateResource(6, URL(origin2, "/resource6")));
+ resources3.push_back(CreateResource(5, URL(origin2, "/resource5"), 6));
+ resources3.push_back(CreateResource(6, URL(origin2, "/resource6"), 7));
ASSERT_EQ(
ServiceWorkerDatabase::STATUS_OK,
database->WriteRegistration(
« content/browser/service_worker/service_worker_database.cc ('K') | « content/browser/service_worker/service_worker_database.proto ('k') | content/browser/service_worker/service_worker_job_unittest.cc » ('j') | content/browser/service_worker/service_worker_script_cache_map.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698