Force deletion of test_shell's RequestProxy on the IO thread.

Force deletion of test_shell's RequestProxy on the IO thread.
This is required by r129062.

BUG=122182
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=131170

[Ami GONE FROM CHROMIUM, 2012-04-05 18:13:05]

Tony: I'm not particularly proud of this change; ISTM there should be a better
way to express the semantics I'm trying to enforce (delete on a particular
thread) but not sure how to here.  Ideas?

[tony, 2012-04-05 18:19:12]

On 2012/04/05 18:13:05, Ami Fischman wrote:
> Tony: I'm not particularly proud of this change; ISTM there should be a better
> way to express the semantics I'm trying to enforce (delete on a particular
> thread) but not sure how to here.  Ideas?

I'm not super familiar with this code.  Darin is probably a better reviewer for
it.

[darin (slow to review), 2012-04-05 20:45:21]

https://chromiumcodereview.appspot.com/10012010/diff/1/webkit/tools/test_shel...
File webkit/tools/test_shell/simple_resource_loader_bridge.cc (right):

https://chromiumcodereview.appspot.com/10012010/diff/1/webkit/tools/test_shel...
webkit/tools/test_shell/simple_resource_loader_bridge.cc:276: public
base::RefCountedThreadSafe<RequestProxy> {
wouldn't it be simpler to just use a Traits class here to force destruction of
the RequestProxy onto the proper thread?  then you wouldn't need to use
PostTaskAndReply, right?

[Ami GONE FROM CHROMIUM, 2012-04-05 22:14:31]

https://chromiumcodereview.appspot.com/10012010/diff/1/webkit/tools/test_shel...
File webkit/tools/test_shell/simple_resource_loader_bridge.cc (right):

https://chromiumcodereview.appspot.com/10012010/diff/1/webkit/tools/test_shel...
webkit/tools/test_shell/simple_resource_loader_bridge.cc:276: public
base::RefCountedThreadSafe<RequestProxy> {
On 2012/04/05 20:45:21, darin wrote:
> wouldn't it be simpler to just use a Traits class here to force destruction of
> the RequestProxy onto the proper thread?  then you wouldn't need to use
> PostTaskAndReply, right?

Hell yes.  That'd qualify as "a better way to express the semantics I'm trying
to enforce".  Thanks for the tip.

Done.

[darin (slow to review), 2012-04-06 17:47:43]

LGTM

[commit-bot: I haz the power, 2012-04-06 17:55:48]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fischman@chromium.org/10012010/2003

[commit-bot: I haz the power, 2012-04-06 19:49:07]

Change committed as 131170