Move PresentationAttributeCacheCleaner timer to main thread's timer task runner.

Move PresentationAttributeCacheCleaner timer to main thread's timer task runner.

This is shared by multiple frames in the renderer main thread. Conceptually
it's UnspecedTimer, but since it's not associated with any particular frame,
and we know it will be used on the main thread, we can just use the main
thread's timer task queue.

BUG=624694
Review-Url: https://codereview.chromium.org/2647703006
Cr-Commit-Position: refs/heads/master@{#445277}
Committed: https://chromium.googlesource.com/chromium/src/+/7ca861148c5375e15b85831ee2826a818d682c93

[jbroman, 2017-01-20 20:48:15]

The CQ bit was checked by jbroman@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-20 20:48:38]

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

[jbroman, 2017-01-20 20:50:29]

jbroman@chromium.org changed reviewers:
+ haraken@chromium.org

[jbroman, 2017-01-20 20:50:30]

There's only one of these (held by DEFINE_STATIC_LOCAL). Continuing to use the
thread's timer task runner seems reasonable.

[commit-bot: I haz the power, 2017-01-21 00:08:00]

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

[commit-bot: I haz the power, 2017-01-21 00:08:01]

Dry run: This issue passed the CQ dry run.

[haraken, 2017-01-21 01:43:27]

LGTM

Shall we add a comment about it?

[jbroman, 2017-01-21 05:14:11]

On 2017/01/21 at 01:43:27, haraken wrote:
> LGTM
> 
> Shall we add a comment about it?

Done.

[jbroman, 2017-01-21 05:19:08]

The CQ bit was checked by jbroman@chromium.org

[jbroman, 2017-01-21 05:19:09]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/2647703006/#ps20001
(title: "add a comment")

[commit-bot: I haz the power, 2017-01-21 05:19:22]

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, 2017-01-21 07:03:24]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484975948855170,
"parent_rev": "677c0342b556d3dd7a5387ffba13cc0079439ad7", "commit_rev":
"7ca861148c5375e15b85831ee2826a818d682c93"}

[commit-bot: I haz the power, 2017-01-21 07:03:53]

Description was changed from

==========
Move PresentationAttributeCacheCleaner timer to main thread's timer task runner.

This is shared by multiple frames in the renderer main thread. Conceptually
it's UnspecedTimer, but since it's not associated with any particular frame,
and we know it will be used on the main thread, we can just use the main
thread's timer task queue.

BUG=624694
==========

to

==========
Move PresentationAttributeCacheCleaner timer to main thread's timer task runner.

This is shared by multiple frames in the renderer main thread. Conceptually
it's UnspecedTimer, but since it's not associated with any particular frame,
and we know it will be used on the main thread, we can just use the main
thread's timer task queue.

BUG=624694

Review-Url: https://codereview.chromium.org/2647703006
Cr-Commit-Position: refs/heads/master@{#445277}
Committed:
https://chromium.googlesource.com/chromium/src/+/7ca861148c5375e15b85831ee282...
==========

[commit-bot: I haz the power, 2017-01-21 07:03:54]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/7ca861148c5375e15b85831ee282...