Revert of Scheduler refactoring to virtually elimate redundant DoWorks

Revert of Scheduler refactoring to virtually elimate redundant DoWorks (patchset #9 id:160001 of https://codereview.chromium.org/2546423002/ )

Reason for revert:
TSAN lock-order-inversion: http://crbug.com/674238

Original issue's description:
> Scheduler refactoring to virtually elimate 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.
>
> BUG=578176
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
>
> Committed: https://crrev.com/3a6ffff92addfa71d401e2718ab99677520f250c
> Cr-Commit-Position: refs/heads/master@{#438173}

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
NOPRESUBMIT=true

Committed: https://crrev.com/9442d5557bc3a88b4864462ee14c7e65b5d93d1d
Cr-Commit-Position: refs/heads/master@{#438659}

[gab, 2016-12-14 19:34:19]

Created Revert of Scheduler refactoring to virtually elimate redundant DoWorks

[gab, 2016-12-14 19:34:33]

Description was changed from

==========
Revert of Scheduler refactoring to virtually elimate redundant DoWorks (patchset
#9 id:160001 of https://codereview.chromium.org/2546423002/ )

Reason for revert:
TSAN lock-order-inversion: http://crbug.com/674238

Original issue's description:
> Scheduler refactoring to virtually elimate 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.
> 
> BUG=578176
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Committed: https://crrev.com/3a6ffff92addfa71d401e2718ab99677520f250c
> Cr-Commit-Position: refs/heads/master@{#438173}

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
==========

to

==========
Revert of Scheduler refactoring to virtually elimate redundant DoWorks (patchset
#9 id:160001 of https://codereview.chromium.org/2546423002/ )

Reason for revert:
TSAN lock-order-inversion: http://crbug.com/674238

Original issue's description:
> Scheduler refactoring to virtually elimate 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.
> 
> BUG=578176
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Committed: https://crrev.com/3a6ffff92addfa71d401e2718ab99677520f250c
> Cr-Commit-Position: refs/heads/master@{#438173}

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
==========

[gab, 2016-12-14 19:34:37]

The CQ bit was checked by gab@chromium.org

[commit-bot: I haz the power, 2016-12-14 19:35:32]

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-14 19:45:11]

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

[commit-bot: I haz the power, 2016-12-14 19:45:12]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[gab, 2016-12-14 19:46:40]

Description was changed from

==========
Revert of Scheduler refactoring to virtually elimate redundant DoWorks (patchset
#9 id:160001 of https://codereview.chromium.org/2546423002/ )

Reason for revert:
TSAN lock-order-inversion: http://crbug.com/674238

Original issue's description:
> Scheduler refactoring to virtually elimate 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.
> 
> BUG=578176
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Committed: https://crrev.com/3a6ffff92addfa71d401e2718ab99677520f250c
> Cr-Commit-Position: refs/heads/master@{#438173}

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
==========

to

==========
Revert of Scheduler refactoring to virtually elimate redundant DoWorks (patchset
#9 id:160001 of https://codereview.chromium.org/2546423002/ )

Reason for revert:
TSAN lock-order-inversion: http://crbug.com/674238

Original issue's description:
> Scheduler refactoring to virtually elimate 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.
> 
> BUG=578176
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Committed: https://crrev.com/3a6ffff92addfa71d401e2718ab99677520f250c
> Cr-Commit-Position: refs/heads/master@{#438173}

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
NOPRESUBMIT=true
==========

[gab, 2016-12-14 19:46:45]

The CQ bit was checked by gab@chromium.org

[commit-bot: I haz the power, 2016-12-14 19:47:24]

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-14 23:04:16]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481744805158740, "parent_rev":
"1dab006db067c0081837a0549b312f71b2e362ff", "commit_rev":
"79602f5f0ecc4be48e0156cf2f888dec17d6088e"}

[commit-bot: I haz the power, 2016-12-14 23:04:53]

Description was changed from

==========
Revert of Scheduler refactoring to virtually elimate redundant DoWorks (patchset
#9 id:160001 of https://codereview.chromium.org/2546423002/ )

Reason for revert:
TSAN lock-order-inversion: http://crbug.com/674238

Original issue's description:
> Scheduler refactoring to virtually elimate 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.
> 
> BUG=578176
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Committed: https://crrev.com/3a6ffff92addfa71d401e2718ab99677520f250c
> Cr-Commit-Position: refs/heads/master@{#438173}

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
NOPRESUBMIT=true
==========

to

==========
Revert of Scheduler refactoring to virtually elimate redundant DoWorks (patchset
#9 id:160001 of https://codereview.chromium.org/2546423002/ )

Reason for revert:
TSAN lock-order-inversion: http://crbug.com/674238

Original issue's description:
> Scheduler refactoring to virtually elimate 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.
> 
> BUG=578176
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Committed: https://crrev.com/3a6ffff92addfa71d401e2718ab99677520f250c
> Cr-Commit-Position: refs/heads/master@{#438173}

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
NOPRESUBMIT=true

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

[commit-bot: I haz the power, 2016-12-14 23:04:54]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-14 23:06:41]

Description was changed from

==========
Revert of Scheduler refactoring to virtually elimate redundant DoWorks (patchset
#9 id:160001 of https://codereview.chromium.org/2546423002/ )

Reason for revert:
TSAN lock-order-inversion: http://crbug.com/674238

Original issue's description:
> Scheduler refactoring to virtually elimate 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.
> 
> BUG=578176
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Committed: https://crrev.com/3a6ffff92addfa71d401e2718ab99677520f250c
> Cr-Commit-Position: refs/heads/master@{#438173}

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
NOPRESUBMIT=true

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

to

==========
Revert of Scheduler refactoring to virtually elimate redundant DoWorks (patchset
#9 id:160001 of https://codereview.chromium.org/2546423002/ )

Reason for revert:
TSAN lock-order-inversion: http://crbug.com/674238

Original issue's description:
> Scheduler refactoring to virtually elimate 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.
>
> BUG=578176
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
>
> Committed: https://crrev.com/3a6ffff92addfa71d401e2718ab99677520f250c
> Cr-Commit-Position: refs/heads/master@{#438173}

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
NOPRESUBMIT=true

Committed: https://crrev.com/9442d5557bc3a88b4864462ee14c7e65b5d93d1d
Cr-Commit-Position: refs/heads/master@{#438659}
==========

[commit-bot: I haz the power, 2016-12-14 23:06:44]

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

[haraken, 2016-12-14 23:52:39]

LGTM