Remove WebFileChooserParams::initialValue
On the Chromium side, FileSelectHelper::RunFileChooser asserts that
default filenames should only be specified for "Save" choosers. Blink
doesn't use those ("download" is a browser thing) so just remove
the field which would be ignored anyway. (Remembering the last used
directory is done through other means in the browser.)
BUG=632454,655298
Review-Url: https://codereview.chromium.org/2446743002
Cr-Commit-Position: refs/heads/master@{#427814}
diff --git a/content/renderer/render_frame_impl.h b/content/renderer/render_frame_impl.h
index d95eef9..9b4542e7 100644
--- a/content/renderer/render_frame_impl.h
+++ b/content/renderer/render_frame_impl.h
@@ -674,6 +674,14 @@
// process; for layout tests, this allows the test to mock out services at
// the Mojo IPC layer.
void MaybeEnableMojoBindings();
+ // Adds the given file chooser request to the file_chooser_completion_ queue
+ // (see that var for more) and requests the chooser be displayed if there are
+ // no other waiting items in the queue.
+ //
+ // Returns true if the chooser was successfully scheduled. False means we
+ // didn't schedule anything.
+ bool ScheduleFileChooser(const FileChooserParams& params,
+ blink::WebFileChooserCompletion* completion);
// Plugin-related functions --------------------------------------------------
@@ -940,15 +948,6 @@
const GURL& frame_url,
base::string16* result);
- // Adds the given file chooser request to the file_chooser_completion_ queue
- // (see that var for more) and requests the chooser be displayed if there are
- // no other waiting items in the queue.
- //
- // Returns true if the chooser was successfully scheduled. False means we
- // didn't schedule anything.
- bool ScheduleFileChooser(const FileChooserParams& params,
- blink::WebFileChooserCompletion* completion);
-
// Loads the appropriate error page for the specified failure into the frame.
// |entry| is only used by PlzNavigate when navigating to a history item.
void LoadNavigationErrorPage(const blink::WebURLRequest& failed_request,
