Revert of Remove WorkerBackingThread::scriptCount

Revert of Remove WorkerBackingThread::scriptCount (patchset #3 id:80001 of https://codereview.chromium.org/2053273002/ )

Reason for revert:
Two compositorworker tests have failed flakily after this change:

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20ASAN/builds/24612

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20ASAN/builds/24613

Original issue's description:
> Remove WorkerBackingThread::scriptCount
>
> This CL removes scriptCount from WorkerBackingThread. The count was
> needed for compositor workers, but we don't need it any more because
> of compositor workers' design change.
>
> Before this change, compositor workers are exempted from
> TerminateExecution call because multiple workers can share the same
> backing thread, but with this change TerminateExecution can be
> called for a compositor worker when it doesn't start the shutdown
> sequence in a timely manner.
>
> BUG=487050
>
> Committed: https://crrev.com/17686215e73797de165dd749f8f77d1388f1ede6
> Cr-Commit-Position: refs/heads/master@{#400102}

TBR=nhiroki@chromium.org,flackr@chromium.org,yhirano@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=487050
Committed: https://crrev.com/bba247eab1816bc571d46f41de6346c3b7fbbfa6
Cr-Commit-Position: refs/heads/master@{#400133}

[Henrik Grunell, 2016-06-16 11:52:28]

The CQ bit was checked by grunell@chromium.org

[Henrik Grunell, 2016-06-16 11:52:29]

Created Revert of Remove WorkerBackingThread::scriptCount

[commit-bot: I haz the power, 2016-06-16 11:52:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2066403003/1

[commit-bot: I haz the power, 2016-06-16 11:53:11]

Description was changed from

==========
Revert of Remove WorkerBackingThread::scriptCount (patchset #3 id:80001 of
https://codereview.chromium.org/2053273002/ )

Reason for revert:
Two compositorworker tests have failed flakily after this change:

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20ASAN/b...

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20ASAN/b...

Original issue's description:
> Remove WorkerBackingThread::scriptCount
> 
> This CL removes scriptCount from WorkerBackingThread. The count was
> needed for compositor workers, but we don't need it any more because
> of compositor workers' design change.
> 
> Before this change, compositor workers are exempted from
> TerminateExecution call because multiple workers can share the same
> backing thread, but with this change TerminateExecution can be
> called for a compositor worker when it doesn't start the shutdown
> sequence in a timely manner.
> 
> BUG=487050
> 
> Committed: https://crrev.com/17686215e73797de165dd749f8f77d1388f1ede6
> Cr-Commit-Position: refs/heads/master@{#400102}

TBR=nhiroki@chromium.org,flackr@chromium.org,yhirano@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=487050
==========

to

==========
Revert of Remove WorkerBackingThread::scriptCount (patchset #3 id:80001 of
https://codereview.chromium.org/2053273002/ )

Reason for revert:
Two compositorworker tests have failed flakily after this change:

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20ASAN/b...

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20ASAN/b...

Original issue's description:
> Remove WorkerBackingThread::scriptCount
> 
> This CL removes scriptCount from WorkerBackingThread. The count was
> needed for compositor workers, but we don't need it any more because
> of compositor workers' design change.
> 
> Before this change, compositor workers are exempted from
> TerminateExecution call because multiple workers can share the same
> backing thread, but with this change TerminateExecution can be
> called for a compositor worker when it doesn't start the shutdown
> sequence in a timely manner.
> 
> BUG=487050
> 
> Committed: https://crrev.com/17686215e73797de165dd749f8f77d1388f1ede6
> Cr-Commit-Position: refs/heads/master@{#400102}

TBR=nhiroki@chromium.org,flackr@chromium.org,yhirano@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=487050
==========

[commit-bot: I haz the power, 2016-06-16 11:53:12]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-16 11:53:14]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-16 11:54:54]

Description was changed from

==========
Revert of Remove WorkerBackingThread::scriptCount (patchset #3 id:80001 of
https://codereview.chromium.org/2053273002/ )

Reason for revert:
Two compositorworker tests have failed flakily after this change:

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20ASAN/b...

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20ASAN/b...

Original issue's description:
> Remove WorkerBackingThread::scriptCount
> 
> This CL removes scriptCount from WorkerBackingThread. The count was
> needed for compositor workers, but we don't need it any more because
> of compositor workers' design change.
> 
> Before this change, compositor workers are exempted from
> TerminateExecution call because multiple workers can share the same
> backing thread, but with this change TerminateExecution can be
> called for a compositor worker when it doesn't start the shutdown
> sequence in a timely manner.
> 
> BUG=487050
> 
> Committed: https://crrev.com/17686215e73797de165dd749f8f77d1388f1ede6
> Cr-Commit-Position: refs/heads/master@{#400102}

TBR=nhiroki@chromium.org,flackr@chromium.org,yhirano@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=487050
==========

to

==========
Revert of Remove WorkerBackingThread::scriptCount (patchset #3 id:80001 of
https://codereview.chromium.org/2053273002/ )

Reason for revert:
Two compositorworker tests have failed flakily after this change:

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20ASAN/b...

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20ASAN/b...

Original issue's description:
> Remove WorkerBackingThread::scriptCount
>
> This CL removes scriptCount from WorkerBackingThread. The count was
> needed for compositor workers, but we don't need it any more because
> of compositor workers' design change.
>
> Before this change, compositor workers are exempted from
> TerminateExecution call because multiple workers can share the same
> backing thread, but with this change TerminateExecution can be
> called for a compositor worker when it doesn't start the shutdown
> sequence in a timely manner.
>
> BUG=487050
>
> Committed: https://crrev.com/17686215e73797de165dd749f8f77d1388f1ede6
> Cr-Commit-Position: refs/heads/master@{#400102}

TBR=nhiroki@chromium.org,flackr@chromium.org,yhirano@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=487050

Committed: https://crrev.com/bba247eab1816bc571d46f41de6346c3b7fbbfa6
Cr-Commit-Position: refs/heads/master@{#400133}
==========

[commit-bot: I haz the power, 2016-06-16 11:54:55]

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

[Henrik Grunell, 2016-06-17 10:02:17]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2075003003/ by grunell@chromium.org.

The reason for reverting is: The reverted CL was not the cause of the failure,
so I'm reverting the revert..