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

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

Issue 2767093004: Implement the BackgroundFetch{Fail,ed} Service Worker events (Closed)
Patch Set: forward declare the data view Created 3 years, 9 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 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
79 case ServiceWorkerMetrics::EventType::NAVIGATION_HINT_LINK_TAP_DOWN: 79 case ServiceWorkerMetrics::EventType::NAVIGATION_HINT_LINK_TAP_DOWN:
80 return "_NAVIGATION_HINT_LINK_TAP_DOWN"; 80 return "_NAVIGATION_HINT_LINK_TAP_DOWN";
81 case ServiceWorkerMetrics::EventType::EXTERNAL_REQUEST: 81 case ServiceWorkerMetrics::EventType::EXTERNAL_REQUEST:
82 return "_EXTERNAL_REQUEST"; 82 return "_EXTERNAL_REQUEST";
83 case ServiceWorkerMetrics::EventType::PAYMENT_REQUEST: 83 case ServiceWorkerMetrics::EventType::PAYMENT_REQUEST:
84 return "_PAYMENT_REQUEST"; 84 return "_PAYMENT_REQUEST";
85 case ServiceWorkerMetrics::EventType::BACKGROUND_FETCH_ABORT: 85 case ServiceWorkerMetrics::EventType::BACKGROUND_FETCH_ABORT:
86 return "_BACKGROUND_FETCH_ABORT"; 86 return "_BACKGROUND_FETCH_ABORT";
87 case ServiceWorkerMetrics::EventType::BACKGROUND_FETCH_CLICK: 87 case ServiceWorkerMetrics::EventType::BACKGROUND_FETCH_CLICK:
88 return "_BACKGROUND_FETCH_CLICK"; 88 return "_BACKGROUND_FETCH_CLICK";
89 case ServiceWorkerMetrics::EventType::BACKGROUND_FETCH_FAIL:
90 return "_BACKGROUND_FETCH_FAIL";
91 case ServiceWorkerMetrics::EventType::BACKGROUND_FETCHED:
92 return "_BACKGROUND_FETCHED";
89 case ServiceWorkerMetrics::EventType::NUM_TYPES: 93 case ServiceWorkerMetrics::EventType::NUM_TYPES:
90 NOTREACHED() << static_cast<int>(event_type); 94 NOTREACHED() << static_cast<int>(event_type);
91 } 95 }
92 return "_UNKNOWN"; 96 return "_UNKNOWN";
93 } 97 }
94 98
95 ServiceWorkerMetrics::WorkerPreparationType GetWorkerPreparationType( 99 ServiceWorkerMetrics::WorkerPreparationType GetWorkerPreparationType(
96 EmbeddedWorkerStatus initial_worker_status, 100 EmbeddedWorkerStatus initial_worker_status,
97 ServiceWorkerMetrics::StartSituation start_situation) { 101 ServiceWorkerMetrics::StartSituation start_situation) {
98 using Situation = ServiceWorkerMetrics::StartSituation; 102 using Situation = ServiceWorkerMetrics::StartSituation;
(...skipping 259 matching lines...) Expand 10 before | Expand all | Expand 10 after
358 case EventType::NAVIGATION_HINT_LINK_TAP_DOWN: 362 case EventType::NAVIGATION_HINT_LINK_TAP_DOWN:
359 return "Navigation Hint Link Tap Down"; 363 return "Navigation Hint Link Tap Down";
360 case EventType::EXTERNAL_REQUEST: 364 case EventType::EXTERNAL_REQUEST:
361 return "External Request"; 365 return "External Request";
362 case EventType::PAYMENT_REQUEST: 366 case EventType::PAYMENT_REQUEST:
363 return "Payment Request"; 367 return "Payment Request";
364 case EventType::BACKGROUND_FETCH_ABORT: 368 case EventType::BACKGROUND_FETCH_ABORT:
365 return "Background Fetch Abort"; 369 return "Background Fetch Abort";
366 case EventType::BACKGROUND_FETCH_CLICK: 370 case EventType::BACKGROUND_FETCH_CLICK:
367 return "Background Fetch Click"; 371 return "Background Fetch Click";
372 case EventType::BACKGROUND_FETCH_FAIL:
373 return "Background Fetch Fail";
374 case EventType::BACKGROUND_FETCHED:
375 return "Background Fetched";
368 case EventType::NUM_TYPES: 376 case EventType::NUM_TYPES:
369 break; 377 break;
370 } 378 }
371 NOTREACHED() << "Got unexpected event type: " << static_cast<int>(event_type); 379 NOTREACHED() << "Got unexpected event type: " << static_cast<int>(event_type);
372 return "error"; 380 return "error";
373 } 381 }
374 382
375 ServiceWorkerMetrics::Site ServiceWorkerMetrics::SiteFromURL(const GURL& url) { 383 ServiceWorkerMetrics::Site ServiceWorkerMetrics::SiteFromURL(const GURL& url) {
376 // UIThreadSearchTermsData::GoogleBaseURLValue() returns the google base 384 // UIThreadSearchTermsData::GoogleBaseURLValue() returns the google base
377 // URL, but not available in content layer. 385 // URL, but not available in content layer.
(...skipping 318 matching lines...) Expand 10 before | Expand all | Expand 10 after
696 time); 704 time);
697 break; 705 break;
698 case EventType::BACKGROUND_FETCH_ABORT: 706 case EventType::BACKGROUND_FETCH_ABORT:
699 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.BackgroundFetchAbortEvent.Time", 707 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.BackgroundFetchAbortEvent.Time",
700 time); 708 time);
701 break; 709 break;
702 case EventType::BACKGROUND_FETCH_CLICK: 710 case EventType::BACKGROUND_FETCH_CLICK:
703 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.BackgroundFetchClickEvent.Time", 711 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.BackgroundFetchClickEvent.Time",
704 time); 712 time);
705 break; 713 break;
714 case EventType::BACKGROUND_FETCH_FAIL:
715 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.BackgroundFetchFailEvent.Time",
716 time);
717 break;
718 case EventType::BACKGROUND_FETCHED:
719 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.BackgroundFetchedEvent.Time",
720 time);
721 break;
706 // Those navigation hints should not be sent as request events. 722 // Those navigation hints should not be sent as request events.
707 case EventType::NAVIGATION_HINT_LINK_MOUSE_DOWN: 723 case EventType::NAVIGATION_HINT_LINK_MOUSE_DOWN:
708 case EventType::NAVIGATION_HINT_LINK_TAP_UNCONFIRMED: 724 case EventType::NAVIGATION_HINT_LINK_TAP_UNCONFIRMED:
709 case EventType::NAVIGATION_HINT_LINK_TAP_DOWN: 725 case EventType::NAVIGATION_HINT_LINK_TAP_DOWN:
710 726
711 case EventType::UNKNOWN: 727 case EventType::UNKNOWN:
712 case EventType::NUM_TYPES: 728 case EventType::NUM_TYPES:
713 NOTREACHED() << "Invalid event type"; 729 NOTREACHED() << "Invalid event type";
714 break; 730 break;
715 } 731 }
(...skipping 287 matching lines...) Expand 10 before | Expand all | Expand 10 after
1003 constexpr base::TimeDelta kMax = base::TimeDelta::FromDays(1); 1019 constexpr base::TimeDelta kMax = base::TimeDelta::FromDays(1);
1004 // Set the bucket count to 50 since that is the recommended value for all 1020 // Set the bucket count to 50 since that is the recommended value for all
1005 // histograms. 1021 // histograms.
1006 const int kBucketCount = 50; 1022 const int kBucketCount = 50;
1007 1023
1008 UMA_HISTOGRAM_CUSTOM_TIMES("ServiceWorker.Runtime", time, kMin, kMax, 1024 UMA_HISTOGRAM_CUSTOM_TIMES("ServiceWorker.Runtime", time, kMin, kMax,
1009 kBucketCount); 1025 kBucketCount);
1010 } 1026 }
1011 1027
1012 } // namespace content 1028 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698