Fix ResourceSchedulingFilter crash after ResourceDispatcher deleted

Fix ResourceSchedulingFilter crash after ResourceDispatcher deleted

It's possible for the ResourceDispatcher to be deleted after the
ResourceSchedulingFilter::DispatchMessage task got posted but before
it ran.  Plumb a WeakPtr into ResourceSchedulingFilter to avoid a UAF.

BUG=664077
Committed: https://crrev.com/77d6eb5f680de17bb02c1496854c59e50099af36
Cr-Commit-Position: refs/heads/master@{#431352}

[alex clarke (OOO till 29th), 2016-11-10 10:40:12]

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

[commit-bot: I haz the power, 2016-11-10 10:40:19]

Dry run: 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-11-10 11:33:32]

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

[commit-bot: I haz the power, 2016-11-10 11:33:32]

Dry run: This issue passed the CQ dry run.

[alex clarke (OOO till 29th), 2016-11-10 11:34:21]

alexclarke@chromium.org changed reviewers:
+ mkwst@chromium.org, skyostil@chromium.org

[alex clarke (OOO till 29th), 2016-11-10 11:35:50]

alexclarke@chromium.org changed reviewers:
+ pfeldman@chromium.org
- mkwst@chromium.org

[alex clarke (OOO till 29th), 2016-11-10 11:35:50]

+Pavel for OWNERS review.

Sorry Mike I'd forgotten you don't own content/

[Mike West, 2016-11-10 12:33:53]

On 2016/11/10 at 11:35:50, alexclarke wrote:
> +Pavel for OWNERS review.
> 
> Sorry Mike I'd forgotten you don't own content/

The change LGTM, even though I'm not cool. :)

[pfeldman, 2016-11-10 19:23:51]

lgtm

[alex clarke (OOO till 29th), 2016-11-10 21:05:41]

The CQ bit was checked by alexclarke@chromium.org

[commit-bot: I haz the power, 2016-11-10 21:06:02]

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-11-10 21:11:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-10 21:23:36]

Description was changed from

==========
Fix ResourceSchedulingFilter crash after ResourceDispatcher deleted

It's possible for the ResourceDispatcher to be deleted after the
ResourceSchedulingFilter::DispatchMessage task got posted but before
it ran.  Plumb a WeakPtr into ResourceSchedulingFilter to avoid a UAF.

BUG=664077
==========

to

==========
Fix ResourceSchedulingFilter crash after ResourceDispatcher deleted

It's possible for the ResourceDispatcher to be deleted after the
ResourceSchedulingFilter::DispatchMessage task got posted but before
it ran.  Plumb a WeakPtr into ResourceSchedulingFilter to avoid a UAF.

BUG=664077

Committed: https://crrev.com/77d6eb5f680de17bb02c1496854c59e50099af36
Cr-Commit-Position: refs/heads/master@{#431352}
==========

[commit-bot: I haz the power, 2016-11-10 21:23:37]

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