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

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

Issue 2027583002: service worker: Avoid starting up for activation during shutdown (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add unittest Created 4 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 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 <limits> 7 #include <limits>
8 #include <string> 8 #include <string>
9 9
10 #include "base/metrics/histogram_macros.h" 10 #include "base/metrics/histogram_macros.h"
(...skipping 294 matching lines...) Expand 10 before | Expand all | Expand 10 after
305 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.WorkerStopped", 305 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.WorkerStopped",
306 static_cast<int>(status), 306 static_cast<int>(status),
307 static_cast<int>(StopStatus::NUM_TYPES)); 307 static_cast<int>(StopStatus::NUM_TYPES));
308 } 308 }
309 309
310 void ServiceWorkerMetrics::RecordStopWorkerTime(base::TimeDelta time) { 310 void ServiceWorkerMetrics::RecordStopWorkerTime(base::TimeDelta time) {
311 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.StopWorker.Time", time); 311 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.StopWorker.Time", time);
312 } 312 }
313 313
314 void ServiceWorkerMetrics::RecordActivateEventStatus( 314 void ServiceWorkerMetrics::RecordActivateEventStatus(
315 ServiceWorkerStatusCode status) { 315 ServiceWorkerStatusCode status,
316 bool is_shutdown) {
316 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.ActivateEventStatus", status, 317 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.ActivateEventStatus", status,
317 SERVICE_WORKER_ERROR_MAX_VALUE); 318 SERVICE_WORKER_ERROR_MAX_VALUE);
319 if (is_shutdown) {
320 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.ActivateEventStatus_InShutdown",
321 status, SERVICE_WORKER_ERROR_MAX_VALUE);
322 } else {
323 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.ActivateEventStatus_NotInShutdown",
324 status, SERVICE_WORKER_ERROR_MAX_VALUE);
325 }
318 } 326 }
319 327
320 void ServiceWorkerMetrics::RecordInstallEventStatus( 328 void ServiceWorkerMetrics::RecordInstallEventStatus(
321 ServiceWorkerStatusCode status) { 329 ServiceWorkerStatusCode status) {
322 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.InstallEventStatus", status, 330 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.InstallEventStatus", status,
323 SERVICE_WORKER_ERROR_MAX_VALUE); 331 SERVICE_WORKER_ERROR_MAX_VALUE);
324 } 332 }
325 333
326 void ServiceWorkerMetrics::RecordEventHandledRatio(EventType event, 334 void ServiceWorkerMetrics::RecordEventHandledRatio(EventType event,
327 size_t handled_events, 335 size_t handled_events,
(...skipping 246 matching lines...) Expand 10 before | Expand all | Expand 10 after
574 } else if (failure_count == 2) { 582 } else if (failure_count == 2) {
575 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.StartWorker.AfterFailureStreak_2", 583 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.StartWorker.AfterFailureStreak_2",
576 status, SERVICE_WORKER_ERROR_MAX_VALUE); 584 status, SERVICE_WORKER_ERROR_MAX_VALUE);
577 } else if (failure_count == 3) { 585 } else if (failure_count == 3) {
578 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.StartWorker.AfterFailureStreak_3", 586 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.StartWorker.AfterFailureStreak_3",
579 status, SERVICE_WORKER_ERROR_MAX_VALUE); 587 status, SERVICE_WORKER_ERROR_MAX_VALUE);
580 } 588 }
581 } 589 }
582 590
583 } // namespace content 591 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698