Remove dated NavigationScheduler TODO.

Remove dated NavigationScheduler TODO.

Task closures created by CancellableTaskFactory keep a weak reference back
to their owning GCed object, allowing the object to be garbage collected
without the task keeping it alive.

Retire the TODO in the NavigationScheduler suggesting otherwise along with
adding a unit test to verify the weakness property.

R=
BUG=585328
Committed: https://crrev.com/abf584361ba33ce3f42c5927512690a471c8ac61
Cr-Commit-Position: refs/heads/master@{#393210}

[sof, 2016-05-12 07:00:10]

sigbjornf@opera.com changed reviewers:
+ alexclarke@chromium.org, oilpan-reviews@chromium.org

[sof, 2016-05-12 07:00:11]

please take a look.

[sof, 2016-05-12 07:05:00]

Description was changed from

==========
Remove dated NavigationScheduler TODO.

Task closures created by CancellableTaskFactory keep a weak reference back
to their owning GCed object, allowing the object to be garbage collected
without the task keeping it alive.

Retire the TODO in the NavigationScheduler suggesting otherwise along with
addin a unit test to verify the weakness property.

R=
BUG=585328
==========

to

==========
Remove dated NavigationScheduler TODO.

Task closures created by CancellableTaskFactory keep a weak reference back
to their owning GCed object, allowing the object to be garbage collected
without the task keeping it alive.

Retire the TODO in the NavigationScheduler suggesting otherwise along with
adding a unit test to verify the weakness property.

R=
BUG=585328
==========

[haraken, 2016-05-12 07:18:18]

LGTM

[sof, 2016-05-12 08:45:01]

The CQ bit was checked by sigbjornf@opera.com

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

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

[commit-bot: I haz the power, 2016-05-12 08:49:16]

Description was changed from

==========
Remove dated NavigationScheduler TODO.

Task closures created by CancellableTaskFactory keep a weak reference back
to their owning GCed object, allowing the object to be garbage collected
without the task keeping it alive.

Retire the TODO in the NavigationScheduler suggesting otherwise along with
adding a unit test to verify the weakness property.

R=
BUG=585328
==========

to

==========
Remove dated NavigationScheduler TODO.

Task closures created by CancellableTaskFactory keep a weak reference back
to their owning GCed object, allowing the object to be garbage collected
without the task keeping it alive.

Retire the TODO in the NavigationScheduler suggesting otherwise along with
adding a unit test to verify the weakness property.

R=
BUG=585328
==========

[commit-bot: I haz the power, 2016-05-12 08:49:17]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-12 08:50:24]

Description was changed from

==========
Remove dated NavigationScheduler TODO.

Task closures created by CancellableTaskFactory keep a weak reference back
to their owning GCed object, allowing the object to be garbage collected
without the task keeping it alive.

Retire the TODO in the NavigationScheduler suggesting otherwise along with
adding a unit test to verify the weakness property.

R=
BUG=585328
==========

to

==========
Remove dated NavigationScheduler TODO.

Task closures created by CancellableTaskFactory keep a weak reference back
to their owning GCed object, allowing the object to be garbage collected
without the task keeping it alive.

Retire the TODO in the NavigationScheduler suggesting otherwise along with
adding a unit test to verify the weakness property.

R=
BUG=585328

Committed: https://crrev.com/abf584361ba33ce3f42c5927512690a471c8ac61
Cr-Commit-Position: refs/heads/master@{#393210}
==========

[commit-bot: I haz the power, 2016-05-12 08:50:25]

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

[alex clarke (OOO till 29th), 2016-05-12 09:02:06]

lgtm