Fix crash that could happen on shutdown if the ResourceMessageFilter tried to...

Fix crash that could happen on shutdown if the ResourceMessageFilter tried to get the IO thread's message loop when it was going away.  Do the same acrobatics as in BufferedResourceHandler by going to the UI thread first, and doing refcounting manually.

Also fix an issue I saw by inspection in BufferedResourceHandler, where Release should be called at the end of the function.

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

[Paul Godavari, 2009-08-17 18:34:42]

http://codereview.chromium.org/171055/diff/1004/10
File chrome/browser/renderer_host/buffered_resource_handler.cc (right):

http://codereview.chromium.org/171055/diff/1004/10#newcode429
Line 429: if (!request_)
Need to Release() here as well, or is this covered in a different clean up path?

[jam, 2009-08-17 18:36:39]

http://codereview.chromium.org/171055/diff/1004/10
File chrome/browser/renderer_host/buffered_resource_handler.cc (right):

http://codereview.chromium.org/171055/diff/1004/10#newcode429
Line 429: if (!request_)
On 2009/08/17 18:34:42, Paul Godavari wrote:
> Need to Release() here as well, or is this covered in a different clean up
path?

you're right, it would leak.  fixed.

[Paul Godavari, 2009-08-17 18:37:53]

LGTM