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

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

Issue 2654663004: [Not for review] record detailed time breakdown of SW related requests.
Patch Set: add stream uma Created 3 years, 10 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_request_handler.cc
diff --git a/content/browser/service_worker/service_worker_request_handler.cc b/content/browser/service_worker/service_worker_request_handler.cc
index fe367b5cc0012104a7cab38288812cdfe77ca38f..8d1cbd6a3b1fb92d66cee7470bb1f1b2b464c165 100644
--- a/content/browser/service_worker/service_worker_request_handler.cc
+++ b/content/browser/service_worker/service_worker_request_handler.cc
@@ -66,12 +66,13 @@ void FinalizeHandlerInitialization(
ResourceType resource_type,
RequestContextType request_context_type,
RequestContextFrameType frame_type,
- scoped_refptr<ResourceRequestBodyImpl> body) {
+ scoped_refptr<ResourceRequestBodyImpl> body,
+ base::TimeTicks request_start = base::TimeTicks()) {
std::unique_ptr<ServiceWorkerRequestHandler> handler(
provider_host->CreateRequestHandler(
request_mode, credentials_mode, redirect_mode, resource_type,
request_context_type, frame_type, blob_storage_context->AsWeakPtr(),
- body, skip_service_worker));
+ body, skip_service_worker, request_start));
if (!handler)
return;
@@ -144,7 +145,8 @@ void ServiceWorkerRequestHandler::InitializeHandler(
ResourceType resource_type,
RequestContextType request_context_type,
RequestContextFrameType frame_type,
- scoped_refptr<ResourceRequestBodyImpl> body) {
+ scoped_refptr<ResourceRequestBodyImpl> body,
+ base::TimeTicks request_start) {
// Create the handler even for insecure HTTP since it's used in the
// case of redirect to HTTPS.
if (!request->url().SchemeIsHTTPOrHTTPS() &&
@@ -162,10 +164,10 @@ void ServiceWorkerRequestHandler::InitializeHandler(
if (!provider_host || !provider_host->IsContextAlive())
return;
- FinalizeHandlerInitialization(request, provider_host, blob_storage_context,
- skip_service_worker, request_mode,
- credentials_mode, redirect_mode, resource_type,
- request_context_type, frame_type, body);
+ FinalizeHandlerInitialization(
+ request, provider_host, blob_storage_context, skip_service_worker,
+ request_mode, credentials_mode, redirect_mode, resource_type,
+ request_context_type, frame_type, body, request_start);
}
ServiceWorkerRequestHandler* ServiceWorkerRequestHandler::GetHandler(

Powered by Google App Engine
This is Rietveld 408576698