Summary: | Add support for input.webkitEntries | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||||
Component: | WebCore Misc. | Assignee: | Chris Dumez <cdumez> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, ggaren, kling, ryanhaddad, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
URL: | https://wicg.github.io/entries-api/#html-forms | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 175976 | ||||||||
Attachments: |
|
Description
Chris Dumez
2017-09-01 16:40:36 PDT
Created attachment 319933 [details]
Patch
Created attachment 319935 [details]
Patch
Comment on attachment 319935 [details] Patch Clearing flags on attachment: 319935 Committed r221645: <http://trac.webkit.org/changeset/221645> All reviewed patches have been landed. Closing bug. (In reply to WebKit Commit Bot from comment #4) > Comment on attachment 319935 [details] > Patch > > Clearing flags on attachment: 319935 > > Committed r221645: <http://trac.webkit.org/changeset/221645> The LayoutTest added with this change is failing on WK1: --- /Volumes/Data/slave/elcapitan-debug-tests-wk1/build/layout-test-results/fast/forms/file/entries-api/drag-folder-webkitEntries-expected.txt +++ /Volumes/Data/slave/elcapitan-debug-tests-wk1/build/layout-test-results/fast/forms/file/entries-api/drag-folder-webkitEntries-actual.txt @@ -16,9 +16,10 @@ PASS files[0].size is 5 PASS files[0].type is "text/plain" PASS files[1].name is "testFiles" -PASS files[1].size is 204 +FAIL files[1].size should be 204. Was 272. PASS files[1].type is "" PASS successfullyParsed is true +Some tests failed. TEST COMPLETE https://build.webkit.org/results/Apple%20El%20Capitan%20Debug%20WK1%20(Tests)/r221660%20(3437)/results.html (In reply to Ryan Haddad from comment #6) > (In reply to WebKit Commit Bot from comment #4) > > Comment on attachment 319935 [details] > > Patch > > > > Clearing flags on attachment: 319935 > > > > Committed r221645: <http://trac.webkit.org/changeset/221645> > The LayoutTest added with this change is failing on WK1: > > --- > /Volumes/Data/slave/elcapitan-debug-tests-wk1/build/layout-test-results/fast/ > forms/file/entries-api/drag-folder-webkitEntries-expected.txt > +++ > /Volumes/Data/slave/elcapitan-debug-tests-wk1/build/layout-test-results/fast/ > forms/file/entries-api/drag-folder-webkitEntries-actual.txt > @@ -16,9 +16,10 @@ > PASS files[0].size is 5 > PASS files[0].type is "text/plain" > PASS files[1].name is "testFiles" > -PASS files[1].size is 204 > +FAIL files[1].size should be 204. Was 272. > PASS files[1].type is "" > PASS successfullyParsed is true > +Some tests failed. > > TEST COMPLETE > > > https://build.webkit.org/results/ > Apple%20El%20Capitan%20Debug%20WK1%20(Tests)/r221660%20(3437)/results.html Oh, I landed 2 patches around the same time and they apparently conflicted. Will fix shortly. (In reply to Chris Dumez from comment #7) > (In reply to Ryan Haddad from comment #6) > > (In reply to WebKit Commit Bot from comment #4) > > > Comment on attachment 319935 [details] > > > Patch > > > > > > Clearing flags on attachment: 319935 > > > > > > Committed r221645: <http://trac.webkit.org/changeset/221645> > > The LayoutTest added with this change is failing on WK1: > > > > --- > > /Volumes/Data/slave/elcapitan-debug-tests-wk1/build/layout-test-results/fast/ > > forms/file/entries-api/drag-folder-webkitEntries-expected.txt > > +++ > > /Volumes/Data/slave/elcapitan-debug-tests-wk1/build/layout-test-results/fast/ > > forms/file/entries-api/drag-folder-webkitEntries-actual.txt > > @@ -16,9 +16,10 @@ > > PASS files[0].size is 5 > > PASS files[0].type is "text/plain" > > PASS files[1].name is "testFiles" > > -PASS files[1].size is 204 > > +FAIL files[1].size should be 204. Was 272. > > PASS files[1].type is "" > > PASS successfullyParsed is true > > +Some tests failed. > > > > TEST COMPLETE > > > > > > https://build.webkit.org/results/ > > Apple%20El%20Capitan%20Debug%20WK1%20(Tests)/r221660%20(3437)/results.html > > Oh, I landed 2 patches around the same time and they apparently conflicted. > Will fix shortly. Rebaselined in <http://trac.webkit.org/changeset/221663>. |