AppCache: Use WeakPtr<> to fix a potential uaf bug.

AppCache: Use WeakPtr<> to fix a potential uaf bug.

BUG=554908
Committed: https://crrev.com/4b5eefe562bd45c059f5dd93079450db5610788a
Cr-Commit-Position: refs/heads/master@{#359930}

[michaeln, 2015-11-12 23:56:25]

Description was changed from

==========
appcache

BUG=554908
==========

to

==========
appcache

BUG=554908
==========

[michaeln, 2015-11-12 23:57:52]

Description was changed from

==========
appcache

BUG=554908
==========

to

==========
AppCache: Use WeakPtr<> to fix a potential uaf bug.

BUG=554908
==========

[michaeln, 2015-11-12 23:57:52]

michaeln@chromium.org changed reviewers:
+ gzobqq@gmail.com

[michaeln, 2015-11-13 00:00:06]

ptal, this doesn't take care of the broader underlying issue, that's for another
cl

[michaeln, 2015-11-13 00:02:39]

https://codereview.chromium.org/1441683004/diff/1/content/browser/appcache/ap...
File content/browser/appcache/appcache_dispatcher_host.h (right):

https://codereview.chromium.org/1441683004/diff/1/content/browser/appcache/ap...
content/browser/appcache/appcache_dispatcher_host.h:12: #include
"base/process/process.h"
i'll add the include here too

[michaeln, 2015-11-16 19:05:24]

ping

[gzobqq, 2015-11-16 19:20:52]

Sorry, missed this. LGTM

[michaeln, 2015-11-16 20:14:41]

np thnx

[michaeln, 2015-11-16 20:15:07]

The CQ bit was checked by michaeln@chromium.org

[commit-bot: I haz the power, 2015-11-16 20:16:16]

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

[commit-bot: I haz the power, 2015-11-16 21:49:22]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-16 21:50:18]

Patchset 2 (id:??) landed as
https://crrev.com/4b5eefe562bd45c059f5dd93079450db5610788a
Cr-Commit-Position: refs/heads/master@{#359930}