Updates to DownloadUrlParameters in preparation for OOPIF changes

content/browser/download/drag_download_file.cc

Index: content/browser/download/drag_download_file.cc
diff --git a/content/browser/download/drag_download_file.cc b/content/browser/download/drag_download_file.cc
index 468c26f5c216a4108a10b0256b11ef4bfdf1ef6d..699b80672f7ba274e5f0114187f54ef0bfed424a 100644
--- a/content/browser/download/drag_download_file.cc
+++ b/content/browser/download/drag_download_file.cc
@@ -66,7 +66,8 @@ class DragDownloadFile::DragDownloadFileUI : public DownloadItem::Observer {
 
     RecordDownloadSource(INITIATED_BY_DRAG_N_DROP);
     std::unique_ptr<content::DownloadUrlParameters> params(
-        DownloadUrlParameters::FromWebContents(web_contents_, url_));
+        DownloadUrlParameters::CreateForWebContentsMainFrame(

[asanka, 2016/05/13 20:53:41]

This one is also fishy.

[brettw, 2016/05/13 21:20:57]

I thought about this one. I was thinking this was dragging a file to the tab so
it didn't really matter, but on further reflection I think this is dragging a
file out of a tab which I agree sounds wrong.

[asanka, 2016/05/18 15:26:37]

Yeah. This is HTML5 drag & drop. See the "dragging files" example at
http://www.html5rocks.com/en/tutorials/dnd/basics/ .

+            web_contents_, url_));
     params->set_referrer(referrer_);
     params->set_referrer_encoding(referrer_encoding_);
     params->set_callback(base::Bind(&DragDownloadFileUI::OnDownloadStarted,