Bug 212639 - Resync web-platform-tests/2dcontext from upstream
Summary: Resync web-platform-tests/2dcontext from upstream
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords: InRadar
Depends on:
Blocks: 213927
  Show dependency treegraph
 
Reported: 2020-06-02 09:14 PDT by Chris Dumez
Modified: 2020-07-16 09:10 PDT (History)
8 users (show)

See Also:


Attachments
Patch (7.62 MB, patch)
2020-06-02 11:03 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (7.62 MB, patch)
2020-06-02 11:24 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (7.62 MB, patch)
2020-06-02 13:15 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (7.78 MB, patch)
2020-06-02 17:01 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (7.79 MB, patch)
2020-06-02 22:26 PDT, Chris Dumez
youennf: review+
ews-feeder: commit-queue-
Details | Formatted Diff | Diff
Patch (7.79 MB, patch)
2020-06-03 10:29 PDT, Chris Dumez
ews-feeder: commit-queue-
Details | Formatted Diff | Diff
Patch (7.79 MB, patch)
2020-06-03 13:22 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2020-06-02 09:14:51 PDT
Resync web-platform-tests/2dcontext from upstream.
Comment 1 Chris Dumez 2020-06-02 09:22:41 PDT
Those tests got moved upstream:
2dcontext -> html/canvas/element
Comment 2 Chris Dumez 2020-06-02 11:03:10 PDT
Created attachment 400840 [details]
Patch
Comment 3 Chris Dumez 2020-06-02 11:24:54 PDT
Created attachment 400847 [details]
Patch
Comment 4 Chris Dumez 2020-06-02 13:15:58 PDT
Created attachment 400850 [details]
Patch
Comment 5 Chris Dumez 2020-06-02 17:01:11 PDT
Created attachment 400868 [details]
Patch
Comment 6 Chris Dumez 2020-06-02 22:26:50 PDT
Created attachment 400898 [details]
Patch
Comment 7 youenn fablet 2020-06-03 00:15:25 PDT
As of LayoutTests/canvas/philip/tests/2d.drawImage.incomplete.emptysrc.html and siblings, they are probably obsoleted by WPT tests.
If so, it might be good to remove them.
Said, wdyt?
Comment 8 EWS 2020-06-03 10:21:57 PDT
Tools/Scripts/svn-apply failed to apply attachment 400898 [details] to trunk.
Please resolve the conflicts and upload a new patch.
Comment 9 Chris Dumez 2020-06-03 10:29:53 PDT
Created attachment 400940 [details]
Patch
Comment 10 EWS 2020-06-03 12:49:29 PDT
commit-queue failed to commit attachment 400940 [details] to WebKit repository.
Comment 11 Chris Dumez 2020-06-03 13:22:42 PDT
Created attachment 400963 [details]
Patch
Comment 12 EWS 2020-06-03 16:01:31 PDT
commit-queue failed to commit attachment 400963 [details] to WebKit repository.
Comment 13 Chris Dumez 2020-06-03 18:05:46 PDT
Committed r262527: <https://trac.webkit.org/changeset/262527>
Comment 14 Radar WebKit Bug Importer 2020-06-03 18:06:36 PDT
<rdar://problem/63954201>
Comment 15 Carlos Alberto Lopez Perez 2020-06-04 04:37:03 PDT
(In reply to Chris Dumez from comment #13)
> Committed r262527: <https://trac.webkit.org/changeset/262527>

I was working on a similar patch on bug 212613 when i noticed this landing.

I don't know how you did this import or what went wrong, but I see it is missing the import of all the new canvas tests.

Upstream WPT moved "2dcontext" -> "html/canvas/element" but on r262527 I see no folder LayoutTests/imported/w3c/web-platform-tests/html/canvas

Where are the new tests?


Also some other tests had resources to that folder that now not longer exist.

See all the new failures: https://build.webkit.org/builders/Apple%20Mojave%20Release%20WK2%20%28Tests%29/builds/13538/steps/layout-test/logs/stdio


What surprises me is how the EWS was green.. maybe the patch landed was different than the one tested by the EWS?
Comment 16 Chris Dumez 2020-06-04 07:44:21 PDT
(In reply to Carlos Alberto Lopez Perez from comment #15)
> (In reply to Chris Dumez from comment #13)
> > Committed r262527: <https://trac.webkit.org/changeset/262527>
> 
> I was working on a similar patch on bug 212613 when i noticed this landing.
> 
> I don't know how you did this import or what went wrong, but I see it is
> missing the import of all the new canvas tests.
> 
> Upstream WPT moved "2dcontext" -> "html/canvas/element" but on r262527 I see
> no folder LayoutTests/imported/w3c/web-platform-tests/html/canvas
> 
> Where are the new tests?
> 
> 
> Also some other tests had resources to that folder that now not longer exist.
> 
> See all the new failures:
> https://build.webkit.org/builders/
> Apple%20Mojave%20Release%20WK2%20%28Tests%29/builds/13538/steps/layout-test/
> logs/stdio
> 
> 
> What surprises me is how the EWS was green.. maybe the patch landed was
> different than the one tested by the EWS?

The commit queue refused to commit the patch, likely because it was too big. I had to land manually. It is likely I messed up my local commit and missed new files based on what you are saying. I will look into fixing this this morning.
Comment 17 Chris Dumez 2020-06-04 08:07:23 PDT
Committed r262539: <https://trac.webkit.org/changeset/262539>