Summary: | Split ATK specific code in WPEViewBackends | ||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Don Olmstead <don.olmstead> | ||||||||||||||
Component: | WPE WebKit | Assignee: | Don Olmstead <don.olmstead> | ||||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||||
Severity: | Normal | CC: | annulen, aperez, bugs-noreply, ews-watchlist, gyuyoung.kim, mcatanzaro, ryuan.choi, sergio | ||||||||||||||
Priority: | P2 | ||||||||||||||||
Version: | WebKit Nightly Build | ||||||||||||||||
Hardware: | Unspecified | ||||||||||||||||
OS: | Unspecified | ||||||||||||||||
Attachments: |
|
Description
Don Olmstead
2021-12-14 13:45:27 PST
Created attachment 447155 [details]
WIP Patch
Created attachment 447158 [details]
WIP Patch
Created attachment 447167 [details]
WIP Patch
Created attachment 447173 [details]
WIP Patch
Created attachment 447204 [details]
Patch
Created attachment 447205 [details]
Patch
Comment on attachment 447205 [details]
Patch
This looks fine to me, but I'll let a WPE port maintainer give the final r+.
The other way to do it would be to create Tools/wpe/backends/wpe/ViewBackendWPE.cpp as the "WPE port version of the WPE ViewBackend" so that it can contain WPE port-specific stuff separate from the generic libwpe stuff, which might be more typical for us and could include more than just ATK dependencies. But maybe that's just too confusing. :)
Committed r287105 (245294@main): <https://commits.webkit.org/245294@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 447205 [details]. |