fix a problem that download dangerous files in a blank page can fail with authentication

fix a problem that download dangerous files in a blank page can fail with authentication

With the chrome download path, when onDownloadStarted() is called, we close the tab if it is blank.
The close call come too early as onDangerousDownload() has not been called yet.
This CL sends the filename and mimeType to java side to check if we can close the tab or not.
Also, if the tab is closed after user validates the dangerous item, the onDownloadComplete() should send a message to the app, rather than tab.

BUG=244052
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=204030

[qinmin, 2013-06-04 00:17:02]

PTAL

[nilesh, 2013-06-04 00:54:00]

LGTM with nit

https://codereview.chromium.org/16140026/diff/1/content/public/android/java/s...
File
content/public/android/java/src/org/chromium/content/browser/DownloadController.java
(right):

https://codereview.chromium.org/16140026/diff/1/content/public/android/java/s...
content/public/android/java/src/org/chromium/content/browser/DownloadController.java:36:
public void onDownloadCompleted(Context context, String url, String mimetype,
String path,
No public in interface methods.

[qinmin, 2013-06-04 02:26:55]

https://codereview.chromium.org/16140026/diff/1/content/public/android/java/s...
File
content/public/android/java/src/org/chromium/content/browser/DownloadController.java
(right):

https://codereview.chromium.org/16140026/diff/1/content/public/android/java/s...
content/public/android/java/src/org/chromium/content/browser/DownloadController.java:36:
public void onDownloadCompleted(Context context, String url, String mimetype,
String path,
On 2013/06/04 00:54:00, nilesh wrote:
> No public in interface methods.

Done.

[commit-bot: I haz the power, 2013-06-04 14:23:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/qinmin@chromium.org/16140026/5001

[commit-bot: I haz the power, 2013-06-04 20:35:53]

Change committed as 204030