Fix GpuChannelHost destruction races.

Fix GpuChannelHost destruction races.

GpuChannelHost::MessageFilter can outlive GpuChannelHost, so make sure it
doesn't use a raw pointer to it.
Also ensure the BrowserGpuChannelHostFactory outlives the IO thread.

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

[Ken Russell (switch to Gerrit), 2013-01-11 23:35:52]

LGTM

[commit-bot: I haz the power, 2013-01-11 23:56:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/piman@chromium.org/11886005/7001

[ncarter (slow), 2013-01-12 00:11:08]

https://codereview.chromium.org/11886005/diff/7001/chrome/test/gpu/gpu_featur...
File chrome/test/gpu/gpu_feature_browsertest.cc (right):

https://codereview.chromium.org/11886005/diff/7001/chrome/test/gpu/gpu_featur...
chrome/test/gpu/gpu_feature_browsertest.cc:264:
IN_PROC_BROWSER_TEST_F(GpuFeatureTest, MAYBE_WebGLAllowed) {
Should fix this too.

https://codereview.chromium.org/11886005/diff/7001/chrome/test/gpu/gpu_featur...
chrome/test/gpu/gpu_feature_browsertest.cc:279:
IN_PROC_BROWSER_TEST_F(GpuFeatureTest, MAYBE_WebGLBlocked) {
And this.

[piman, 2013-01-12 00:14:34]

On Fri, Jan 11, 2013 at 4:11 PM, <nick@chromium.org> wrote:

>
> https://codereview.chromium.**org/11886005/diff/7001/chrome/**
>
test/gpu/gpu_feature_**browsertest.cc<https://codereview.chromium.org/11886005/diff/7001/chrome/test/gpu/gpu_feature_browsertest.cc>
> File chrome/test/gpu/gpu_feature_**browsertest.cc (right):
>
> https://codereview.chromium.**org/11886005/diff/7001/chrome/**
>
test/gpu/gpu_feature_**browsertest.cc#newcode264<https://codereview.chromium.org/11886005/diff/7001/chrome/test/gpu/gpu_feature_browsertest.cc#newcode264>
> chrome/test/gpu/gpu_feature_**browsertest.cc:264:
> IN_PROC_BROWSER_TEST_F(**GpuFeatureTest, MAYBE_WebGLAllowed) {
> Should fix this too.
>
> https://codereview.chromium.**org/11886005/diff/7001/chrome/**
>
test/gpu/gpu_feature_**browsertest.cc#newcode279<https://codereview.chromium.org/11886005/diff/7001/chrome/test/gpu/gpu_feature_browsertest.cc#newcode279>
> chrome/test/gpu/gpu_feature_**browsertest.cc:279:
> IN_PROC_BROWSER_TEST_F(**GpuFeatureTest, MAYBE_WebGLBlocked) {
> And this.
>
>
https://codereview.chromium.**org/11886005/<https://codereview.chromium.org/1...
>

Yeah, noticed those after the fact. Do you mind if I do it as a follow-up
CL?

Antoine

[ncarter (slow), 2013-01-12 00:17:11]

On 2013/01/12 00:14:34, piman wrote:
> On Fri, Jan 11, 2013 at 4:11 PM, <mailto:nick@chromium.org> wrote:
> 
> >
> > https://codereview.chromium.**org/11886005/diff/7001/chrome/**
> >
>
test/gpu/gpu_feature_**browsertest.cc<https://codereview.chromium.org/11886005/diff/7001/chrome/test/gpu/gpu_feature_browsertest.cc>
> > File chrome/test/gpu/gpu_feature_**browsertest.cc (right):
> >
> > https://codereview.chromium.**org/11886005/diff/7001/chrome/**
> >
>
test/gpu/gpu_feature_**browsertest.cc#newcode264<https://codereview.chromium.org/11886005/diff/7001/chrome/test/gpu/gpu_feature_browsertest.cc#newcode264>
> > chrome/test/gpu/gpu_feature_**browsertest.cc:264:
> > IN_PROC_BROWSER_TEST_F(**GpuFeatureTest, MAYBE_WebGLAllowed) {
> > Should fix this too.
> >
> > https://codereview.chromium.**org/11886005/diff/7001/chrome/**
> >
>
test/gpu/gpu_feature_**browsertest.cc#newcode279<https://codereview.chromium.org/11886005/diff/7001/chrome/test/gpu/gpu_feature_browsertest.cc#newcode279>
> > chrome/test/gpu/gpu_feature_**browsertest.cc:279:
> > IN_PROC_BROWSER_TEST_F(**GpuFeatureTest, MAYBE_WebGLBlocked) {
> > And this.
> >
> >
>
https://codereview.chromium.**org/11886005/%3Chttps://codereview.chromium.org...>
> >
> 
> Yeah, noticed those after the fact. Do you mind if I do it as a follow-up
> CL?
> 
> Antoine

No problem.

[commit-bot: I haz the power, 2013-01-12 04:05:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/piman@chromium.org/11886005/7001

[commit-bot: I haz the power, 2013-01-12 17:16:28]

Change committed as 176557