Fix to allow browser close after download initiated from chrome:// url

Fix to allow browser close after download initiated from chrome:// url.

R=brettw
BUG=12745
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=19752

[rafaelw, 2009-06-30 19:09:06]

Charles,

If you can review this as well, I'd appreciate it. Note your comment in
RendererAbortedProvisionalLoad.

Is it the case that the downloads are still depending on finding the renderer by
id? If not, would it be safe to just go head and call CancelPending() in
RendererAbortedProvisionalLoad?

[Charlie Reis, 2009-06-30 19:59:25]

There's been a lot of refactoring since that comment in
RendererAbortedProvisionalLoad, so it's possible that downloads don't query in
the same way anymore.  It would certainly be nicer if that were true, but I'm
not able to test it at the moment.  (Out of the office.)

[brettw, 2009-07-01 15:25:22]

LGTM

http://codereview.chromium.org/150122/diff/1/2
File chrome/browser/renderer_host/render_view_host_manager.cc (right):

http://codereview.chromium.org/150122/diff/1/2#newcode222
Line 222: if (pending_render_view_host_) {
Don't add {} here since you didn't below (and the rest of the file doesn't
either).