Bug 217907

Summary: Switch to a static pattern for DumpRenderTree preferences generation to avoid ambiguous rules
Product: WebKit Reporter: Sam Weinig <sam>
Component: New BugsAssignee: Sam Weinig <sam>
Status: RESOLVED FIXED    
Severity: Normal CC: darin, simon.fraser, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch simon.fraser: review+, sam: commit-queue+

Description Sam Weinig 2020-10-19 09:37:26 PDT
Switch to a static pattern for DumpRenderTree preferences generation to mavoid ambiguous rules
Comment 1 Sam Weinig 2020-10-19 09:39:22 PDT
Created attachment 411753 [details]
Patch
Comment 2 Darin Adler 2020-10-19 09:40:21 PDT
mavoid
Comment 3 Sam Weinig 2020-10-19 10:03:39 PDT
Created attachment 411757 [details]
Patch
Comment 4 Darin Adler 2020-10-19 11:11:27 PDT
Comment on attachment 411757 [details]
Patch

Lets do more of this in the WebCore DerivedSources.make too?
Comment 5 Sam Weinig 2020-10-19 11:13:10 PDT
(In reply to Darin Adler from comment #4)
> Comment on attachment 411757 [details]
> Patch
> 
> Lets do more of this in the WebCore DerivedSources.make too?

Yeah, if it works out here, I will do it for the rest as well.
Comment 6 Sam Weinig 2020-10-19 12:40:38 PDT
Committed r268682: <https://trac.webkit.org/changeset/268682>
Comment 7 Radar WebKit Bug Importer 2020-10-19 12:41:20 PDT
<rdar://problem/70451920>
Comment 8 Ryan Haddad 2020-10-19 15:08:13 PDT
Reverted r268495, r268571, and r268682 for reason:

Caused failures for some internal build configurations

Committed r268693: <https://trac.webkit.org/changeset/268693>
Comment 9 Ryan Haddad 2020-10-19 18:56:51 PDT
Re-landing these changes as the underlying issue may have been with CI infrastructure (and should now be resolved)

Committed r268705: <https://trac.webkit.org/changeset/268705>
Collapse All Comments
Expand All Comments
Add Comment