| 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
Per Arne Vollan
2021-03-10 12:07:29 PST
Created attachment 422853 [details]
Patch
Created attachment 422857 [details]
Patch
Created attachment 422859 [details]
Patch
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.)
(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! Created attachment 422862 [details]
Patch
|