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-

Description John Wilander 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.
Comment 1 Radar WebKit Bug Importer 2021-10-11 21:14:30 PDT 
<rdar://problem/84130892>
Comment 2 John Wilander 2021-10-11 21:16:41 PDT 
Created attachment 440883 [details]
Cherry-pick patch
Comment 3 Alex Christensen 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?
Comment 4 John Wilander 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.
Comment 5 John Wilander 2021-10-12 08:16:08 PDT 
Created attachment 440935 [details]
Patch with better explanation of the fix for the test case.
Comment 6 John Wilander 2021-10-12 08:17:41 PDT 
See the Tools change log for the updated description.
Comment 7 Alex Christensen 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
Comment 8 Kate Cheney 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.
Comment 9 John Wilander 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.