Bug 176257 – Add support for input.webkitEntries

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Chris Dumez]

Add support for input.webkitEntries:
- https://wicg.github.io/entries-api/#html-forms

[Radar WebKit Bug Importer]

<rdar://problem/34218671>

[Chris Dumez]

Created attachment 319933 [details]
Patch

[Chris Dumez]

Created attachment 319935 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 319935 [details]
Patch

Clearing flags on attachment: 319935

Committed r221645: <http://trac.webkit.org/changeset/221645>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Ryan Haddad]

(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

[Chris Dumez]

(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.

[Chris Dumez]

(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>.