Remove Render.Workers.MaxWorkerCountInRendererProcess UMA.

Remove Render.Workers.MaxWorkerCountInRendererProcess UMA.

This UMA and code is misleading because the shared worker ref count in
RenderProcessHostImpl actually is not the number of running shared
workers. We've added comments since that clarify it.

If we want this UMA back we can add a new one later. I'll add current
stats to the bug.

BUG=403258
Committed: https://crrev.com/3616a4304d02fdaba982fc443ed8b908cf786e78
Cr-Commit-Position: refs/heads/master@{#440951}

[falken, 2016-12-27 08:05:10]

falken@chromium.org changed reviewers:
+ kinuko@chromium.org

[falken, 2016-12-27 08:05:10]

kinuko@ can you review this? This has bothered me a bit  since I learned about
the shared worker ref count as part of investigating 
https://bugs.chromium.org/p/chromium/issues/detail?id=636377 and
https://bugs.chromium.org/p/chromium/issues/detail?id=608049

[falken, 2016-12-27 08:11:33]

The CQ bit was checked by falken@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-27 08:11:48]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-27 09:23:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-27 09:23:08]

Dry run: This issue passed the CQ dry run.

[kinuko, 2016-12-27 15:39:17]

Yes, lgtm. Thanks for clarifying!

[falken, 2016-12-28 01:53:26]

falken@chromium.org changed reviewers:
+ holte@chromium.org

[falken, 2016-12-28 01:53:27]

holte: can you review histograms?

[falken, 2016-12-28 01:53:27]

holte: can you review histograms?

[Steven Holte, 2016-12-28 19:45:54]

histograms lgtm

[falken, 2016-12-29 10:58:04]

The CQ bit was checked by falken@chromium.org

[commit-bot: I haz the power, 2016-12-29 10:58:14]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-29 11:54:00]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1483009084219050, "parent_rev":
"03d1c4698b273a3ff37aced02a6a7f34e6867760", "commit_rev":
"1ce61f5e9461f940c3b847e03cba2eac567a72bb"}

[commit-bot: I haz the power, 2016-12-29 11:54:28]

Description was changed from

==========
Remove Render.Workers.MaxWorkerCountInRendererProcess UMA.

This UMA and code is misleading because the shared worker ref count in
RenderProcessHostImpl actually is not the number of running shared
workers. We've added comments since that clarify it.

If we want this UMA back we can add a new one later. I'll add current
stats to the bug.

BUG=403258
==========

to

==========
Remove Render.Workers.MaxWorkerCountInRendererProcess UMA.

This UMA and code is misleading because the shared worker ref count in
RenderProcessHostImpl actually is not the number of running shared
workers. We've added comments since that clarify it.

If we want this UMA back we can add a new one later. I'll add current
stats to the bug.

BUG=403258

Review-Url: https://codereview.chromium.org/2607513002
==========

[commit-bot: I haz the power, 2016-12-29 11:54:29]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2017-01-02 15:52:06]

Description was changed from

==========
Remove Render.Workers.MaxWorkerCountInRendererProcess UMA.

This UMA and code is misleading because the shared worker ref count in
RenderProcessHostImpl actually is not the number of running shared
workers. We've added comments since that clarify it.

If we want this UMA back we can add a new one later. I'll add current
stats to the bug.

BUG=403258

Review-Url: https://codereview.chromium.org/2607513002
==========

to

==========
Remove Render.Workers.MaxWorkerCountInRendererProcess UMA.

This UMA and code is misleading because the shared worker ref count in
RenderProcessHostImpl actually is not the number of running shared
workers. We've added comments since that clarify it.

If we want this UMA back we can add a new one later. I'll add current
stats to the bug.

BUG=403258

Committed: https://crrev.com/3616a4304d02fdaba982fc443ed8b908cf786e78
Cr-Commit-Position: refs/heads/master@{#440951}
==========

[commit-bot: I haz the power, 2017-01-02 15:52:08]

Patchset 1 (id:??) landed as
https://crrev.com/3616a4304d02fdaba982fc443ed8b908cf786e78
Cr-Commit-Position: refs/heads/master@{#440951}