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

Unified Diff: tools/metrics/histograms/histograms.xml

Side-by-side diff isn't available for this file because of its large size.
Issue 2607513002: Remove Render.Workers.MaxWorkerCountInRendererProcess UMA. (Closed)
Patch Set: Created 3 years, 11 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:
Download patch
« no previous file with comments | « content/public/browser/render_process_host.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 456e5f4a83e30f559265773946e25da076915bbf..a0185e9eaf5f4842178f06eec7d435c1db138d2c 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -51647,6 +51647,13 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</histogram>
<histogram name="Render.Workers.MaxWorkerCountInRendererProcess">
+ <obsolete>
+ Deprecated Dec 2016. This metric did not report the number of shared workers
+ correctly, since it used the RenderProcessHostImpl ref counts, which in the
+ shared worker case is more like a boolean flag (zero or one or maybe two)
+ than the number of shared workers. So its results can be thought of as
+ roughly the number of service workers only.
+ </obsolete>
<owner>kinuko@chromium.org</owner>
<summary>
Maximum number of workers (SharedWorker or ServiceWorker) that are
« no previous file with comments | « content/public/browser/render_process_host.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698