Bug 198105 - run-webkit-tests: Remove feature detection support
Summary: run-webkit-tests: Remove feature detection support
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: Fujii Hironori
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2019-05-22 02:37 PDT by Fujii Hironori
Modified: 2019-05-23 19:28 PDT (History)
9 users (show)

See Also:


Attachments
Patch (21.84 KB, patch)
2019-05-22 02:54 PDT, Fujii Hironori
no flags Details | Formatted Diff | Diff
Patch (21.67 KB, patch)
2019-05-22 19:30 PDT, Fujii Hironori
no flags Details | Formatted Diff | Diff
Patch for landing (21.64 KB, patch)
2019-05-23 19:16 PDT, Fujii Hironori
no flags Details | Formatted Diff | Diff
Patch for landing (21.64 KB, patch)
2019-05-23 19:18 PDT, Fujii Hironori
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Fujii Hironori 2019-05-22 02:37:16 PDT
run-webkit-tests: Remove feature detection support

Feature detection support has been added to Windows port's DRT
and old run-webkit-tests in Bug 35610. It has been added for NRWT
in Bug 41842. But, it doesn't work for Windows port's DRT because
feature names mismatch. Since USE(ACCELERATED_COMPOSITING) flag
has been removed in Bug 127833, it is not needed anymore.
Comment 1 Fujii Hironori 2019-05-22 02:54:26 PDT
Created attachment 370391 [details]
Patch
Comment 2 Jonathan Bedard 2019-05-22 08:18:26 PDT
Comment on attachment 370391 [details]
Patch

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

> LayoutTests/ChangeLog:10
> +        Skip all compositing, animations/3d and transforms/3d temporarily.

If this is actually temporary, we should put FIXMEs in expectations, otherwise we'll lose track of why they were skipped.
Comment 3 Fujii Hironori 2019-05-22 19:19:11 PDT
Comment on attachment 370391 [details]
Patch

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

>> LayoutTests/ChangeLog:10
>> +        Skip all compositing, animations/3d and transforms/3d temporarily.
> 
> If this is actually temporary, we should put FIXMEs in expectations, otherwise we'll lose track of why they were skipped.

In WinCairo TestExpectations, I added the Skip in "TEMPORARY SKIPS" section. I think no additional comment is needed.
https://github.com/WebKit/webkit/blob/d3d6b28920d2f449f918a3c2da9c48d15bb59002/LayoutTests/platform/wincairo/TestExpectations#L1502

In AppleWin TestExpectations, I'm going to move the Skip to "UNREVIEWED failures" section.
https://github.com/WebKit/webkit/blob/d3d6b28920d2f449f918a3c2da9c48d15bb59002/LayoutTests/platform/win/TestExpectations#L2601
Comment 4 Fujii Hironori 2019-05-22 19:30:48 PDT
Created attachment 370477 [details]
Patch

* Addressed review feedbacks.
Comment 5 Jonathan Bedard 2019-05-23 07:05:14 PDT
Comment on attachment 370477 [details]
Patch

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

> Tools/ChangeLog:8
> +        Feature detection support has been added to Windows port's DRT and

Should be: 'Feature detection support was added to DRT for Windows and...'

> Tools/ChangeLog:9
> +        old run-webkit-tests in Bug 35610. It has been added for NRWT in

Likewise: 'It was added for NRWT in...'

> Tools/ChangeLog:10
> +        Bug 41842. But, it doesn't work for Windows port's DRT because

'But it doesn't work for DRT for Windows because of...'
Comment 6 Fujii Hironori 2019-05-23 19:16:53 PDT
Created attachment 370544 [details]
Patch for landing
Comment 7 Fujii Hironori 2019-05-23 19:18:02 PDT
Created attachment 370545 [details]
Patch for landing
Comment 8 Fujii Hironori 2019-05-23 19:26:12 PDT
webkitpy EWS failure is irrelevant for the patch. BuildBot is also failing.
https://build.webkit.org/builders/Apple%20Mojave%20Release%20WK2%20%28Tests%29/builds/4289
Comment 9 Fujii Hironori 2019-05-23 19:27:49 PDT
Committed r245733: <https://trac.webkit.org/changeset/245733>
Comment 10 Radar WebKit Bug Importer 2019-05-23 19:28:24 PDT
<rdar://problem/51092639>