Revert of Check that all shared workers are terminated.

Revert of Check that all shared workers are terminated. (patchset #1 id:1 of https://codereview.chromium.org/2222063002/ )

Reason for revert:
Reverting temporary instrumentation following investigation

We want to get this in prior to dev branch.

BUG=636377
Original issue's description:
> Check that all shared workers are terminated.
>
> Adds temporary checks to determine whether shared workers
> prevent RenderProcessHost(s) from cleaning up due to
> RenderProcessHostImpl::worker_ref_count_ > 0.
>
> BUG=608049
>
> Committed: https://crrev.com/529173d1bbeb53b3f9438648fad264b4db042721
> Cr-Commit-Position: refs/heads/master@{#410730}

TBR=atwilson@chromium.org,jam@chromium.org,horo@chromium.org,alokp@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=608049
Committed: https://crrev.com/463ac7e96bf55b106225481811b0db752284bbed
Cr-Commit-Position: refs/heads/master@{#412660}

[manzagop (departed), 2016-08-17 20:13:16]

The CQ bit was checked by manzagop@chromium.org

[manzagop (departed), 2016-08-17 20:13:16]

Created Revert of Check that all shared workers are terminated.

[commit-bot: I haz the power, 2016-08-17 20:13:41]

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-08-17 20:13:42]

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

[commit-bot: I haz the power, 2016-08-17 20:13:43]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[erikchen, 2016-08-17 20:19:20]

erikchen@chromium.org changed reviewers:
+ erikchen@chromium.org

[erikchen, 2016-08-17 20:19:21]

lgtm rubber stamp

[manzagop (departed), 2016-08-17 20:39:03]

The CQ bit was checked by manzagop@chromium.org

[commit-bot: I haz the power, 2016-08-17 20:39:43]

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-08-17 20:42:14]

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

[commit-bot: I haz the power, 2016-08-17 20:42:15]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[manzagop (departed), 2016-08-17 20:58:37]

The patch failed to apply so I had to manually recreate it.

[manzagop (departed), 2016-08-17 20:58:43]

The CQ bit was checked by manzagop@chromium.org

[manzagop (departed), 2016-08-17 20:58:44]

The patchset sent to the CQ was uploaded after l-g-t-m from
erikchen@chromium.org
Link to the patchset: https://codereview.chromium.org/2254963002/#ps170001
(title: "Manually create the revert CL")

[commit-bot: I haz the power, 2016-08-17 20:59:34]

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-08-17 22:11:06]

Description was changed from

==========
Revert of Check that all shared workers are terminated. (patchset #1 id:1 of
https://codereview.chromium.org/2222063002/ )

Reason for revert:
Reverting temporary instrumentation following investigation

We want to get this in prior to dev branch.

BUG=636377

Original issue's description:
> Check that all shared workers are terminated.
> 
> Adds temporary checks to determine whether shared workers
> prevent RenderProcessHost(s) from cleaning up due to
> RenderProcessHostImpl::worker_ref_count_ > 0.
> 
> BUG=608049
> 
> Committed: https://crrev.com/529173d1bbeb53b3f9438648fad264b4db042721
> Cr-Commit-Position: refs/heads/master@{#410730}

TBR=atwilson@chromium.org,jam@chromium.org,horo@chromium.org,alokp@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=608049
==========

to

==========
Revert of Check that all shared workers are terminated. (patchset #1 id:1 of
https://codereview.chromium.org/2222063002/ )

Reason for revert:
Reverting temporary instrumentation following investigation

We want to get this in prior to dev branch.

BUG=636377

Original issue's description:
> Check that all shared workers are terminated.
> 
> Adds temporary checks to determine whether shared workers
> prevent RenderProcessHost(s) from cleaning up due to
> RenderProcessHostImpl::worker_ref_count_ > 0.
> 
> BUG=608049
> 
> Committed: https://crrev.com/529173d1bbeb53b3f9438648fad264b4db042721
> Cr-Commit-Position: refs/heads/master@{#410730}

TBR=atwilson@chromium.org,jam@chromium.org,horo@chromium.org,alokp@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=608049
==========

[commit-bot: I haz the power, 2016-08-17 22:11:07]

Committed patchset #2 (id:170001)

[commit-bot: I haz the power, 2016-08-17 22:14:38]

Description was changed from

==========
Revert of Check that all shared workers are terminated. (patchset #1 id:1 of
https://codereview.chromium.org/2222063002/ )

Reason for revert:
Reverting temporary instrumentation following investigation

We want to get this in prior to dev branch.

BUG=636377

Original issue's description:
> Check that all shared workers are terminated.
> 
> Adds temporary checks to determine whether shared workers
> prevent RenderProcessHost(s) from cleaning up due to
> RenderProcessHostImpl::worker_ref_count_ > 0.
> 
> BUG=608049
> 
> Committed: https://crrev.com/529173d1bbeb53b3f9438648fad264b4db042721
> Cr-Commit-Position: refs/heads/master@{#410730}

TBR=atwilson@chromium.org,jam@chromium.org,horo@chromium.org,alokp@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=608049
==========

to

==========
Revert of Check that all shared workers are terminated. (patchset #1 id:1 of
https://codereview.chromium.org/2222063002/ )

Reason for revert:
Reverting temporary instrumentation following investigation

We want to get this in prior to dev branch.

BUG=636377

Original issue's description:
> Check that all shared workers are terminated.
>
> Adds temporary checks to determine whether shared workers
> prevent RenderProcessHost(s) from cleaning up due to
> RenderProcessHostImpl::worker_ref_count_ > 0.
>
> BUG=608049
>
> Committed: https://crrev.com/529173d1bbeb53b3f9438648fad264b4db042721
> Cr-Commit-Position: refs/heads/master@{#410730}

TBR=atwilson@chromium.org,jam@chromium.org,horo@chromium.org,alokp@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=608049

Committed: https://crrev.com/463ac7e96bf55b106225481811b0db752284bbed
Cr-Commit-Position: refs/heads/master@{#412660}
==========

[commit-bot: I haz the power, 2016-08-17 22:14:39]

Patchset 2 (id:??) landed as
https://crrev.com/463ac7e96bf55b106225481811b0db752284bbed
Cr-Commit-Position: refs/heads/master@{#412660}

[horo, 2016-08-27 00:57:45]

A revert of this CL (patchset #2 id:170001) has been created in
https://codereview.chromium.org/2286003002/ by horo@chromium.org.

The reason for reverting is: Shared Worker bug fix patch landed.
https://codereview.chromium.org/2249173003/

BUG=639193.