Bug 168848 - Add a way to test file input without relying on drag and drop
Summary: Add a way to test file input without relying on drag and drop
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Andy Estes
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-02-24 14:38 PST by Andy Estes
Modified: 2017-04-27 10:24 PDT (History)
9 users (show)

See Also:


Attachments
Patch (81.44 KB, patch)
2017-02-24 15:06 PST, Andy Estes
no flags Details | Formatted Diff | Diff
Patch (81.44 KB, patch)
2017-02-24 16:35 PST, Andy Estes
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews102 for mac-elcapitan (1.38 MB, application/zip)
2017-02-24 18:35 PST, Build Bot
no flags Details
Archive of layout-test-results from ews115 for mac-elcapitan (1.84 MB, application/zip)
2017-02-24 19:04 PST, Build Bot
no flags Details
Archive of layout-test-results from ews125 for ios-simulator-wk2 (deleted)
2017-02-24 19:27 PST, Build Bot
no flags Details
Archive of layout-test-results from ews123 for ios-simulator-wk2 (deleted)
2017-02-24 21:30 PST, Build Bot
no flags Details
Patch (91.08 KB, patch)
2017-02-26 18:04 PST, Andy Estes
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from webkit-cq-01 for mac-elcapitan (898.00 KB, application/zip)
2017-02-26 19:13 PST, WebKit Commit Bot
no flags Details
Patch (89.29 KB, patch)
2017-02-27 00:21 PST, Andy Estes
no flags Details | Formatted Diff | Diff
Patch (88.58 KB, patch)
2017-02-27 00:24 PST, Andy Estes
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andy Estes 2017-02-24 14:38:57 PST
Add a way to test file input without relying on drag and drop
Comment 1 Andy Estes 2017-02-24 15:00:38 PST
rdar://problem/22258242
Comment 2 Andy Estes 2017-02-24 15:06:14 PST
Created attachment 302693 [details]
Patch
Comment 3 Tim Horton 2017-02-24 15:18:36 PST
Comment on attachment 302693 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=302693&action=review

> Tools/DumpRenderTree/mac/UIDelegate.mm:368
> +    printf("OPEN FILE PANEL\n");

I don't think you meant to include this.
Comment 4 Andy Estes 2017-02-24 15:20:09 PST
(In reply to comment #3)
> Comment on attachment 302693 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=302693&action=review
> 
> > Tools/DumpRenderTree/mac/UIDelegate.mm:368
> > +    printf("OPEN FILE PANEL\n");
> 
> I don't think you meant to include this.

I did, actually. It matches the existing logging we had in WebKitTestRunner's UI delegate.

Thanks for the review!
Comment 5 Tim Horton 2017-02-24 15:24:25 PST
(In reply to comment #4)
> (In reply to comment #3)
> > Comment on attachment 302693 [details]
> > Patch
> > 
> > View in context:
> > https://bugs.webkit.org/attachment.cgi?id=302693&action=review
> > 
> > > Tools/DumpRenderTree/mac/UIDelegate.mm:368
> > > +    printf("OPEN FILE PANEL\n");
> > 
> > I don't think you meant to include this.
> 
> I did, actually. It matches the existing logging we had in
> WebKitTestRunner's UI delegate.
> 
> Thanks for the review!

Heh! OK.
Comment 6 Andy Estes 2017-02-24 16:35:00 PST
Created attachment 302707 [details]
Patch
Comment 7 Build Bot 2017-02-24 18:35:34 PST
Comment on attachment 302707 [details]
Patch

Attachment 302707 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/3188431

New failing tests:
fast/events/domactivate-sets-underlying-click-event-as-handled.html
accessibility/axpress-on-aria-button.html
accessibility/file-upload-button-with-axpress.html
Comment 8 Build Bot 2017-02-24 18:35:59 PST
Created attachment 302725 [details]
Archive of layout-test-results from ews102 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 9 Build Bot 2017-02-24 19:04:34 PST
Comment on attachment 302707 [details]
Patch

Attachment 302707 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/3188535

New failing tests:
fast/events/domactivate-sets-underlying-click-event-as-handled.html
accessibility/axpress-on-aria-button.html
accessibility/file-upload-button-with-axpress.html
Comment 10 Build Bot 2017-02-24 19:04:55 PST
Created attachment 302726 [details]
Archive of layout-test-results from ews115 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews115  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 11 Build Bot 2017-02-24 19:27:20 PST
Comment on attachment 302707 [details]
Patch

Attachment 302707 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/3188429

New failing tests:
http/tests/local/fileapi/file-last-modified-after-delete-using-open-panel.html
fast/files/apply-blob-url-to-xhr-using-open-panel.html
http/tests/local/formdata/send-form-data-with-empty-file-filename-using-open-panel.html
fast/files/file-reader-abort-using-open-panel.html
media/video-src-blob-using-open-panel.html
fast/files/apply-blob-url-to-img-using-open-panel.html
fast/forms/file/input-file-value-using-open-panel.html
http/tests/local/blob/send-hybrid-blob-using-open-panel.html
fast/forms/file/input-file-write-files-using-open-panel.html
fast/files/file-list-test-using-open-panel.html
fast/history/page-cache-createObjectURL-using-open-panel.html
fast/dom/FileList-iterator-using-open-panel.html
fast/forms/file/file-input-reset-using-open-panel.html
http/tests/local/formdata/send-form-data-constructed-from-form-using-open-panel.html
http/tests/local/fileapi/file-last-modified-using-open-panel.html
fast/forms/file/file-reset-in-change-using-open-panel.html
fast/files/file-reader-directory-crash-using-open-panel.html
fast/forms/file/get-file-upload-using-open-panel.html
Comment 12 Build Bot 2017-02-24 19:27:47 PST
Created attachment 302729 [details]
Archive of layout-test-results from ews125 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews125  Port: ios-simulator-wk2  Platform: Mac OS X 10.11.6
Comment 13 Build Bot 2017-02-24 21:29:50 PST
Comment on attachment 302707 [details]
Patch

Attachment 302707 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/3189004

New failing tests:
http/tests/local/fileapi/file-last-modified-after-delete-using-open-panel.html
fast/files/apply-blob-url-to-xhr-using-open-panel.html
http/tests/local/formdata/send-form-data-with-empty-file-filename-using-open-panel.html
fast/files/file-reader-abort-using-open-panel.html
fast/forms/file/input-file-write-files-using-open-panel.html
media/video-src-blob-using-open-panel.html
fast/files/apply-blob-url-to-img-using-open-panel.html
fast/forms/file/input-file-value-using-open-panel.html
http/tests/local/blob/send-hybrid-blob-using-open-panel.html
fast/files/file-reader-directory-crash-using-open-panel.html
fast/files/file-list-test-using-open-panel.html
http/tests/local/fileapi/file-last-modified-using-open-panel.html
fast/forms/file/file-input-reset-using-open-panel.html
http/tests/local/formdata/send-form-data-constructed-from-form-using-open-panel.html
fast/history/page-cache-createObjectURL-using-open-panel.html
fast/forms/file/file-reset-in-change-using-open-panel.html
fast/dom/FileList-iterator-using-open-panel.html
fast/forms/file/get-file-upload-using-open-panel.html
Comment 14 Build Bot 2017-02-24 21:30:00 PST
Created attachment 302733 [details]
Archive of layout-test-results from ews123 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews123  Port: ios-simulator-wk2  Platform: Mac OS X 10.11.6
Comment 15 Andy Estes 2017-02-26 18:04:15 PST
Created attachment 302799 [details]
Patch
Comment 16 WebKit Commit Bot 2017-02-26 19:13:15 PST
Comment on attachment 302799 [details]
Patch

Rejecting attachment 302799 [details] from commit-queue.

New failing tests:
media/modern-media-controls/volume-down-support/volume-down-support.html
Full output: http://webkit-queues.webkit.org/results/3199114
Comment 17 WebKit Commit Bot 2017-02-26 19:13:20 PST
Created attachment 302801 [details]
Archive of layout-test-results from webkit-cq-01 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the commit-queue.
Bot: webkit-cq-01  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 18 Andy Estes 2017-02-26 21:10:59 PST
(In reply to comment #13)
> Comment on attachment 302707 [details]
> Patch
> 
> Attachment 302707 [details] did not pass ios-sim-ews (ios-simulator-wk2):
> Output: http://webkit-queues.webkit.org/results/3189004
> 
> New failing tests:
> http/tests/local/fileapi/file-last-modified-after-delete-using-open-panel.
> html
> fast/files/apply-blob-url-to-xhr-using-open-panel.html
> http/tests/local/formdata/send-form-data-with-empty-file-filename-using-open-
> panel.html
> fast/files/file-reader-abort-using-open-panel.html
> fast/forms/file/input-file-write-files-using-open-panel.html
> media/video-src-blob-using-open-panel.html
> fast/files/apply-blob-url-to-img-using-open-panel.html
> fast/forms/file/input-file-value-using-open-panel.html
> http/tests/local/blob/send-hybrid-blob-using-open-panel.html
> fast/files/file-reader-directory-crash-using-open-panel.html
> fast/files/file-list-test-using-open-panel.html
> http/tests/local/fileapi/file-last-modified-using-open-panel.html
> fast/forms/file/file-input-reset-using-open-panel.html
> http/tests/local/formdata/send-form-data-constructed-from-form-using-open-
> panel.html
> fast/history/page-cache-createObjectURL-using-open-panel.html
> fast/forms/file/file-reset-in-change-using-open-panel.html
> fast/dom/FileList-iterator-using-open-panel.html
> fast/forms/file/get-file-upload-using-open-panel.html

These timeouts will be fixed by https://bugs.webkit.org/show_bug.cgi?id=141906
Comment 19 Andy Estes 2017-02-27 00:21:08 PST
Created attachment 302827 [details]
Patch
Comment 20 Andy Estes 2017-02-27 00:24:13 PST
Created attachment 302828 [details]
Patch
Comment 21 WebKit Commit Bot 2017-02-27 01:51:21 PST
Comment on attachment 302828 [details]
Patch

Clearing flags on attachment: 302828

Committed r213042: <http://trac.webkit.org/changeset/213042>
Comment 22 WebKit Commit Bot 2017-02-27 01:51:28 PST
All reviewed patches have been landed.  Closing bug.