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

Unified Diff: content/browser/renderer_host/render_process_host_impl.cc

Issue 2607513002: Remove Render.Workers.MaxWorkerCountInRendererProcess UMA. (Closed)
Patch Set: Created 4 years 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/render_process_host_impl.cc
diff --git a/content/browser/renderer_host/render_process_host_impl.cc b/content/browser/renderer_host/render_process_host_impl.cc
index ef28915be20a5a70e87a2bf61413ac59a2212bab..18d86d268a0c4f6aafa2cda59e90f71cb0298f25 100644
--- a/content/browser/renderer_host/render_process_host_impl.cc
+++ b/content/browser/renderer_host/render_process_host_impl.cc
@@ -683,7 +683,6 @@ RenderProcessHostImpl::RenderProcessHostImpl(
#if BUILDFLAG(ENABLE_WEBRTC)
webrtc_eventlog_host_(id_),
#endif
- max_worker_count_(0),
permission_service_context_(new PermissionServiceContext(this)),
channel_connected_(false),
sent_render_process_ready_(false),
@@ -1388,8 +1387,6 @@ void RenderProcessHostImpl::IncrementServiceWorkerRefCount() {
DCHECK_CURRENTLY_ON(BrowserThread::UI);
DCHECK(!is_worker_ref_count_disabled_);
++service_worker_ref_count_;
- if (GetWorkerRefCount() > max_worker_count_)
- max_worker_count_ = GetWorkerRefCount();
}
void RenderProcessHostImpl::DecrementServiceWorkerRefCount() {
@@ -1405,8 +1402,6 @@ void RenderProcessHostImpl::IncrementSharedWorkerRefCount() {
DCHECK_CURRENTLY_ON(BrowserThread::UI);
DCHECK(!is_worker_ref_count_disabled_);
++shared_worker_ref_count_;
- if (GetWorkerRefCount() > max_worker_count_)
- max_worker_count_ = GetWorkerRefCount();
}
void RenderProcessHostImpl::DecrementSharedWorkerRefCount() {
@@ -2144,13 +2139,6 @@ void RenderProcessHostImpl::Cleanup() {
base::TimeTicks::Now() - survive_for_worker_start_time_);
}
- if (max_worker_count_ > 0) {
- // Record the max number of workers (SharedWorker or ServiceWorker)
- // that are simultaneously hosted in this renderer process.
- UMA_HISTOGRAM_COUNTS("Render.Workers.MaxWorkerCountInRendererProcess",
- max_worker_count_);
- }
-
// We cannot clean up twice; if this fails, there is an issue with our
// control flow.
DCHECK(!deleting_soon_);
« no previous file with comments | « content/browser/renderer_host/render_process_host_impl.h ('k') | content/public/browser/render_process_host.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698