WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
235398
Update LayoutTests/resources/testharness.js to 2021-6-24 version
https://bugs.webkit.org/show_bug.cgi?id=235398
Summary
Update LayoutTests/resources/testharness.js to 2021-6-24 version
Kimmo Kinnunen
Reported
2022-01-20 04:15:39 PST
Update LayoutTests/resources/testharness.js to 2021-6-24 version to match LayoutTests/imported/w3c/web-platform-tests/resources/testharness.js The harness has useful step_wait functions now
Attachments
Patch
(96.31 KB, patch)
2022-01-20 04:20 PST
,
Kimmo Kinnunen
no flags
Details
Formatted Diff
Diff
Patch
(452.06 KB, patch)
2022-01-21 03:21 PST
,
Kimmo Kinnunen
no flags
Details
Formatted Diff
Diff
Patch for landing
(456.99 KB, patch)
2022-01-27 03:12 PST
,
Kimmo Kinnunen
no flags
Details
Formatted Diff
Diff
Patch for landing
(626.92 KB, patch)
2022-01-27 23:34 PST
,
Kimmo Kinnunen
no flags
Details
Formatted Diff
Diff
Patch for landing
(640.95 KB, patch)
2022-01-28 02:28 PST
,
Kimmo Kinnunen
no flags
Details
Formatted Diff
Diff
Patch for landing
(641.29 KB, patch)
2022-01-28 03:21 PST
,
Kimmo Kinnunen
no flags
Details
Formatted Diff
Diff
Show Obsolete
(5)
View All
Add attachment
proposed patch, testcase, etc.
Kimmo Kinnunen
Comment 1
2022-01-20 04:20:03 PST
Created
attachment 449563
[details]
Patch
youenn fablet
Comment 2
2022-01-20 04:25:10 PST
Comment on
attachment 449563
[details]
Patch r=me once bots are happy.
Sam Sneddon [:gsnedders]
Comment 3
2022-01-20 06:24:16 PST
Comment on
attachment 449563
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=449563&action=review
What's the reason for going for the 2021-06-24 version rather than something more recent?
> LayoutTests/resources/testharnessreport.js:11 > - * Typically such integration will attach callbacks when each test is > - * has run, using add_result_callback(callback(test)), or when the whole test file has > - * completed, using add_completion_callback(callback(tests, harness_status)). > + * Typically test system integration will attach callbacks when each test has > + * run, using add_result_callback(callback(test)), or when the whole test file > + * has completed, using > + * add_completion_callback(callback(tests, harness_status)).
Don't replace this file; keep our own version.
Kimmo Kinnunen
Comment 4
2022-01-21 03:21:57 PST
Created
attachment 449648
[details]
Patch
Kimmo Kinnunen
Comment 5
2022-01-27 03:12:16 PST
Created
attachment 450121
[details]
Patch for landing
Radar WebKit Bug Importer
Comment 6
2022-01-27 04:16:18 PST
<
rdar://problem/88126222
>
Kimmo Kinnunen
Comment 7
2022-01-27 23:34:52 PST
Created
attachment 450207
[details]
Patch for landing
Kimmo Kinnunen
Comment 8
2022-01-28 02:28:00 PST
Created
attachment 450216
[details]
Patch for landing
Kimmo Kinnunen
Comment 9
2022-01-28 03:21:35 PST
Created
attachment 450219
[details]
Patch for landing
youenn fablet
Comment 10
2022-01-28 04:37:12 PST
Comment on
attachment 450219
[details]
Patch for landing View in context:
https://bugs.webkit.org/attachment.cgi?id=450219&action=review
> LayoutTests/http/tests/paymentrequest/payment-is-showing.https.html:-292 > - );
Why are we removing this one?
EWS
Comment 11
2022-01-30 21:48:44 PST
Committed
r288806
(
246582@main
): <
https://commits.webkit.org/246582@main
> All reviewed patches have been landed. Closing bug and clearing flags on
attachment 450219
[details]
.
Kimmo Kinnunen
Comment 12
2022-02-01 02:09:34 PST
> What's the reason for going for the 2021-06-24 version rather than something more recent?
Just to stay consistent with LayoutTests/imported/w3c/web-platform-tests/resources/testharness.js
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug