Fix a layout test crash caused by simple_resource_loader_bridge.cc

Fixed the following layout test:
    LayoutTests/http/tests/xmlhttprequest/redirect-cross-origin-post.html

The crash happens in simple_resource_loader_bridge.cc, there is a
case when MaybeUpdateUploadProgress is posted as a task and in the
queue while a redirect response by the server cancels the upload.
Leaving request_->get_upload() returns NULL, and of cource using of
the pointer failes.

[jcampan, 2009-03-23 17:58:25]

http://codereview.chromium.org/50077/diff/1/2
File webkit/tools/test_shell/simple_resource_loader_bridge.cc (right):

http://codereview.chromium.org/50077/diff/1/2#newcode325
Line 325: if (!request_->has_upload())
We should probably stop the upload_progress_timer_ timer too in that case.

[Alpha Left Google, 2009-03-24 02:39:47]

On 2009/03/23 17:58:25, jcampan wrote:
> http://codereview.chromium.org/50077/diff/1/2
> File webkit/tools/test_shell/simple_resource_loader_bridge.cc (right):
> 
> http://codereview.chromium.org/50077/diff/1/2#newcode325
> Line 325: if (!request_->has_upload())
> We should probably stop the upload_progress_timer_ timer too in that case.

Done.

[jcampan, 2009-03-24 16:47:08]

LGTM