Bug 201808 - Update testharness.js from upstream
Summary: Update testharness.js 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:
 
Reported: 2019-09-15 12:41 PDT by Chris Dumez
Modified: 2019-09-16 16:54 PDT (History)
18 users (show)

See Also:


Attachments
Patch (75.80 KB, patch)
2019-09-15 12:43 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews214 for win-future (13.66 MB, application/zip)
2019-09-15 14:36 PDT, EWS Watchlist
no flags Details
Patch (125.29 KB, patch)
2019-09-15 14:45 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (176.92 KB, patch)
2019-09-15 15:34 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (236.92 KB, patch)
2019-09-15 15:36 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (176.92 KB, patch)
2019-09-15 16:34 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews212 for win-future (13.82 MB, application/zip)
2019-09-15 17:52 PDT, EWS Watchlist
no flags Details
Patch (179.14 KB, patch)
2019-09-15 18:31 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (178.70 KB, patch)
2019-09-16 13:00 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 2019-09-15 12:41:48 PDT
Update testharness.js from upstream.
Comment 1 Chris Dumez 2019-09-15 12:43:08 PDT
Created attachment 378820 [details]
Patch
Comment 2 EWS Watchlist 2019-09-15 14:36:05 PDT
Comment on attachment 378820 [details]
Patch

Attachment 378820 [details] did not pass win-ews (win):
Output: https://webkit-queues.webkit.org/results/13034569

New failing tests:
fast/custom-elements/throw-on-dynamic-markup-insertion-counter-reactions.html
fast/css/parse-justify-self.html
fast/events/clipboard-event-constructor.html
fast/shadow-dom/stylesheet-title-in-shadow-tree.html
js/promises-tests/promises-in-workers.html
fast/css/DOMQuad-serialization.html
fast/custom-elements/throw-on-dynamic-markup-insertion-counter-construct.html
fast/shadow-dom/event-path-with-window.html
http/tests/fetch/redirectmode-and-preload.html
imported/blink/editing/selection/selectstart-event-crash.html
fast/shadow-dom/slotchange-for-slot-mutation.html
fast/mediacapturefromelement/CanvasCaptureMediaStream-offscreencanvas.html
fast/shadow-dom/offsetParent-across-shadow-boundaries.html
fast/css/Element-style.html
css3/flexbox/style-change.html
Comment 3 EWS Watchlist 2019-09-15 14:36:08 PDT
Created attachment 378822 [details]
Archive of layout-test-results from ews214 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews214  Port: win-future  Platform: CYGWIN_NT-10.0-17763-3.0.5-338.x86_64-x86_64-64bit
Comment 4 Chris Dumez 2019-09-15 14:45:35 PDT
Created attachment 378823 [details]
Patch
Comment 5 Chris Dumez 2019-09-15 15:34:27 PDT
Created attachment 378824 [details]
Patch
Comment 6 Chris Dumez 2019-09-15 15:36:20 PDT
Created attachment 378826 [details]
Patch
Comment 7 Chris Dumez 2019-09-15 16:34:15 PDT
Created attachment 378830 [details]
Patch
Comment 8 EWS Watchlist 2019-09-15 17:52:09 PDT
Comment on attachment 378830 [details]
Patch

Attachment 378830 [details] did not pass win-ews (win):
Output: https://webkit-queues.webkit.org/results/13034916

New failing tests:
streams/readable-byte-stream-controller-worker.html
streams/readable-stream-byob-reader-worker.html
streams/readable-stream-byob-request-worker.html
Comment 9 EWS Watchlist 2019-09-15 17:52:12 PDT
Created attachment 378832 [details]
Archive of layout-test-results from ews212 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews212  Port: win-future  Platform: CYGWIN_NT-10.0-17763-3.0.5-338.x86_64-x86_64-64bit
Comment 10 Chris Dumez 2019-09-15 18:31:58 PDT
Created attachment 378833 [details]
Patch
Comment 11 Darin Adler 2019-09-16 12:54:32 PDT
Comment on attachment 378833 [details]
Patch

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

Mostly just a rubber stamp of the idea of doing this, but a couple comments.

> LayoutTests/fast/events/clipboard-event-constructor.html:64
> +}, "bubbles / cancelabble / composed members");

Typo here, only one "b" in "cancelable".

> LayoutTests/fast/mediastream/captureStream/canvas2d-expected.txt:4
> -PASS captureStream cloning 
> +FAIL captureStream cloning assert_array_equals: property 5, expected 127 but got 128

Do we have to change this, since we are putting a test expectation in for this test?
Comment 12 Chris Dumez 2019-09-16 12:58:59 PDT
(In reply to Darin Adler from comment #11)
> Comment on attachment 378833 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=378833&action=review
> 
> Mostly just a rubber stamp of the idea of doing this, but a couple comments.
> 
> > LayoutTests/fast/events/clipboard-event-constructor.html:64
> > +}, "bubbles / cancelabble / composed members");
> 
> Typo here, only one "b" in "cancelable".

Ah, oops, my macBookPro keyboard has been having trouble with the 'b' key :/

> 
> > LayoutTests/fast/mediastream/captureStream/canvas2d-expected.txt:4
> > -PASS captureStream cloning 
> > +FAIL captureStream cloning assert_array_equals: property 5, expected 127 but got 128
> 
> Do we have to change this, since we are putting a test expectation in for
> this test?

You're right, I'll revert.
Comment 13 Chris Dumez 2019-09-16 13:00:14 PDT
Created attachment 378884 [details]
Patch
Comment 14 WebKit Commit Bot 2019-09-16 14:04:15 PDT
Comment on attachment 378884 [details]
Patch

Clearing flags on attachment: 378884

Committed r249918: <https://trac.webkit.org/changeset/249918>
Comment 15 WebKit Commit Bot 2019-09-16 14:04:18 PDT
All reviewed patches have been landed.  Closing bug.
Comment 16 Radar WebKit Bug Importer 2019-09-16 14:06:07 PDT
<rdar://problem/55413591>
Comment 17 Truitt Savell 2019-09-16 16:43:04 PDT
It looks like the new css-dark-mode/ test are failing on Mojave from https://trac.webkit.org/changeset/249918/webkit

results:
https://build.webkit.org/results/Apple%20Mojave%20Release%20WK2%20(Tests)/r249923%20(6685)/results.html
Comment 18 Chris Dumez 2019-09-16 16:54:01 PDT
(In reply to Truitt Savell from comment #17)
> It looks like the new css-dark-mode/ test are failing on Mojave from
> https://trac.webkit.org/changeset/249918/webkit
> 
> results:
> https://build.webkit.org/results/Apple%20Mojave%20Release%20WK2%20(Tests)/
> r249923%20(6685)/results.html

Fixed in <https://trac.webkit.org/changeset/249929>.