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

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

Issue 1965013002: Implement headers attributes in ForeignFetchResponse (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: using instead of typedef Created 4 years, 7 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
Index: content/browser/service_worker/service_worker_response_info.cc
diff --git a/content/browser/service_worker/service_worker_response_info.cc b/content/browser/service_worker/service_worker_response_info.cc
index aafb657db99cb5048f4d3e9ce3d51ac73023de7b..520b80712b542d28e1442274af3ca42d578d4aa8 100644
--- a/content/browser/service_worker/service_worker_response_info.cc
+++ b/content/browser/service_worker/service_worker_response_info.cc
@@ -35,6 +35,8 @@ void ServiceWorkerResponseInfo::ResetDataForRequest(net::URLRequest* request) {
info->ResetData();
}
+ServiceWorkerResponseInfo::~ServiceWorkerResponseInfo() {}
+
void ServiceWorkerResponseInfo::GetExtraResponseInfo(
ResourceResponseInfo* response_info) const {
response_info->was_fetched_via_service_worker =
@@ -49,6 +51,7 @@ void ServiceWorkerResponseInfo::GetExtraResponseInfo(
response_info->service_worker_ready_time = service_worker_ready_time_;
response_info->is_in_cache_storage = response_is_in_cache_storage_;
response_info->cache_storage_cache_name = response_cache_storage_cache_name_;
+ response_info->cors_exposed_header_names = cors_exposed_header_names_;
}
void ServiceWorkerResponseInfo::OnPrepareToRestart(
@@ -71,13 +74,15 @@ void ServiceWorkerResponseInfo::OnStartCompleted(
base::TimeTicks service_worker_start_time,
base::TimeTicks service_worker_ready_time,
bool response_is_in_cache_storage,
- const std::string& response_cache_storage_cache_name) {
+ const std::string& response_cache_storage_cache_name,
+ const ServiceWorkerHeaderList& cors_exposed_header_names) {
was_fetched_via_service_worker_ = was_fetched_via_service_worker;
was_fallback_required_ = was_fallback_required;
original_url_via_service_worker_ = original_url_via_service_worker;
response_type_via_service_worker_ = response_type_via_service_worker;
response_is_in_cache_storage_ = response_is_in_cache_storage;
response_cache_storage_cache_name_ = response_cache_storage_cache_name;
+ cors_exposed_header_names_ = cors_exposed_header_names;
// Update times, if not already set by a previous Job.
if (service_worker_start_time_.is_null()) {
@@ -94,6 +99,7 @@ void ServiceWorkerResponseInfo::ResetData() {
blink::WebServiceWorkerResponseTypeDefault;
response_is_in_cache_storage_ = false;
response_cache_storage_cache_name_ = std::string();
+ cors_exposed_header_names_.clear();
}
ServiceWorkerResponseInfo::ServiceWorkerResponseInfo() {}

Powered by Google App Engine
This is Rietveld 408576698