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

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

Issue 2250623003: Keep ServiceWorkerMetrics::Site in ServiceWorkerVersion (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase 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_url_request_job.h" 5 #include "content/browser/service_worker/service_worker_url_request_job.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <limits> 10 #include <limits>
(...skipping 661 matching lines...) Expand 10 before | Expand all | Expand 10 after
672 // 3) The request is for New Tab Page. This is because it tends to dominate 672 // 3) The request is for New Tab Page. This is because it tends to dominate
673 // the stats and makes the results largely skewed. 673 // the stats and makes the results largely skewed.
674 if (resource_type_ != RESOURCE_TYPE_MAIN_FRAME) 674 if (resource_type_ != RESOURCE_TYPE_MAIN_FRAME)
675 return; 675 return;
676 if (!worker_already_activated_) 676 if (!worker_already_activated_)
677 return; 677 return;
678 if (version->skip_recording_startup_time() && 678 if (version->skip_recording_startup_time() &&
679 initial_worker_status_ != EmbeddedWorkerStatus::RUNNING) { 679 initial_worker_status_ != EmbeddedWorkerStatus::RUNNING) {
680 return; 680 return;
681 } 681 }
682 if (ServiceWorkerMetrics::ShouldExcludeURLFromHistogram(request()->url())) 682 if (version->should_exclude_from_uma())
683 return; 683 return;
684 ServiceWorkerMetrics::RecordActivatedWorkerPreparationTimeForMainFrame( 684 ServiceWorkerMetrics::RecordActivatedWorkerPreparationTimeForMainFrame(
685 worker_ready_time_ - request()->creation_time(), initial_worker_status_, 685 worker_ready_time_ - request()->creation_time(), initial_worker_status_,
686 version->embedded_worker()->start_situation()); 686 version->embedded_worker()->start_situation());
687 } 687 }
688 688
689 void ServiceWorkerURLRequestJob::DidDispatchFetchEvent( 689 void ServiceWorkerURLRequestJob::DidDispatchFetchEvent(
690 ServiceWorkerStatusCode status, 690 ServiceWorkerStatusCode status,
691 ServiceWorkerFetchEventResult fetch_result, 691 ServiceWorkerFetchEventResult fetch_result,
692 const ServiceWorkerResponse& response, 692 const ServiceWorkerResponse& response,
(...skipping 347 matching lines...) Expand 10 before | Expand all | Expand 10 after
1040 CreateFetchRequest(), active_worker, resource_type_, request()->net_log(), 1040 CreateFetchRequest(), active_worker, resource_type_, request()->net_log(),
1041 base::Bind(&ServiceWorkerURLRequestJob::DidPrepareFetchEvent, 1041 base::Bind(&ServiceWorkerURLRequestJob::DidPrepareFetchEvent,
1042 weak_factory_.GetWeakPtr(), active_worker), 1042 weak_factory_.GetWeakPtr(), active_worker),
1043 base::Bind(&ServiceWorkerURLRequestJob::DidDispatchFetchEvent, 1043 base::Bind(&ServiceWorkerURLRequestJob::DidDispatchFetchEvent,
1044 weak_factory_.GetWeakPtr()))); 1044 weak_factory_.GetWeakPtr())));
1045 worker_start_time_ = base::TimeTicks::Now(); 1045 worker_start_time_ = base::TimeTicks::Now();
1046 fetch_dispatcher_->Run(); 1046 fetch_dispatcher_->Run();
1047 } 1047 }
1048 1048
1049 } // namespace content 1049 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/service_worker/service_worker_metrics.cc ('k') | content/browser/service_worker/service_worker_version.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698