Fix: Omnibox is not updated properly when the main frame results in a file download
Fix: Omnibox is not updated properly when the main frame results in a file download

This is the second attempt, to address the cases where downloadable
URLs are input multiple times.

It seems that depending on the thread execution order, 
TabContents::DidFailProvisionalLoadWithError is sometimes called after
TabContents::OnStartDownload. In such cases, URL display invalidation
happens before the navigation entry removal. The result is
that the downlodable URL remains in the omnibox.

This change ensures that the URL invalidation happens right after
the navigation entry removal.

BUG=1904

TESTED=manually, gcl try

TEST=Type any URLs that causes download, e.g.,
ftp://ftp.vim.org/pub/vim/pc/gvim72.zip into the omnibox.
Observe that the URL is changed back to the original once the download starts.
Repeat this multiple times, with other URLs, to confirm it always works.



Review URL: http://codereview.chromium.org/147006

git-svn-id: http://src.chromium.org/svn/trunk/src@19094 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Former-commit-id: 766ac8535b8ce9a7723fb28df8a0a4ef2a190f38
meego-tablet-ux/meego-app-browser,meego-tablet-ux/meego-app-browser,meego-tablet-ux/meego-app-browser,meego-tablet-ux/meego-app-browser,meego-tablet-ux/meego-app-browser,meego-tablet-ux/meego-app-browser,meego-tablet-ux/meego-app-browser,meego-tablet-ux/meego-app-browser,meego-tablet-ux/meego-app-browser,meego-tablet-ux/meego-app-browser