Using WeakPtr for requests to MediaStreamDispatcher.

Using WeakPtr for requests to MediaStreamDispatcher.

This is to ensure no dangling pointers to MediaStreamImpl (and other requesters). It's one of several fixes for the bug. No actual crash or issue has been reported that this CL fixes; it's for removing potential issues.


BUG=112408
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=131330

[tommi (sloooow) - chröme, 2012-03-29 21:24:52]

lgtm

[commit-bot: I haz the power, 2012-03-29 21:25:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/grunell@chromium.org/9903014/1

[commit-bot: I haz the power, 2012-03-29 21:25:20]

Presubmit check for 9903014-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit Messages **
If this change requires manual test instructions to QA team, add
TEST=[instructions].

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    content/renderer

Presubmit checks took 1.2s to calculate.

[tommi (sloooow) - chröme, 2012-03-29 21:27:49]

Hey John - do you mind taking a look?  We need a stamp for the pepper file (no
owner file).

[jam, 2012-03-29 23:11:02]

lgtm

[commit-bot: I haz the power, 2012-03-30 05:42:04]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/grunell@chromium.org/9903014/1

[commit-bot: I haz the power, 2012-03-30 06:00:49]

Try job failure for 9903014-1 (retry) on win_rel for step "compile" (clobber
build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[Henrik Grunell, 2012-04-02 08:19:27]

Tommi, I had forgot to update the unit test. Please review that one file in
patch 2 before I commit.

[tommi (sloooow) - chröme, 2012-04-03 09:34:18]

lgtm

[commit-bot: I haz the power, 2012-04-03 09:34:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/grunell@chromium.org/9903014/12002

[commit-bot: I haz the power, 2012-04-03 09:53:56]

Try job failure for 9903014-12002 (retry) on win_rel for step "compile" (clobber
build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2012-04-08 19:28:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/grunell@chromium.org/9903014/26001

[commit-bot: I haz the power, 2012-04-08 22:57:28]

Try job failure for 9903014-26001 (retry) (retry) on win_rel for step
"ipc_tests" (clobber build).
It's a second try, previously, step "ipc_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...