Review use of run_downloads_in_background() in ephy-window.c
I discovered some problems here in #1445 (closed).
run_downloads_in_background() is called in both ephy_window_close() and tab_view_close_page_cb(). Those seem to be redundant: why are both needed?
But also, it's not called in continue_window_close_after_modified_forms_check(), which is surely a bug. If it's called at the bottom of ephy_window_close(), then it must also be called in continue_window_close_after_modified_forms_check() or the check would get skipped whenever the check for modified forms is not disabled.
I suspect the check in ephy_window_close() can just be removed. I bet it's enough to have it in tab_view_close_page_cb(). Haven't checked.