RESOLVED FIXED 239877
[LBSE] ENABLE_LAYER_BASED_SVG_ENGINE needs to match for iOS and macOS 
https://bugs.webkit.org/show_bug.cgi?id=239877
Summary [LBSE] ENABLE_LAYER_BASED_SVG_ENGINE needs to match for iOS and macOS
Simon Fraser (smfr)
Reported 2022-04-28 17:05:58 PDT
https://trac.webkit.org/changeset/286204/webkit added: #if !defined(ENABLE_LAYER_BASED_SVG_ENGINE) && PLATFORM(MAC) #define ENABLE_LAYER_BASED_SVG_ENGINE 1 #endif We don't want the value of ENABLE_LAYER_BASED_SVG_ENGINE to differ between macOS and iOS. For testing and performance consistency, it has to be the same for all Cocoa platforms.
Attachments
Patch, v1 (1.19 KB, patch)
2022-05-06 01:26 PDT, Nikolas Zimmermann 		no flags
DetailsFormatted DiffDiff
Patch, v2 (3.45 KB, patch)
2022-05-16 04:25 PDT, Nikolas Zimmermann 		simon.fraser: review+
zimmermann: commit-queue+
DetailsFormatted DiffDiff
[fast-cq] Patch, v3 (removed ChangeLogs for landing) (1.87 KB, patch)
2022-05-18 04:16 PDT, Nikolas Zimmermann 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Nikolas Zimmermann
Comment 1 2022-05-05 13:32:22 PDT
I was not aware of that. Shall I prepare a fix?
Simon Fraser (smfr)
Comment 2 2022-05-05 14:24:28 PDT
Please do.
Radar WebKit Bug Importer
Comment 3 2022-05-05 17:06:13 PDT
<rdar://problem/92829309>
Nikolas Zimmermann
Comment 4 2022-05-06 01:26:15 PDT
Created attachment 458939 [details] Patch, v1
Nikolas Zimmermann
Comment 5 2022-05-16 04:25:31 PDT
Created attachment 459414 [details] Patch, v2
Nikolas Zimmermann
Comment 6 2022-05-18 01:31:18 PDT
Comment on attachment 459414 [details] Patch, v2 Let's see if cq can process this even with the ChangeLog changes...
Nikolas Zimmermann
Comment 7 2022-05-18 04:16:00 PDT
Created attachment 459535 [details] [fast-cq] Patch, v3 (removed ChangeLogs for landing)
EWS
Comment 8 2022-05-18 04:24:23 PDT
Committed r294389 (250684@main): <https://commits.webkit.org/250684@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 459535 [details].
Note You need to log in before you can comment on or make changes to this bug.