Style resolver needs context to resolve things correctly. It is currently provided in an ad-hoc manner as a collection of function arguments. Factor them into a struct for consistency and future extensibility.
Created attachment 442140 [details] Patch
Created attachment 442141 [details] Patch
Created attachment 442152 [details] Patch
Created attachment 442161 [details] Patch
Comment on attachment 442161 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=442161&action=review > Source/WebCore/style/StyleResolver.h:83 > +struct ResolveContext { Shouldn't this be called ResolutionContext rather? Resolve is a noun too, but it doesn't mean the same thing, and a noun seems better than a verb here.
> Shouldn't this be called ResolutionContext rather? Resolve is a noun too, > but it doesn't mean the same thing, and a noun seems better than a verb here. Good idea
Created attachment 442168 [details] Patch
Committed r284693 (243412@main): <https://commits.webkit.org/243412@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 442168 [details].
<rdar://problem/84553304>
(In reply to EWS from comment #8) > Committed r284693 (243412@main): <https://commits.webkit.org/243412@main> > > All reviewed patches have been landed. Closing bug and clearing flags on > attachment 442168 [details]. This caused the clean build of WebKitGTK to break with a linking error. Reported at bug 232160