RESOLVED FIXED 231309
Add AdAttributionDaemon plist on iOS 
https://bugs.webkit.org/show_bug.cgi?id=231309
Summary Add AdAttributionDaemon plist on iOS
Alex Christensen
Reported 2021-10-06 10:44:21 PDT
Add AdAttributionDaemon plist on iOS
Attachments
Patch (10.45 KB, patch)
2021-10-06 10:45 PDT, Alex Christensen 		no flags
DetailsFormatted DiffDiff
Patch (11.83 KB, patch)
2021-10-06 11:26 PDT, Alex Christensen 		no flags
DetailsFormatted DiffDiff
Patch (11.79 KB, patch)
2021-10-06 13:24 PDT, Alex Christensen 		no flags
DetailsFormatted DiffDiff
Patch (12.48 KB, patch)
2021-10-14 20:14 PDT, Alex Christensen 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Alex Christensen
Comment 1 2021-10-06 10:45:24 PDT
Created attachment 440386 [details] Patch
Alex Christensen
Comment 2 2021-10-06 11:26:09 PDT
Created attachment 440396 [details] Patch
Alex Christensen
Comment 3 2021-10-06 13:24:16 PDT
Created attachment 440427 [details] Patch
Alex Christensen
Comment 4 2021-10-06 17:22:30 PDT
I verified that this puts the plist in the correct location and it results in the daemon starting. This should be ready to review. The daemon currently uses too much memory, so we'll need a separate bug to reduce the memory and land that first.
Radar WebKit Bug Importer
Comment 5 2021-10-13 10:45:15 PDT
<rdar://problem/84205891>
Alex Christensen
Comment 6 2021-10-14 20:14:38 PDT
Created attachment 441325 [details] Patch
John Wilander
Comment 7 2021-10-14 20:28:57 PDT
Comment on attachment 441325 [details] Patch This is not my area of expertise. Please check the GTK bots here before landing. Also, please monitor bots to make sure we don't break a build or test suite somewhere once this lands.
EWS
Comment 8 2021-10-14 21:47:54 PDT
Committed r284233 (243042@main): <https://commits.webkit.org/243042@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 441325 [details].
David Kilzer (:ddkilzer)
Comment 9 2021-10-15 12:14:20 PDT
*** Bug 231817 has been marked as a duplicate of this bug. ***
Note You need to log in before you can comment on or make changes to this bug.