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

Side by Side Diff: content/browser/service_worker/service_worker_metrics.cc

Issue 1264333002: First step to refactor ServiceWorkerVersion to make event dispatching more modular. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rockot nits Created 4 years, 11 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/service_worker/service_worker_metrics.h" 5 #include "content/browser/service_worker/service_worker_metrics.h"
6 6
7 #include "base/metrics/histogram_macros.h" 7 #include "base/metrics/histogram_macros.h"
8 #include "base/metrics/sparse_histogram.h" 8 #include "base/metrics/sparse_histogram.h"
9 #include "base/strings/string_util.h" 9 #include "base/strings/string_util.h"
10 #include "content/common/service_worker/service_worker_types.h" 10 #include "content/common/service_worker/service_worker_types.h"
(...skipping 150 matching lines...) Expand 10 before | Expand all | Expand 10 after
161 void ServiceWorkerMetrics::RecordEventHandledRatio(EventType event, 161 void ServiceWorkerMetrics::RecordEventHandledRatio(EventType event,
162 size_t handled_events, 162 size_t handled_events,
163 size_t fired_events) { 163 size_t fired_events) {
164 if (!fired_events) 164 if (!fired_events)
165 return; 165 return;
166 EventHandledRatioType type = EVENT_HANDLED_SOME; 166 EventHandledRatioType type = EVENT_HANDLED_SOME;
167 if (fired_events == handled_events) 167 if (fired_events == handled_events)
168 type = EVENT_HANDLED_ALL; 168 type = EVENT_HANDLED_ALL;
169 else if (handled_events == 0) 169 else if (handled_events == 0)
170 type = EVENT_HANDLED_NONE; 170 type = EVENT_HANDLED_NONE;
171
171 // For now Fetch is the only type that is recorded. 172 // For now Fetch is the only type that is recorded.
172 DCHECK_EQ(EVENT_TYPE_FETCH, event); 173 if (event != EventType::FETCH)
174 return;
173 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.EventHandledRatioType.Fetch", type, 175 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.EventHandledRatioType.Fetch", type,
174 NUM_EVENT_HANDLED_RATIO_TYPE); 176 NUM_EVENT_HANDLED_RATIO_TYPE);
175 } 177 }
176 178
179 void ServiceWorkerMetrics::RecordEventTimeout(EventType event) {
180 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.RequestTimeouts.Count",
181 static_cast<int>(event),
182 static_cast<int>(EventType::NUM_TYPES));
183 }
184
177 void ServiceWorkerMetrics::RecordFetchEventStatus( 185 void ServiceWorkerMetrics::RecordFetchEventStatus(
178 bool is_main_resource, 186 bool is_main_resource,
179 ServiceWorkerStatusCode status) { 187 ServiceWorkerStatusCode status) {
180 if (is_main_resource) { 188 if (is_main_resource) {
181 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.FetchEvent.MainResource.Status", 189 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.FetchEvent.MainResource.Status",
182 status, SERVICE_WORKER_ERROR_MAX_VALUE); 190 status, SERVICE_WORKER_ERROR_MAX_VALUE);
183 } else { 191 } else {
184 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.FetchEvent.Subresource.Status", 192 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.FetchEvent.Subresource.Status",
185 status, SERVICE_WORKER_ERROR_MAX_VALUE); 193 status, SERVICE_WORKER_ERROR_MAX_VALUE);
186 } 194 }
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
233 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.URLRequestJob.FallbackedRequestMode", 241 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.URLRequestJob.FallbackedRequestMode",
234 mode, FETCH_REQUEST_MODE_LAST + 1); 242 mode, FETCH_REQUEST_MODE_LAST + 1);
235 } 243 }
236 244
237 void ServiceWorkerMetrics::RecordTimeBetweenEvents( 245 void ServiceWorkerMetrics::RecordTimeBetweenEvents(
238 const base::TimeDelta& time) { 246 const base::TimeDelta& time) {
239 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.TimeBetweenEvents", time); 247 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.TimeBetweenEvents", time);
240 } 248 }
241 249
242 } // namespace content 250 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/service_worker/service_worker_metrics.h ('k') | content/browser/service_worker/service_worker_provider_host.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698