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

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

Issue 2209963002: Record some foreign fetch registration metrics. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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 366 matching lines...) Expand 10 before | Expand all | Expand 10 after
377 status, SERVICE_WORKER_ERROR_MAX_VALUE); 377 status, SERVICE_WORKER_ERROR_MAX_VALUE);
378 } 378 }
379 } 379 }
380 380
381 void ServiceWorkerMetrics::RecordInstallEventStatus( 381 void ServiceWorkerMetrics::RecordInstallEventStatus(
382 ServiceWorkerStatusCode status) { 382 ServiceWorkerStatusCode status) {
383 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.InstallEventStatus", status, 383 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.InstallEventStatus", status,
384 SERVICE_WORKER_ERROR_MAX_VALUE); 384 SERVICE_WORKER_ERROR_MAX_VALUE);
385 } 385 }
386 386
387 void ServiceWorkerMetrics::RecordForeignFetchRegistrationCount(
388 size_t scope_count,
389 size_t origin_count) {
390 UMA_HISTOGRAM_COUNTS_100("ServiceWorker.ForeignFetch.SubScopeCount",
391 scope_count);
Mark P 2016/08/11 22:42:18 Why is the histogram "subscopecount" yet you're us
Marijn Kruisselbrink 2016/08/12 20:33:23 Changed it to consistently be "scope count", at le
392 if (scope_count > 0) {
393 // Only record number of origins if service worker registered for at least
394 // one foreign fetch scope.
395 UMA_HISTOGRAM_COUNTS_100("ServiceWorker.ForeignFetch.OriginCount",
396 origin_count);
397 }
398 }
399
387 void ServiceWorkerMetrics::RecordEventHandledRatio(EventType event, 400 void ServiceWorkerMetrics::RecordEventHandledRatio(EventType event,
388 size_t handled_events, 401 size_t handled_events,
389 size_t fired_events) { 402 size_t fired_events) {
390 if (!fired_events) 403 if (!fired_events)
391 return; 404 return;
392 EventHandledRatioType type = EVENT_HANDLED_SOME; 405 EventHandledRatioType type = EVENT_HANDLED_SOME;
393 if (fired_events == handled_events) 406 if (fired_events == handled_events)
394 type = EVENT_HANDLED_ALL; 407 type = EVENT_HANDLED_ALL;
395 else if (handled_events == 0) 408 else if (handled_events == 0)
396 type = EVENT_HANDLED_NONE; 409 type = EVENT_HANDLED_NONE;
(...skipping 278 matching lines...) Expand 10 before | Expand all | Expand 10 after
675 } else if (failure_count == 2) { 688 } else if (failure_count == 2) {
676 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.StartWorker.AfterFailureStreak_2", 689 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.StartWorker.AfterFailureStreak_2",
677 status, SERVICE_WORKER_ERROR_MAX_VALUE); 690 status, SERVICE_WORKER_ERROR_MAX_VALUE);
678 } else if (failure_count == 3) { 691 } else if (failure_count == 3) {
679 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.StartWorker.AfterFailureStreak_3", 692 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.StartWorker.AfterFailureStreak_3",
680 status, SERVICE_WORKER_ERROR_MAX_VALUE); 693 status, SERVICE_WORKER_ERROR_MAX_VALUE);
681 } 694 }
682 } 695 }
683 696
684 } // namespace content 697 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698