Bug 238026

Summary: Removed telemetry from Networking.sb
Product: WebKit Reporter: Adam Mazander <mazander>
Component: WebKit Misc.Assignee: Adam Mazander <mazander>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, mazander, pvollan, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Local Build   
Hardware: iPhone / iPad   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Adam Mazander
Reported 2022-03-17 08:33:11 PDT
Remove unused telemetry from Networking.sb on iOS
Attachments
Patch (15.28 KB, patch)
2022-03-17 09:02 PDT, Adam Mazander 		no flags
DetailsFormatted DiffDiff
Patch (16.36 KB, patch)
2022-03-17 10:00 PDT, Adam Mazander 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Adam Mazander
Comment 1 2022-03-17 09:02:27 PDT
Created attachment 454979 [details] Patch
Adam Mazander
Comment 2 2022-03-17 09:06:05 PDT
<rdar://90431433>
Per Arne Vollan
Comment 3 2022-03-17 09:13:06 PDT
Comment on attachment 454979 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=454979&action=review > Source/WebKit/Resources/SandboxProfiles/ios/com.apple.WebKit.Networking.sb:231 > (allow network-outbound (with telemetry) (apply require-any filters)))) It does not look like all telemetry is removed. Should we remove the remaining ones as well?
Adam Mazander
Comment 4 2022-03-17 10:00:47 PDT
Created attachment 454985 [details] Patch
Adam Mazander
Comment 5 2022-03-17 10:02:19 PDT
Yes we have enough data, thanks Per
Per Arne Vollan
Comment 6 2022-03-17 10:04:09 PDT
Comment on attachment 454985 [details] Patch R=me.
EWS
Comment 7 2022-03-17 13:00:01 PDT
Found 1 new test failure: imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-050.html
EWS
Comment 8 2022-03-17 13:15:48 PDT
Committed r291433 (248558@main): <https://commits.webkit.org/248558@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 454985 [details].
Note You need to log in before you can comment on or make changes to this bug.