Commit 79516b3d authored by Kent Tamura's avatar Kent Tamura Committed by Commit Bot

Cleanup of blink::WebFileChooserParams

- Remove unnecessary #include
- Remove unused |capture| member
- Update comments

Bug: 869257
Change-Id: Ia16c39bd31be1a0d38f67efe6823cb62229fe0c8
Reviewed-on: https://chromium-review.googlesource.com/1157947Reviewed-by: default avatarKinuko Yasuda <kinuko@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579720}
parent f3d5d112
...@@ -198,6 +198,7 @@ ...@@ -198,6 +198,7 @@
#include "third_party/blink/public/web/web_context_features.h" #include "third_party/blink/public/web/web_context_features.h"
#include "third_party/blink/public/web/web_document.h" #include "third_party/blink/public/web/web_document.h"
#include "third_party/blink/public/web/web_element_collection.h" #include "third_party/blink/public/web/web_element_collection.h"
#include "third_party/blink/public/web/web_file_chooser_completion.h"
#include "third_party/blink/public/web/web_find_options.h" #include "third_party/blink/public/web/web_find_options.h"
#include "third_party/blink/public/web/web_frame_owner_properties.h" #include "third_party/blink/public/web/web_frame_owner_properties.h"
#include "third_party/blink/public/web/web_frame_serializer.h" #include "third_party/blink/public/web/web_frame_serializer.h"
......
...@@ -139,6 +139,7 @@ ...@@ -139,6 +139,7 @@
#include "third_party/blink/public/web/web_dom_event.h" #include "third_party/blink/public/web/web_dom_event.h"
#include "third_party/blink/public/web/web_dom_message_event.h" #include "third_party/blink/public/web/web_dom_message_event.h"
#include "third_party/blink/public/web/web_element.h" #include "third_party/blink/public/web/web_element.h"
#include "third_party/blink/public/web/web_file_chooser_completion.h"
#include "third_party/blink/public/web/web_file_chooser_params.h" #include "third_party/blink/public/web/web_file_chooser_params.h"
#include "third_party/blink/public/web/web_form_control_element.h" #include "third_party/blink/public/web/web_form_control_element.h"
#include "third_party/blink/public/web/web_form_element.h" #include "third_party/blink/public/web/web_form_element.h"
......
...@@ -29,6 +29,7 @@ ...@@ -29,6 +29,7 @@
#include "third_party/blink/public/platform/web_url_response.h" #include "third_party/blink/public/platform/web_url_response.h"
#include "third_party/blink/public/web/web_console_message.h" #include "third_party/blink/public/web/web_console_message.h"
#include "third_party/blink/public/web/web_element.h" #include "third_party/blink/public/web/web_element.h"
#include "third_party/blink/public/web/web_file_chooser_completion.h"
#include "third_party/blink/public/web/web_frame.h" #include "third_party/blink/public/web/web_frame.h"
#include "third_party/blink/public/web/web_frame_widget.h" #include "third_party/blink/public/web/web_frame_widget.h"
#include "third_party/blink/public/web/web_local_frame.h" #include "third_party/blink/public/web/web_local_frame.h"
......
...@@ -34,17 +34,16 @@ ...@@ -34,17 +34,16 @@
#include "third_party/blink/public/platform/web_string.h" #include "third_party/blink/public/platform/web_string.h"
#include "third_party/blink/public/platform/web_url.h" #include "third_party/blink/public/platform/web_url.h"
#include "third_party/blink/public/platform/web_vector.h" #include "third_party/blink/public/platform/web_vector.h"
#include "third_party/blink/public/web/web_file_chooser_completion.h"
namespace blink { namespace blink {
struct WebFileChooserParams { struct WebFileChooserParams {
// If |multiSelect| is true, the dialog allows the user to select multiple // If |multi_select| is true, the dialog allows the user to select multiple
// files. // files.
bool multi_select; bool multi_select;
// If |directory| is true, the dialog allows the user to select a directory. // If |directory| is true, the dialog allows the user to select a directory.
bool directory; bool directory;
// If |saveAs| is true, the dialog allows the user to select a possibly // If |save_as| is true, the dialog allows the user to select a possibly
// non-existent file. This can be used for a "Save As" dialog. // non-existent file. This can be used for a "Save As" dialog.
bool save_as; bool save_as;
// |title| is the title for a file chooser dialog. It can be an empty string. // |title| is the title for a file chooser dialog. It can be an empty string.
...@@ -56,19 +55,17 @@ struct WebFileChooserParams { ...@@ -56,19 +55,17 @@ struct WebFileChooserParams {
// This list comes from an 'accept' attribute value of an INPUT element, and // This list comes from an 'accept' attribute value of an INPUT element, and
// it contains only lower-cased MIME type strings and file extensions. // it contains only lower-cased MIME type strings and file extensions.
WebVector<WebString> accept_types; WebVector<WebString> accept_types;
// |selectedFiles| has filenames which a file upload control already selected. // |selected_files| has filenames which a file upload control already
// A WebViewClient implementation may ask a user to select // selected. A WebLocalFrameClient implementation may ask a user to select
// - removing a file from the selected files, // - removing a file from the selected files,
// - appending other files, or // - appending other files, or
// - replacing with other files // - replacing with other files
// before opening a file chooser dialog. // before opening a file chooser dialog.
WebVector<WebString> selected_files; WebVector<WebString> selected_files;
// See http://www.w3.org/TR/html-media-capture/ for the semantics of the // See http://www.w3.org/TR/html-media-capture/ for the semantics of the
// capture attribute. If |useMediaCapture| is true, the media types // capture attribute. If |use_media_capture| is true, the media types
// indicated in |acceptTypes| should be obtained from the device's // indicated in |accept_types| should be obtained from the device's
// environment using a media capture mechanism. |capture| is deprecated and // environment using a media capture mechanism.
// provided for compatibility reasons.
WebString capture;
bool use_media_capture; bool use_media_capture;
// Whether WebFileChooserCompletion needs local paths or not. If the result // Whether WebFileChooserCompletion needs local paths or not. If the result
// of file chooser is handled by the implementation of // of file chooser is handled by the implementation of
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment