Bug 231553

Summary: PCM: Cherry-picking patch for branch (Oct 2021)
Product: WebKit Reporter: John Wilander <wilander>
Component: WebKit Misc.Assignee: John Wilander <wilander>
Status: NEW    
Severity: Normal CC: achristensen, ahmad.saleem792, katherine_cheney, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Cherry-pick patch
wilander: commit-queue-
Patch with better explanation of the fix for the test case. achristensen: review+, wilander: commit-queue-

John Wilander
Reported 2021-10-11 21:13:56 PDT
This is just a bug to allow for review of a branch-specific patch. It won't apply to trunk.
Attachments
Cherry-pick patch (66.03 KB, patch)
2021-10-11 21:16 PDT, John Wilander 		wilander: commit-queue-
DetailsFormatted DiffDiff
Patch with better explanation of the fix for the test case. (66.33 KB, patch)
2021-10-12 08:16 PDT, John Wilander 		achristensen: review+
wilander: commit-queue-
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2021-10-11 21:14:30 PDT
<rdar://problem/84130892>
John Wilander
Comment 2 2021-10-11 21:16:41 PDT
Created attachment 440883 [details] Cherry-pick patch
Alex Christensen
Comment 3 2021-10-11 21:46:56 PDT
Comment on attachment 440883 [details] Cherry-pick patch View in context: https://bugs.webkit.org/attachment.cgi?id=440883&action=review > LayoutTests/ChangeLog:7 > + Plus a fix of an API test that was timing out. Could you describe this change more? What part of the patch is this? Why was it needed? What was missed before?
John Wilander
Comment 4 2021-10-12 08:15:15 PDT
Comment on attachment 440883 [details] Cherry-pick patch New patch with explanation for the fixed test case coming up.
John Wilander
Comment 5 2021-10-12 08:16:08 PDT
Created attachment 440935 [details] Patch with better explanation of the fix for the test case.
John Wilander
Comment 6 2021-10-12 08:17:41 PDT
See the Tools change log for the updated description.
Alex Christensen
Comment 7 2021-10-12 08:37:52 PDT
Comment on attachment 440935 [details] Patch with better explanation of the fix for the test case. This is good. The parts of the original change in PrivateClickMeasurementManagerInterface and PrivateClickMeasurementManagerProxy are not needed because those abstractions don't exist on the branch, but this does all the functional change of the original. The included interface simplification does not change behavior. r=me
Kate Cheney
Comment 8 2021-10-12 08:55:22 PDT
Comment on attachment 440935 [details] Patch with better explanation of the fix for the test case. View in context: https://bugs.webkit.org/attachment.cgi?id=440935&action=review LGTM > Source/WebKit/NetworkProcess/NetworkSession.h:140 > + void setPrivateClickMeasurementAppBundleIDForTesting(String&& appBundleIDForTesting); unnecessary parameter name.
John Wilander
Comment 9 2021-10-12 10:59:16 PDT
(In reply to Kate Cheney from comment #8) > Comment on attachment 440935 [details] > Patch with better explanation of the fix for the test case. > > View in context: > https://bugs.webkit.org/attachment.cgi?id=440935&action=review > > LGTM > > > Source/WebKit/NetworkProcess/NetworkSession.h:140 > > + void setPrivateClickMeasurementAppBundleIDForTesting(String&& appBundleIDForTesting); > > unnecessary parameter name. Fixed. Thanks, both of you! I appreciate it.
Note You need to log in before you can comment on or make changes to this bug.