Partial revert of 'Refactor CompositorWorkerManager'

Partial revert of 'Refactor CompositorWorkerManager'

This is a partial revert of:
https://codereview.chromium.org/1449953002

Most of that CL is inactive code, but it did change the way the
compositor thread was created. Instead of a base::Thread, I'd used a
WebThreadImplForWorkerScheduler and this seems to have had some
performance impact. While I try to figure out why, this CL should get
us back to our prior performance.

BUG=565789
Committed: https://crrev.com/12eb990f0528415c6fd33e577e09a80c5a9760b8
Cr-Commit-Position: refs/heads/master@{#367051}

[Ian Vollick, 2015-12-28 16:34:54]

Description was changed from

==========
Partial revert of 'Refactor CompositorWorkerManager'

This is a partial revert of:
https://codereview.chromium.org/1449953002

Most of that CL is inactive code, but it did change the way the
compositor thread was created. Instead of a base::Thread, I'd used a
WebThreadImplForWorkerScheduler and this seems to have had some
performance impact. While I try to figure out why, this CL should get
us back to our prior performance.

BUG=565789
==========

to

==========
Partial revert of 'Refactor CompositorWorkerManager'

This is a partial revert of:
https://codereview.chromium.org/1449953002

Most of that CL is inactive code, but it did change the way the
compositor thread was created. Instead of a base::Thread, I'd used a
WebThreadImplForWorkerScheduler and this seems to have had some
performance impact. While I try to figure out why, this CL should get
us back to our prior performance.

BUG=565789
==========

[Ian Vollick, 2015-12-28 16:34:54]

vollick@chromium.org changed reviewers:
+ ikilpatrick@chromium.org, jam@chromium.org, kinuko@chromium.org,
piman@chromium.org

[Ian Vollick, 2015-12-28 16:37:47]

On 2015/12/28 16:34:54, vollick wrote:
> mailto:vollick@chromium.org changed reviewers:
> + mailto:ikilpatrick@chromium.org, mailto:jam@chromium.org,
mailto:kinuko@chromium.org,
> mailto:piman@chromium.org

This revert is pretty tiny and trivial but touches some important code.

If anyone is able to take a quick look at this, I'd really apprecate it.

[haraken, 2015-12-29 01:28:11]

On 2015/12/28 16:37:47, vollick wrote:
> On 2015/12/28 16:34:54, vollick wrote:
> > mailto:vollick@chromium.org changed reviewers:
> > + mailto:ikilpatrick@chromium.org, mailto:jam@chromium.org,
> mailto:kinuko@chromium.org,
> > mailto:piman@chromium.org
> 
> This revert is pretty tiny and trivial but touches some important code.
> 
> If anyone is able to take a quick look at this, I'd really apprecate it.

LGTM (but I'm not an owner of content/renderer/...)

[Ian Vollick, 2015-12-29 01:29:29]

Description was changed from

==========
Partial revert of 'Refactor CompositorWorkerManager'

This is a partial revert of:
https://codereview.chromium.org/1449953002

Most of that CL is inactive code, but it did change the way the
compositor thread was created. Instead of a base::Thread, I'd used a
WebThreadImplForWorkerScheduler and this seems to have had some
performance impact. While I try to figure out why, this CL should get
us back to our prior performance.

BUG=565789
==========

to

==========
Partial revert of 'Refactor CompositorWorkerManager'

This is a partial revert of:
https://codereview.chromium.org/1449953002

Most of that CL is inactive code, but it did change the way the
compositor thread was created. Instead of a base::Thread, I'd used a
WebThreadImplForWorkerScheduler and this seems to have had some
performance impact. While I try to figure out why, this CL should get
us back to our prior performance.

TBR=piman@chromium.org
BUG=565789
==========

[Ian Vollick, 2015-12-29 01:33:05]

Description was changed from

==========
Partial revert of 'Refactor CompositorWorkerManager'

This is a partial revert of:
https://codereview.chromium.org/1449953002

Most of that CL is inactive code, but it did change the way the
compositor thread was created. Instead of a base::Thread, I'd used a
WebThreadImplForWorkerScheduler and this seems to have had some
performance impact. While I try to figure out why, this CL should get
us back to our prior performance.

TBR=piman@chromium.org
BUG=565789
==========

to

==========
Partial revert of 'Refactor CompositorWorkerManager'

This is a partial revert of:
https://codereview.chromium.org/1449953002

Most of that CL is inactive code, but it did change the way the
compositor thread was created. Instead of a base::Thread, I'd used a
WebThreadImplForWorkerScheduler and this seems to have had some
performance impact. While I try to figure out why, this CL should get
us back to our prior performance.

BUG=565789
==========

[kinuko, 2015-12-29 03:32:08]

lgtm.

[Ian Vollick, 2015-12-29 03:32:28]

The CQ bit was checked by vollick@chromium.org

[commit-bot: I haz the power, 2015-12-29 03:32:40]

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

[commit-bot: I haz the power, 2015-12-29 04:38:20]

Description was changed from

==========
Partial revert of 'Refactor CompositorWorkerManager'

This is a partial revert of:
https://codereview.chromium.org/1449953002

Most of that CL is inactive code, but it did change the way the
compositor thread was created. Instead of a base::Thread, I'd used a
WebThreadImplForWorkerScheduler and this seems to have had some
performance impact. While I try to figure out why, this CL should get
us back to our prior performance.

BUG=565789
==========

to

==========
Partial revert of 'Refactor CompositorWorkerManager'

This is a partial revert of:
https://codereview.chromium.org/1449953002

Most of that CL is inactive code, but it did change the way the
compositor thread was created. Instead of a base::Thread, I'd used a
WebThreadImplForWorkerScheduler and this seems to have had some
performance impact. While I try to figure out why, this CL should get
us back to our prior performance.

BUG=565789
==========

[commit-bot: I haz the power, 2015-12-29 04:38:21]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-29 04:39:20]

Description was changed from

==========
Partial revert of 'Refactor CompositorWorkerManager'

This is a partial revert of:
https://codereview.chromium.org/1449953002

Most of that CL is inactive code, but it did change the way the
compositor thread was created. Instead of a base::Thread, I'd used a
WebThreadImplForWorkerScheduler and this seems to have had some
performance impact. While I try to figure out why, this CL should get
us back to our prior performance.

BUG=565789
==========

to

==========
Partial revert of 'Refactor CompositorWorkerManager'

This is a partial revert of:
https://codereview.chromium.org/1449953002

Most of that CL is inactive code, but it did change the way the
compositor thread was created. Instead of a base::Thread, I'd used a
WebThreadImplForWorkerScheduler and this seems to have had some
performance impact. While I try to figure out why, this CL should get
us back to our prior performance.

BUG=565789

Committed: https://crrev.com/12eb990f0528415c6fd33e577e09a80c5a9760b8
Cr-Commit-Position: refs/heads/master@{#367051}
==========

[commit-bot: I haz the power, 2015-12-29 04:39:21]

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