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

Unified Diff: content/renderer/service_worker/service_worker_context_client.cc

Issue 2837943006: [WIP] Support PerformanceResourceTiming for Service Worker Navigation Preload (Closed)
Patch Set: clean up 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
Index: content/renderer/service_worker/service_worker_context_client.cc
diff --git a/content/renderer/service_worker/service_worker_context_client.cc b/content/renderer/service_worker/service_worker_context_client.cc
index ca6f42f1148bd1c4d67458b6bb33e4315cbe6606..b25f19517081fa60d9ebb16ab02c4043b55d56c7 100644
--- a/content/renderer/service_worker/service_worker_context_client.cc
+++ b/content/renderer/service_worker/service_worker_context_client.cc
@@ -421,7 +421,10 @@ class ServiceWorkerContextClient::NavigationPreloadRequest final
client->OnNavigationPreloadResponse(fetch_event_id_, std::move(response_),
nullptr);
// This will delete |this|.
- client->OnNavigationPreloadComplete(fetch_event_id_);
+ client->OnNavigationPreloadComplete(
+ fetch_event_id_, response_head.response_start,
+ response_head.encoded_data_length, 0 /* encoded_body_length */,
+ 0 /* decoded_body_length */);
}
void OnDataDownloaded(int64_t data_length,
@@ -484,7 +487,9 @@ class ServiceWorkerContextClient::NavigationPreloadRequest final
nullptr);
}
// This will delete |this|.
- client->OnNavigationPreloadComplete(fetch_event_id_);
+ client->OnNavigationPreloadComplete(
+ fetch_event_id_, status.completion_time, status.encoded_data_length,
+ status.encoded_body_length, status.decoded_body_length);
}
private:
@@ -1636,7 +1641,14 @@ void ServiceWorkerContextClient::OnNavigationPreloadError(
}
void ServiceWorkerContextClient::OnNavigationPreloadComplete(
- int fetch_event_id) {
+ int fetch_event_id,
+ base::TimeTicks completion_time,
+ int64_t encoded_data_length,
+ int64_t encoded_body_length,
+ int64_t decoded_body_length) {
+ proxy_->OnNavigationPreloadComplete(
+ fetch_event_id, (completion_time - base::TimeTicks()).InSecondsF(),
+ encoded_data_length, encoded_body_length, decoded_body_length);
context_->preload_requests.Remove(fetch_event_id);
}

Powered by Google App Engine
This is Rietveld 408576698