Revert of [Reland] Scheduler refactoring to virtually eliminate redundant DoWorks

Revert of [Reland] Scheduler refactoring to virtually eliminate redundant DoWorks (patchset #11 id:200001 of https://codereview.chromium.org/2546423002/ )

Reason for revert:
http://crbug.com/674895 (revert of dependent CL must land before this one: https://codereview.chromium.org/2583333002/)

Original issue's description:
> Scheduler refactoring to virtually eliminate redundant DoWorks
>
> Note this patch, while useful in it's own right, needs base messageloop
> cancellation support for full effect.  Until that patch lands there may be
> some pointless wakeups for canceled DoWorks. This is of particular concern
> for using the scheduler on the compositor thread.
>
> Includes fix for telemetry and tsan issues.
>
> BUG=578176, 674238, 674157
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
>
> Committed: https://crrev.com/3a6ffff92addfa71d401e2718ab99677520f250c
> Committed: https://crrev.com/8c6cf02ab73c8026ac2b0ca2aa8d6fd9bd8f2db0
> Cr-Original-Commit-Position: refs/heads/master@{#438173}
> Cr-Commit-Position: refs/heads/master@{#438857}

TBR=skyostil@chromium.org,haraken@chromium.org,alexclarke@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=578176, 674238, 674157, 674895
Committed: https://crrev.com/97d232ea886d11bd5f7f5b3157637e1ae92ae980
Cr-Commit-Position: refs/heads/master@{#439663}

[gab, 2016-12-19 19:45:28]

Created Revert of [Reland] Scheduler refactoring to virtually eliminate
redundant DoWorks

[gab, 2016-12-19 19:45:40]

Description was changed from

==========
Revert of [Reland] Scheduler refactoring to virtually eliminate redundant
DoWorks (patchset #11 id:200001 of https://codereview.chromium.org/2546423002/ )

Reason for revert:
http://crbug.com/674895 (revert of dependent CL must land before this one:
https://codereview.chromium.org/2583333002/)

Original issue's description:
> Scheduler refactoring to virtually eliminate redundant DoWorks
> 
> Note this patch, while useful in it's own right, needs base messageloop
> cancellation support for full effect.  Until that patch lands there may be
> some pointless wakeups for canceled DoWorks. This is of particular concern
> for using the scheduler on the compositor thread.
> 
> Includes fix for telemetry and tsan issues.
> 
> BUG=578176, 674238, 674157
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Committed: https://crrev.com/3a6ffff92addfa71d401e2718ab99677520f250c
> Committed: https://crrev.com/8c6cf02ab73c8026ac2b0ca2aa8d6fd9bd8f2db0
> Cr-Original-Commit-Position: refs/heads/master@{#438173}
> Cr-Commit-Position: refs/heads/master@{#438857}

TBR=skyostil@chromium.org,haraken@chromium.org,alexclarke@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=578176, 674238, 674157
==========

to

==========
Revert of [Reland] Scheduler refactoring to virtually eliminate redundant
DoWorks (patchset #11 id:200001 of https://codereview.chromium.org/2546423002/ )

Reason for revert:
http://crbug.com/674895 (revert of dependent CL must land before this one:
https://codereview.chromium.org/2583333002/)

Original issue's description:
> Scheduler refactoring to virtually eliminate redundant DoWorks
> 
> Note this patch, while useful in it's own right, needs base messageloop
> cancellation support for full effect.  Until that patch lands there may be
> some pointless wakeups for canceled DoWorks. This is of particular concern
> for using the scheduler on the compositor thread.
> 
> Includes fix for telemetry and tsan issues.
> 
> BUG=578176, 674238, 674157
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Committed: https://crrev.com/3a6ffff92addfa71d401e2718ab99677520f250c
> Committed: https://crrev.com/8c6cf02ab73c8026ac2b0ca2aa8d6fd9bd8f2db0
> Cr-Original-Commit-Position: refs/heads/master@{#438173}
> Cr-Commit-Position: refs/heads/master@{#438857}

TBR=skyostil@chromium.org,haraken@chromium.org,alexclarke@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=578176, 674238, 674157, 674895
==========

[haraken, 2016-12-20 00:00:06]

LGTM to revert

[haraken, 2016-12-20 00:03:46]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-12-20 00:04: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-20 01:49:45]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482192226144870, "parent_rev":
"be8c54c1616c49cb83fac893637e3d7b156bb0ca", "commit_rev":
"39a4c1318ada9334beb07ecbe51e09dd13d2e486"}

[commit-bot: I haz the power, 2016-12-20 01:50:37]

Description was changed from

==========
Revert of [Reland] Scheduler refactoring to virtually eliminate redundant
DoWorks (patchset #11 id:200001 of https://codereview.chromium.org/2546423002/ )

Reason for revert:
http://crbug.com/674895 (revert of dependent CL must land before this one:
https://codereview.chromium.org/2583333002/)

Original issue's description:
> Scheduler refactoring to virtually eliminate redundant DoWorks
> 
> Note this patch, while useful in it's own right, needs base messageloop
> cancellation support for full effect.  Until that patch lands there may be
> some pointless wakeups for canceled DoWorks. This is of particular concern
> for using the scheduler on the compositor thread.
> 
> Includes fix for telemetry and tsan issues.
> 
> BUG=578176, 674238, 674157
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Committed: https://crrev.com/3a6ffff92addfa71d401e2718ab99677520f250c
> Committed: https://crrev.com/8c6cf02ab73c8026ac2b0ca2aa8d6fd9bd8f2db0
> Cr-Original-Commit-Position: refs/heads/master@{#438173}
> Cr-Commit-Position: refs/heads/master@{#438857}

TBR=skyostil@chromium.org,haraken@chromium.org,alexclarke@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=578176, 674238, 674157, 674895
==========

to

==========
Revert of [Reland] Scheduler refactoring to virtually eliminate redundant
DoWorks (patchset #11 id:200001 of https://codereview.chromium.org/2546423002/ )

Reason for revert:
http://crbug.com/674895 (revert of dependent CL must land before this one:
https://codereview.chromium.org/2583333002/)

Original issue's description:
> Scheduler refactoring to virtually eliminate redundant DoWorks
> 
> Note this patch, while useful in it's own right, needs base messageloop
> cancellation support for full effect.  Until that patch lands there may be
> some pointless wakeups for canceled DoWorks. This is of particular concern
> for using the scheduler on the compositor thread.
> 
> Includes fix for telemetry and tsan issues.
> 
> BUG=578176, 674238, 674157
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Committed: https://crrev.com/3a6ffff92addfa71d401e2718ab99677520f250c
> Committed: https://crrev.com/8c6cf02ab73c8026ac2b0ca2aa8d6fd9bd8f2db0
> Cr-Original-Commit-Position: refs/heads/master@{#438173}
> Cr-Commit-Position: refs/heads/master@{#438857}

TBR=skyostil@chromium.org,haraken@chromium.org,alexclarke@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=578176, 674238, 674157, 674895

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

[commit-bot: I haz the power, 2016-12-20 01:50:39]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-20 01:52:23]

Description was changed from

==========
Revert of [Reland] Scheduler refactoring to virtually eliminate redundant
DoWorks (patchset #11 id:200001 of https://codereview.chromium.org/2546423002/ )

Reason for revert:
http://crbug.com/674895 (revert of dependent CL must land before this one:
https://codereview.chromium.org/2583333002/)

Original issue's description:
> Scheduler refactoring to virtually eliminate redundant DoWorks
> 
> Note this patch, while useful in it's own right, needs base messageloop
> cancellation support for full effect.  Until that patch lands there may be
> some pointless wakeups for canceled DoWorks. This is of particular concern
> for using the scheduler on the compositor thread.
> 
> Includes fix for telemetry and tsan issues.
> 
> BUG=578176, 674238, 674157
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Committed: https://crrev.com/3a6ffff92addfa71d401e2718ab99677520f250c
> Committed: https://crrev.com/8c6cf02ab73c8026ac2b0ca2aa8d6fd9bd8f2db0
> Cr-Original-Commit-Position: refs/heads/master@{#438173}
> Cr-Commit-Position: refs/heads/master@{#438857}

TBR=skyostil@chromium.org,haraken@chromium.org,alexclarke@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=578176, 674238, 674157, 674895

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

to

==========
Revert of [Reland] Scheduler refactoring to virtually eliminate redundant
DoWorks (patchset #11 id:200001 of https://codereview.chromium.org/2546423002/ )

Reason for revert:
http://crbug.com/674895 (revert of dependent CL must land before this one:
https://codereview.chromium.org/2583333002/)

Original issue's description:
> Scheduler refactoring to virtually eliminate redundant DoWorks
>
> Note this patch, while useful in it's own right, needs base messageloop
> cancellation support for full effect.  Until that patch lands there may be
> some pointless wakeups for canceled DoWorks. This is of particular concern
> for using the scheduler on the compositor thread.
>
> Includes fix for telemetry and tsan issues.
>
> BUG=578176, 674238, 674157
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
>
> Committed: https://crrev.com/3a6ffff92addfa71d401e2718ab99677520f250c
> Committed: https://crrev.com/8c6cf02ab73c8026ac2b0ca2aa8d6fd9bd8f2db0
> Cr-Original-Commit-Position: refs/heads/master@{#438173}
> Cr-Commit-Position: refs/heads/master@{#438857}

TBR=skyostil@chromium.org,haraken@chromium.org,alexclarke@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=578176, 674238, 674157, 674895

Committed: https://crrev.com/97d232ea886d11bd5f7f5b3157637e1ae92ae980
Cr-Commit-Position: refs/heads/master@{#439663}
==========

[commit-bot: I haz the power, 2016-12-20 01:52:24]

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