Bug 223035

Summary: [iOS] Add additional telemetry to WebContent sandbox
Product: WebKit Reporter: Per Arne Vollan <pvollan>
Component: WebKit Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, ggaren, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
ggaren: review+
Patch none

Per Arne Vollan
Reported 2021-03-10 12:07:29 PST
Add additional telemetry to WebContent sandbox on iOS.
Attachments
Patch (7.60 KB, patch)
2021-03-10 12:13 PST, Per Arne Vollan 		no flags
DetailsFormatted DiffDiff
Patch (7.84 KB, patch)
2021-03-10 12:35 PST, Per Arne Vollan 		no flags
DetailsFormatted DiffDiff
Patch (7.84 KB, patch)
2021-03-10 12:36 PST, Per Arne Vollan 		ggaren: review+
DetailsFormatted DiffDiff
Patch (7.41 KB, patch)
2021-03-10 12:53 PST, Per Arne Vollan 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Per Arne Vollan
Comment 1 2021-03-10 12:08:01 PST
<rdar://75275161>
Per Arne Vollan
Comment 2 2021-03-10 12:13:52 PST
Created attachment 422853 [details] Patch
Per Arne Vollan
Comment 3 2021-03-10 12:35:49 PST
Created attachment 422857 [details] Patch
Per Arne Vollan
Comment 4 2021-03-10 12:36:58 PST
Created attachment 422859 [details] Patch
Geoffrey Garen
Comment 5 2021-03-10 12:38:02 PST
Comment on attachment 422859 [details] Patch r=me Can you remove the "XXX" comments before landing? I find bug reports in code to be distracting; and I think a bug report is a better way to track future work. If any of these comments warrant a bug report, let's file it. (It seems like the task to deny access if telemetry says that's OK probably warrants a bug report. I'm not sure that the rest do.)
Per Arne Vollan
Comment 6 2021-03-10 12:48:23 PST
(In reply to Geoffrey Garen from comment #5) > Comment on attachment 422859 [details] > Patch > > r=me > > Can you remove the "XXX" comments before landing? I find bug reports in code > to be distracting; and I think a bug report is a better way to track future > work. If any of these comments warrant a bug report, let's file it. (It > seems like the task to deny access if telemetry says that's OK probably > warrants a bug report. I'm not sure that the rest do.) Will do, thanks for reviewing!
Per Arne Vollan
Comment 7 2021-03-10 12:53:36 PST
Created attachment 422862 [details] Patch
Per Arne Vollan
Comment 8 2021-03-10 13:09:05 PST
Landed <https://trac.webkit.org/changeset/274231/webkit>.
Note You need to log in before you can comment on or make changes to this bug.