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

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

Issue 2181983002: Revert "Speculatively launch Service Workers on mouse/touch events. [3/5]" (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 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
64 case ServiceWorkerMetrics::EventType::FETCH_SUB_RESOURCE: 64 case ServiceWorkerMetrics::EventType::FETCH_SUB_RESOURCE:
65 return "_FETCH_SUB_RESOURCE"; 65 return "_FETCH_SUB_RESOURCE";
66 case ServiceWorkerMetrics::EventType::UNKNOWN: 66 case ServiceWorkerMetrics::EventType::UNKNOWN:
67 return "_UNKNOWN"; 67 return "_UNKNOWN";
68 case ServiceWorkerMetrics::EventType::FOREIGN_FETCH: 68 case ServiceWorkerMetrics::EventType::FOREIGN_FETCH:
69 return "_FOREIGN_FETCH"; 69 return "_FOREIGN_FETCH";
70 case ServiceWorkerMetrics::EventType::FETCH_WAITUNTIL: 70 case ServiceWorkerMetrics::EventType::FETCH_WAITUNTIL:
71 return "_FETCH_WAITUNTIL"; 71 return "_FETCH_WAITUNTIL";
72 case ServiceWorkerMetrics::EventType::FOREIGN_FETCH_WAITUNTIL: 72 case ServiceWorkerMetrics::EventType::FOREIGN_FETCH_WAITUNTIL:
73 return "_FOREIGN_FETCH_WAITUNTIL"; 73 return "_FOREIGN_FETCH_WAITUNTIL";
74 case ServiceWorkerMetrics::EventType::NAVIGATION_HINT_LINK_MOUSE_DOWN:
75 return "_NAVIGATION_HINT_LINK_MOUSE_DOWN";
76 case ServiceWorkerMetrics::EventType::NAVIGATION_HINT_LINK_TAP_UNCONFIRMED:
77 return "_NAVIGATION_HINT_LINK_TAP_UNCONFIRMED";
78 case ServiceWorkerMetrics::EventType::NAVIGATION_HINT_LINK_TAP_DOWN:
79 return "_NAVIGATION_HINT_LINK_TAP_DOWN";
80 case ServiceWorkerMetrics::EventType::NUM_TYPES: 74 case ServiceWorkerMetrics::EventType::NUM_TYPES:
81 NOTREACHED() << static_cast<int>(event_type); 75 NOTREACHED() << static_cast<int>(event_type);
82 } 76 }
83 return "_UNKNOWN"; 77 return "_UNKNOWN";
84 } 78 }
85 79
86 std::string GetWorkerPreparationSuffix( 80 std::string GetWorkerPreparationSuffix(
87 EmbeddedWorkerStatus initial_worker_status, 81 EmbeddedWorkerStatus initial_worker_status,
88 ServiceWorkerMetrics::StartSituation start_situation) { 82 ServiceWorkerMetrics::StartSituation start_situation) {
89 switch (initial_worker_status) { 83 switch (initial_worker_status) {
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
193 case EventType::FETCH_SUB_RESOURCE: 187 case EventType::FETCH_SUB_RESOURCE:
194 return "Fetch Subresource"; 188 return "Fetch Subresource";
195 case EventType::UNKNOWN: 189 case EventType::UNKNOWN:
196 return "Unknown"; 190 return "Unknown";
197 case EventType::FOREIGN_FETCH: 191 case EventType::FOREIGN_FETCH:
198 return "Foreign Fetch"; 192 return "Foreign Fetch";
199 case EventType::FETCH_WAITUNTIL: 193 case EventType::FETCH_WAITUNTIL:
200 return "Fetch WaitUntil"; 194 return "Fetch WaitUntil";
201 case EventType::FOREIGN_FETCH_WAITUNTIL: 195 case EventType::FOREIGN_FETCH_WAITUNTIL:
202 return "Foreign Fetch WaitUntil"; 196 return "Foreign Fetch WaitUntil";
203 case EventType::NAVIGATION_HINT_LINK_MOUSE_DOWN:
204 return "Navigation Hint Link Mouse Down";
205 case EventType::NAVIGATION_HINT_LINK_TAP_UNCONFIRMED:
206 return "Navigation Hint Link Tap Unconfirmed";
207 case EventType::NAVIGATION_HINT_LINK_TAP_DOWN:
208 return "Navigation Hint Link Tap Down";
209 case EventType::NUM_TYPES: 197 case EventType::NUM_TYPES:
210 break; 198 break;
211 } 199 }
212 NOTREACHED() << "Got unexpected event type: " << static_cast<int>(event_type); 200 NOTREACHED() << "Got unexpected event type: " << static_cast<int>(event_type);
213 return "error"; 201 return "error";
214 } 202 }
215 203
216 bool ServiceWorkerMetrics::ShouldExcludeSiteFromHistogram(Site site) { 204 bool ServiceWorkerMetrics::ShouldExcludeSiteFromHistogram(Site site) {
217 return site == ServiceWorkerMetrics::Site::NEW_TAB_PAGE; 205 return site == ServiceWorkerMetrics::Site::NEW_TAB_PAGE;
218 } 206 }
(...skipping 247 matching lines...) Expand 10 before | Expand all | Expand 10 after
466 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.NotificationCloseEvent.Time", 454 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.NotificationCloseEvent.Time",
467 time); 455 time);
468 break; 456 break;
469 case EventType::PUSH: 457 case EventType::PUSH:
470 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.PushEvent.Time", time); 458 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.PushEvent.Time", time);
471 break; 459 break;
472 case EventType::MESSAGE: 460 case EventType::MESSAGE:
473 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.ExtendableMessageEvent.Time", 461 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.ExtendableMessageEvent.Time",
474 time); 462 time);
475 break; 463 break;
476 // Those navigation hints should not be sent as request events.
477 case EventType::NAVIGATION_HINT_LINK_MOUSE_DOWN:
478 case EventType::NAVIGATION_HINT_LINK_TAP_UNCONFIRMED:
479 case EventType::NAVIGATION_HINT_LINK_TAP_DOWN:
480 464
481 case EventType::UNKNOWN: 465 case EventType::UNKNOWN:
482 case EventType::NUM_TYPES: 466 case EventType::NUM_TYPES:
483 NOTREACHED() << "Invalid event type"; 467 NOTREACHED() << "Invalid event type";
484 break; 468 break;
485 } 469 }
486 } 470 }
487 471
488 void ServiceWorkerMetrics::RecordFetchEventStatus( 472 void ServiceWorkerMetrics::RecordFetchEventStatus(
489 bool is_main_resource, 473 bool is_main_resource,
(...skipping 151 matching lines...) Expand 10 before | Expand all | Expand 10 after
641 } else if (failure_count == 2) { 625 } else if (failure_count == 2) {
642 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.StartWorker.AfterFailureStreak_2", 626 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.StartWorker.AfterFailureStreak_2",
643 status, SERVICE_WORKER_ERROR_MAX_VALUE); 627 status, SERVICE_WORKER_ERROR_MAX_VALUE);
644 } else if (failure_count == 3) { 628 } else if (failure_count == 3) {
645 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.StartWorker.AfterFailureStreak_3", 629 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.StartWorker.AfterFailureStreak_3",
646 status, SERVICE_WORKER_ERROR_MAX_VALUE); 630 status, SERVICE_WORKER_ERROR_MAX_VALUE);
647 } 631 }
648 } 632 }
649 633
650 } // namespace content 634 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/service_worker/service_worker_metrics.h ('k') | content/public/browser/service_worker_context.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698