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

Side by Side Diff: content/browser/loader/resource_loader.cc

Issue 1146253004: Add workerReady timing for ServiceWorker controlled requests [2/3] (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 6 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/loader/resource_loader.h" 5 #include "content/browser/loader/resource_loader.h"
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/message_loop/message_loop.h" 8 #include "base/message_loop/message_loop.h"
9 #include "base/metrics/histogram.h" 9 #include "base/metrics/histogram.h"
10 #include "base/profiler/scoped_tracker.h" 10 #include "base/profiler/scoped_tracker.h"
(...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
61 response->head.was_fetched_via_proxy = request->was_fetched_via_proxy(); 61 response->head.was_fetched_via_proxy = request->was_fetched_via_proxy();
62 response->head.proxy_server = response_info.proxy_server; 62 response->head.proxy_server = response_info.proxy_server;
63 response->head.socket_address = request->GetSocketAddress(); 63 response->head.socket_address = request->GetSocketAddress();
64 if (ServiceWorkerRequestHandler* handler = 64 if (ServiceWorkerRequestHandler* handler =
65 ServiceWorkerRequestHandler::GetHandler(request)) { 65 ServiceWorkerRequestHandler::GetHandler(request)) {
66 handler->GetExtraResponseInfo( 66 handler->GetExtraResponseInfo(
67 &response->head.was_fetched_via_service_worker, 67 &response->head.was_fetched_via_service_worker,
68 &response->head.was_fallback_required_by_service_worker, 68 &response->head.was_fallback_required_by_service_worker,
69 &response->head.original_url_via_service_worker, 69 &response->head.original_url_via_service_worker,
70 &response->head.response_type_via_service_worker, 70 &response->head.response_type_via_service_worker,
71 &response->head.service_worker_start_time); 71 &response->head.service_worker_start_time,
72 &response->head.service_worker_ready_time);
mmenke 2015/06/05 15:24:02 Hrm...Wonder if we should just pass in response, o
Kunihiko Sakamoto 2015/06/08 01:51:43 That makes sense, done.
72 } 73 }
73 AppCacheInterceptor::GetExtraResponseInfo( 74 AppCacheInterceptor::GetExtraResponseInfo(
74 request, 75 request,
75 &response->head.appcache_id, 76 &response->head.appcache_id,
76 &response->head.appcache_manifest_url); 77 &response->head.appcache_manifest_url);
77 if (info->is_load_timing_enabled()) 78 if (info->is_load_timing_enabled())
78 request->GetLoadTimingInfo(&response->head.load_timing); 79 request->GetLoadTimingInfo(&response->head.load_timing);
79 } 80 }
80 81
81 } // namespace 82 } // namespace
(...skipping 684 matching lines...) Expand 10 before | Expand all | Expand 10 after
766 case net::URLRequestStatus::FAILED: 767 case net::URLRequestStatus::FAILED:
767 status = STATUS_UNDEFINED; 768 status = STATUS_UNDEFINED;
768 break; 769 break;
769 } 770 }
770 771
771 UMA_HISTOGRAM_ENUMERATION("Net.Prefetch.Pattern", status, STATUS_MAX); 772 UMA_HISTOGRAM_ENUMERATION("Net.Prefetch.Pattern", status, STATUS_MAX);
772 } 773 }
773 } 774 }
774 775
775 } // namespace content 776 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698