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

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

Issue 2052613003: Speculatively launch Service Workers on mouse/touch events. [3/5] (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: use OverrideThreadForMessage to call StartServiceWorkerForNavigationHinton UI thread Created 4 years, 5 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";
74 case ServiceWorkerMetrics::EventType::NUM_TYPES: 80 case ServiceWorkerMetrics::EventType::NUM_TYPES:
75 NOTREACHED() << static_cast<int>(event_type); 81 NOTREACHED() << static_cast<int>(event_type);
76 } 82 }
77 return "_UNKNOWN"; 83 return "_UNKNOWN";
78 } 84 }
79 85
80 std::string GetWorkerPreparationSuffix( 86 std::string GetWorkerPreparationSuffix(
81 EmbeddedWorkerStatus initial_worker_status, 87 EmbeddedWorkerStatus initial_worker_status,
82 ServiceWorkerMetrics::StartSituation start_situation) { 88 ServiceWorkerMetrics::StartSituation start_situation) {
83 switch (initial_worker_status) { 89 switch (initial_worker_status) {
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
187 case EventType::FETCH_SUB_RESOURCE: 193 case EventType::FETCH_SUB_RESOURCE:
188 return "Fetch Subresource"; 194 return "Fetch Subresource";
189 case EventType::UNKNOWN: 195 case EventType::UNKNOWN:
190 return "Unknown"; 196 return "Unknown";
191 case EventType::FOREIGN_FETCH: 197 case EventType::FOREIGN_FETCH:
192 return "Foreign Fetch"; 198 return "Foreign Fetch";
193 case EventType::FETCH_WAITUNTIL: 199 case EventType::FETCH_WAITUNTIL:
194 return "Fetch WaitUntil"; 200 return "Fetch WaitUntil";
195 case EventType::FOREIGN_FETCH_WAITUNTIL: 201 case EventType::FOREIGN_FETCH_WAITUNTIL:
196 return "Foreign Fetch WaitUntil"; 202 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 Unconfiemed";
nhiroki 2016/07/25 06:15:13 s/Unconfiemed/Unconfirmed/
horo 2016/07/25 08:24:27 Done.
207 case EventType::NAVIGATION_HINT_LINK_TAP_DOWN:
208 return "Navigation Hint Link Tap Down";
197 case EventType::NUM_TYPES: 209 case EventType::NUM_TYPES:
198 break; 210 break;
199 } 211 }
200 NOTREACHED() << "Got unexpected event type: " << static_cast<int>(event_type); 212 NOTREACHED() << "Got unexpected event type: " << static_cast<int>(event_type);
201 return "error"; 213 return "error";
202 } 214 }
203 215
204 bool ServiceWorkerMetrics::ShouldExcludeSiteFromHistogram(Site site) { 216 bool ServiceWorkerMetrics::ShouldExcludeSiteFromHistogram(Site site) {
205 return site == ServiceWorkerMetrics::Site::NEW_TAB_PAGE; 217 return site == ServiceWorkerMetrics::Site::NEW_TAB_PAGE;
206 } 218 }
(...skipping 247 matching lines...) Expand 10 before | Expand all | Expand 10 after
454 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.NotificationCloseEvent.Time", 466 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.NotificationCloseEvent.Time",
455 time); 467 time);
456 break; 468 break;
457 case EventType::PUSH: 469 case EventType::PUSH:
458 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.PushEvent.Time", time); 470 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.PushEvent.Time", time);
459 break; 471 break;
460 case EventType::MESSAGE: 472 case EventType::MESSAGE:
461 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.ExtendableMessageEvent.Time", 473 UMA_HISTOGRAM_MEDIUM_TIMES("ServiceWorker.ExtendableMessageEvent.Time",
462 time); 474 time);
463 break; 475 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:
464 480
465 case EventType::UNKNOWN: 481 case EventType::UNKNOWN:
466 case EventType::NUM_TYPES: 482 case EventType::NUM_TYPES:
467 NOTREACHED() << "Invalid event type"; 483 NOTREACHED() << "Invalid event type";
468 break; 484 break;
469 } 485 }
470 } 486 }
471 487
472 void ServiceWorkerMetrics::RecordFetchEventStatus( 488 void ServiceWorkerMetrics::RecordFetchEventStatus(
473 bool is_main_resource, 489 bool is_main_resource,
(...skipping 151 matching lines...) Expand 10 before | Expand all | Expand 10 after
625 } else if (failure_count == 2) { 641 } else if (failure_count == 2) {
626 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.StartWorker.AfterFailureStreak_2", 642 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.StartWorker.AfterFailureStreak_2",
627 status, SERVICE_WORKER_ERROR_MAX_VALUE); 643 status, SERVICE_WORKER_ERROR_MAX_VALUE);
628 } else if (failure_count == 3) { 644 } else if (failure_count == 3) {
629 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.StartWorker.AfterFailureStreak_3", 645 UMA_HISTOGRAM_ENUMERATION("ServiceWorker.StartWorker.AfterFailureStreak_3",
630 status, SERVICE_WORKER_ERROR_MAX_VALUE); 646 status, SERVICE_WORKER_ERROR_MAX_VALUE);
631 } 647 }
632 } 648 }
633 649
634 } // namespace content 650 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698