Remove WebFileChooserParams::initialValue

third_party/WebKit/Source/web/ChromeClientImpl.cpp

Index: third_party/WebKit/Source/web/ChromeClientImpl.cpp
diff --git a/third_party/WebKit/Source/web/ChromeClientImpl.cpp b/third_party/WebKit/Source/web/ChromeClientImpl.cpp
index 29638d43dbcbfb59d35e6b354ee90c6c5c9996e8..398c2967e3a79781579d27dd08d2403e767f583d 100644
--- a/third_party/WebKit/Source/web/ChromeClientImpl.cpp
+++ b/third_party/WebKit/Source/web/ChromeClientImpl.cpp
@@ -711,8 +711,6 @@ void ChromeClientImpl::openFileChooser(LocalFrame* frame,
   params.directory = fileChooser->settings().allowsDirectoryUpload;
   params.acceptTypes = fileChooser->settings().acceptTypes();
   params.selectedFiles = fileChooser->settings().selectedFiles;
-  if (params.selectedFiles.size() > 0)
-    params.initialValue = params.selectedFiles[0];

[dcheng, 2016/10/25 00:27:35]

Is it possible to remove initialValue completely?

[jsbell, 2016/10/25 16:24:55]

Used in PepperFileChooserHost::OnShow apparently, which does have a "save as"
path. We could refactor it to use FileChooserParams directly  and split up
runFileChooser() - not sure that would be cleaner.

I've renamed the param (|defaultName|), updated the docs and added a DCHECK.

   params.useMediaCapture = fileChooser->settings().useMediaCapture;
   params.needLocalPath = fileChooser->settings().allowsDirectoryUpload;
   params.requestor = frame->document()->url();