Revert of Idle cleanup for worker context

Revert of Idle cleanup for worker context (patchset #5 id:200001 of https://codereview.chromium.org/2353033003/ )

Reason for revert:
Speculative revert to see if it helps with crbug.com/645736 .

Original issue's description:
> Idle cleanup for worker context
> 
> Currently, Skia flushes unused items out of its caches after ~1 second
> of non-use (50 flushes). This works fine while Skia is in-use, but when
> a worker context goes idle we stop calling into Skia altogether. This
> means that Skia will never flush out unused cache items from the last
> piece of work it did. This can lead to some rather large temporaries
> being kept around (~20mb).
> 
> This change adds an idle cleanup process which flushes Skia's caches
> and cleans up worker context resources after a worker context is idle
> for 1 second.
> 
> BUG=624630
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel;master.tryserver.blink:linux_precise_blink_dbg
> 
> Committed: https://crrev.com/051ae83bf29b52cefd82235ebfb90f203912afbb
> Committed: https://crrev.com/6714c23b526197220d8b9abdbd38b6d9c206d912
> Cr-Original-Commit-Position: refs/heads/master@{#420496}
> Cr-Commit-Position: refs/heads/master@{#421092}

TBR=danakj@chromium.org,jbauman@chromium.org,ericrk@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=624630

[brianderson, 2016-09-30 22:47:53]

The CQ bit was checked by brianderson@chromium.org

[brianderson, 2016-09-30 22:47:53]

Created Revert of Idle cleanup for worker context

[commit-bot: I haz the power, 2016-09-30 22:48:15]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[danakj, 2016-09-30 22:58:23]

I mean LGTM if you wanna try it but I have doubts this helps :/

[brianderson, 2016-09-30 23:37:55]

The CQ bit was unchecked by brianderson@chromium.org

[brianderson, 2016-09-30 23:38:32]

It's not this patch. Unchecking commit and closing.

[brianderson, 2016-09-30 23:38:42]

Description was changed from

==========
Revert of Idle cleanup for worker context (patchset #5 id:200001 of
https://codereview.chromium.org/2353033003/ )

Reason for revert:
Speculative revert to see if it helps with crbug.com/645736 .

Original issue's description:
> Idle cleanup for worker context
> 
> Currently, Skia flushes unused items out of its caches after ~1 second
> of non-use (50 flushes). This works fine while Skia is in-use, but when
> a worker context goes idle we stop calling into Skia altogether. This
> means that Skia will never flush out unused cache items from the last
> piece of work it did. This can lead to some rather large temporaries
> being kept around (~20mb).
> 
> This change adds an idle cleanup process which flushes Skia's caches
> and cleans up worker context resources after a worker context is idle
> for 1 second.
> 
> BUG=624630
>
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel;master.tryserver.blink:linux_precise_blink_dbg
> 
> Committed: https://crrev.com/051ae83bf29b52cefd82235ebfb90f203912afbb
> Committed: https://crrev.com/6714c23b526197220d8b9abdbd38b6d9c206d912
> Cr-Original-Commit-Position: refs/heads/master@{#420496}
> Cr-Commit-Position: refs/heads/master@{#421092}

TBR=danakj@chromium.org,jbauman@chromium.org,ericrk@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=624630
==========

to

==========
Revert of Idle cleanup for worker context (patchset #5 id:200001 of
https://codereview.chromium.org/2353033003/ )

Reason for revert:
Speculative revert to see if it helps with crbug.com/645736 .

Original issue's description:
> Idle cleanup for worker context
> 
> Currently, Skia flushes unused items out of its caches after ~1 second
> of non-use (50 flushes). This works fine while Skia is in-use, but when
> a worker context goes idle we stop calling into Skia altogether. This
> means that Skia will never flush out unused cache items from the last
> piece of work it did. This can lead to some rather large temporaries
> being kept around (~20mb).
> 
> This change adds an idle cleanup process which flushes Skia's caches
> and cleans up worker context resources after a worker context is idle
> for 1 second.
> 
> BUG=624630
>
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel;master.tryserver.blink:linux_precise_blink_dbg
> 
> Committed: https://crrev.com/051ae83bf29b52cefd82235ebfb90f203912afbb
> Committed: https://crrev.com/6714c23b526197220d8b9abdbd38b6d9c206d912
> Cr-Original-Commit-Position: refs/heads/master@{#420496}
> Cr-Commit-Position: refs/heads/master@{#421092}

TBR=danakj@chromium.org,jbauman@chromium.org,ericrk@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=624630
==========